• vort3
    vort3
    web poll Juick Жуйк, голосование.

    В браузерной версии в глобальной панельке навигации сверху текущий раздел сайта — это не активная ссылка. В том смысле, что находясь на вкладке Discover (простите, не помню как в русском интерфейсе) нельзя нажать там снова на кнопку Discover, чтобы обновить страницу. На остальные две кнопки нажать можно, а на эту нельзя, так как ты уже на ней находишься. Поэтому, чтобы обновить ленту, надо использовать функцию обновления страницы в браузере.

    Предлагаю патч, чтобы отключить это поведение и сделать три кнопки в этой панельке всегда активными, чтобы нажимая на кнопку того раздела, в котором ты уже находишься, можно было его обновить.

    Есть возражения? Дискасс.
    ♡ recommended by @vt
  • Shura
    Shura
    Я — за. Меня бесит то, что нажатие на кнопку вызывает выделение и выскакивающее меню
    /1 ·
  • glupovat
    glupovat
    · Edited
    патч не приложился, ревьюить нечего
    /2 ·
  • vt
    vt
    @glupovat, Он прислал, там все ок
    /3 in reply to /2 ·
  • glupovat
    glupovat
    @vt, тогда накатывай
    /4 in reply to /3 ·
  • vt
    vt
    @glupovat, Я пытаюсь вспомнить из каких соображений это было сделано, чуть позже
    /5 in reply to /4 ·
  • alex0b
    alex0b
    Да пофик! Кати на прод)
    /6 ·
  • vt
    vt
    @alex0b, Рекомендую этот комментарий
    /7 in reply to /6 ·
  • vort3
    vort3
    @glupovat, Ревьювь:

    p.vort3.com
    /8 in reply to /2 ·
  • vort3
    vort3
    @Shura, Вот такая же тема, лучше бы это было активной ссылкой, имхо.
    /9 in reply to /1 ·
  • vt
    vt
    @vort3, Я выкатил, но ничего не изменилось, активной ссылкой это не стало
    А disabled там было видимо для предотвращения какого-то глюка браузера, а не для отключения активности. Отключено где-то еще, например в стиле active
    /10 in reply to /9 ·
  • vt
    vt
    · Edited
    @Shura, А вот мы и вспомнили проблему!
    stackoverflow.com
    И вот так в этих мобильных браузерах все, не пользуйтесь этим говном, ну плиз
    /11 in reply to /1 ·
  • vort3
    vort3
    @vt, Отправил ещё патч, оно зачем-то дополнительно было ещё и через CSS выключено. Мне было X лет, когда я узнал, что CSS умеет выключать ссылки. Чекай.
    /12 in reply to /10 ·
  • vort3
    vort3
    @vt,
    Everything works fine on IE, Firefox and Safari […] but not on Chrome

    Да, спасибо, я и не пользуюсь хромом. А на фуррифокс ЕМНИП обычно ты бочку катишь.
    /13 in reply to /11 ·
  • vort3
    vort3
    @vt, А ты точно выкатил? Я не вижу в гите свой патч. Или ты не оттуда выкатил?
    /14 in reply to /10 ·
  • vt
    vt
    @vort3, Я просто выкатил то, что у меня локально сейчас, если все ок, тогда пушу в гит
    /15 in reply to /14 ·
  • vort3
    vort3
    @vt, Попробуй второй патч применить локально, чекнем, я думаю должно сработать.
    /16 in reply to /15 ·
  • Shura
    Shura
    @vt, кстати на нижней панели такой проблемы нет
    /17 in reply to /11 ·
  • vt
    vt
    @Shura, Ну правильно, хром колбасит от того, что два active-элемента и он отказывается сбрасывать выделение
    /18 in reply to /17 ·
  • vort3
    vort3
    @Shura, Я знаю. Только к верхнему навбару применялась функция, которая отключает активность ссылки для текущей страницы.
    /19 in reply to /17 ·
  • Shura
    Shura
    @vt, У меня firefox
    /20 in reply to /18 ·
  • vt
    vt
    @Shura, ну значит в мобильном варианте проблема есть и у него
    /21 in reply to /20 ·
  • vort3
    vort3
    @vt, Там document.querySelectorAll('#header_wrapper a'), таких элемента всего 4, для которых отключается активность ссылки, если текущая активная страница совпадает со ссылкой. Это вся верхняя панель. Там не может быть несколько active элементов, поскольку только один из них может вести на ту страницу, которая уже открыта в браузере.
    /22 in reply to /18 ·
  • vt
    vt
    @vort3, А ты залогинен? Там в каком-то случае discover есть в обоих панелях
    /23 in reply to /22 ·
  • vort3
    vort3
    @vt, Я? Да. Но скрипт ищет элементы только в #header_wrapper.
    /24 in reply to /23 ·
  • vort3
    vort3
    @vt, Ты CSS патч применил? Я не пойму никак, то ли второй патч не сработал опять, то ли ты просто ещё не выкатил его.
    /25 in reply to /23 ·
  • vt
    vt
    @vort3, Да проблема не в disabled, а изначально в том, что два одинаковых id на странице, и один из них выделяется. В одних случаях disabled помогает сбросить выделение, в других — нет
    /26 in reply to /24 ·
  • vt
    vt
    @vort3, Нет еще
    /27 in reply to /25 ·
  • vort3
    vort3
    @vt, Свойства disabled уже нет там, где оно было. Проблема не в нём, а в css свойстве pointer-events.
    /28 in reply to /26 ·
  • Shura
    Shura
    @vt, с колокольчиком та же проблема — в верхней панели выделяется
    /29 in reply to /26 ·
  • vort3
    vort3
    @Shura, Это работает с четырьмя кнопками в верхней панели. Сейчас @vt накатит второй патч и проверим, пофиксилось ли. Там оказалось сложнее, чем я думал.
    /30 in reply to /29 ·
  • lurker
    lurker
    Вт уже предлагали, он сказал что не будет менять т к так и задумано
    /31 ·
  • vort3
    vort3
    @lurker, А мне он ответил, что надо спросить у юзеров.
    /32 in reply to /31 ·
  • vt
    vt
    @lurker, Мне уже насрать, потому что я теперь уверен, что мобильные браузеры это наебалово, а сделано это было для них
    /33 in reply to /31 ·
  • vort3
    vort3
    @lurker, Вот видишь, люди меняются.
    /34 in reply to /31 ·
  • vt
    vt
    @vort3, Мы про разные проблемы — pointer events это отключение ссылки как таковой. А у @Shura проблема в том, что появляется выделение. И оно появится и даже когда ссылка активна, скорее всего
    /35 in reply to /28 ·
  • vort3
    vort3
    @vt, Я может свяжусь с @Shura и разузнаю, можно ли это пофиксить.

    Ты CSS патч применять собираешься или нет? Я хочу со спокойной душой пойти спать, если это сработало, или фиксить дальше. Скоро час ночи так-то.
    /36 in reply to /35 ·
  • vt
    vt
    @vort3, Я так-то еще работаю, сел применять патч, запускаю, и тут вижу в логе, что у нас две недели сломан топ, лол. Запустил, можно проверять, иду чинить топ
    /37 in reply to /36 ·
  • vort3
    vort3
    @vt, У кого у нас, какой топ? top который как htop показывает процессы, или в смысле топ постов?

    Проверил, работает мой патч, спасяб. Если ничего не сломается, можно будет в гит закинуть.

    А ты патч с отключением login with telegram не закидывал сюда же? Попробую разлогиниться и проверить.
    /38 in reply to /37 ·
  • vt
    vt
    @vort3, Топ постов остановился 2 недели назад
    /39 in reply to /38 ·
  • vort3
    vort3
    @vt, Забавно. Я туда и не захожу, просто глобальную ленту читаю, практически всю, кроме ситуаций, когда я на два дня выпал из жизни и в ленте сотня новых постов.
    /40 in reply to /39 ·
  • Shura
    Shura
    @vort3, Работает!
    /41 in reply to /36 ·
  • vort3
    vort3
    @Shura, Что именно работает, перестали кнопки выделяться как текст?
    /42 in reply to /41 ·
  • Shura
    Shura
    @vort3, Да
    /43 in reply to /42 ·
  • vort3
    vort3
    @Shura, Огонь, спасяб за отзыв.
    /44 in reply to /43 ·