Проект

Общее

Профиль

Ошибка #31

Количество зрителей отображается крассным цветом, если страница стрима была открыта до начала трансляции

Добавил(а) Павел Рюмин больше 2 лет назад. Обновлено больше 2 лет назад.

Статус:
Решена
Приоритет:
Нормальный
Назначена:
Александр Трошенков
Дата начала:
01.03.2017
Дата завершения:
Готовность:

60%

Трудозатраты:
Браузер:
Ссылка на картинку:

История

#1 Обновлено Александр Трошенков больше 2 лет назад

  • Параметр Статус изменился с Новая на Решена

У меня не было красного цвета.
Либо я не туда зашел :)
На всякий случай, я кинул !important на элементы с кол-вом зрителей.

https://github.com/GoodGame/Main/tree/wifi-issue31

layout\templates\streams\includes\channel\info.twig
public\css\sass\_channel-page.scss

#2 Обновлено Павел Рюмин больше 2 лет назад

  • Параметр Статус изменился с Решена на Новая

Правильное поведение:
https://puu.sh/usAQS.png

Неправильное поведение (баг):
https://puu.sh/usCLm.png

Как воспроизвести:
Заходишь на свой канал, пока он офлайн, видишь под плеером красную надпись ОФЛАЙН. Запускаешь стрим и ждешь, пока появится картинка. Вместо надписи ОФЛАЙН появляется количество зрителей красным цветом

#3 Обновлено Александр Трошенков больше 2 лет назад

  • Параметр Статус изменился с Новая на Решена
  • Параметр Готовность изменился с 0 на 50

Готово.
Видеозапись пришлось кинуть в скайп

#4 Обновлено Александр Трошенков больше 2 лет назад

Александр Трошенков писал(а):

lokki7 Павел Рюмин
Человечки еще потерялись:
https://puu.sh/uv7mk.png

Готово

#5 Обновлено Александр Трошенков больше 2 лет назад

  • Параметр Статус изменился с Решена на Не готова

[13:11:38 | Изменены 13:11:52] lokki7 Павел Рюмин: Есть пожелание вот по этой задачке
https://github.com/GoodGame/Main/compare/production...wifi-issue31

Плохо дублировать стиль в js. Ведь если верстальщик будет делать правки на этой странице, то он не будет знать, что нужно еще и тут поправить. В итоге вылезет баг.
Лучше сделай на страничке сразу оба блока, не нужный скрой через display:none
[13:12:03] lokki7 Павел Рюмин: и при смене статуса переключай
[13:12:43] Александр Трошенков: jQuery добавляет класс
[13:13:02] Александр Трошенков: Стили я просто слегка отформатировал, но не трогал )
[13:13:02] lokki7 Павел Рюмин: и <span class="icon icon-users"></span>
[13:13:19] Александр Трошенков: Понял, ок
[13:13:41] lokki7 Павел Рюмин: о, лучше вообще вставь этот <span class="icon icon-users"></span> в блок и сделай ему display:none на .off
[13:13:50] lokki7 Павел Рюмин: тогда он сам будет появляться и скрываться
[13:14:00] lokki7 Павел Рюмин: останется только .removeClass('off').addClass('on')
[13:14:08] Александр Трошенков: Да )

#6 Обновлено Александр Трошенков больше 2 лет назад

  • Параметр Готовность изменился с 50 на 60

Сделал, проверю позже.

#7 Обновлено Александр Трошенков больше 2 лет назад

  • Параметр Статус изменился с Не готова на Решена

#8 Обновлено Александр Трошенков больше 2 лет назад

  • Параметр Статус изменился с Решена на Не готова

#9 Обновлено Александр Трошенков больше 2 лет назад

  • Параметр Статус изменился с Не готова на Решена

Экспортировать в Atom PDF