Карточки тем: для перехода в тему «Новые и непрочитанные» или темы активности пользователя нужно кликнуть по заголовку

Приоритет/Серьезность: Средний

Платформа: Любая платформа с темой Horizon

Описание: На некоторых страницах, включая Новые и непрочитанные темы или темы активности пользователя, навигация по темам не работает при клике на пустые области карточек тем. Пользователям необходимо кликать на заголовок, чтобы открыть тему.

Шаги для воспроизведения:

  1. Откройте https://try.discourse.org/t/welcome-to-our-demo/57 и кликните на пустые области любой темы ниже раздела Новые и непрочитанные темы.
  2. Откройте Profile - samsaffron - Discourse Demo и кликните на пустые области первой темы.
  3. Перейдите на главную страницу https://try.discourse.org/ и кликните на пустые области некоторых тем.

В случаях 1 и 2 навигация не происходит, если не кликнуть непосредственно на текст заголовка. В случае 3 клик в любом месте карточки темы ведет к открытию этой темы.

Анализ:

В файле discourse/themes/horizon/javascripts/discourse/initializers/topic-list-columns.gjs at main · discourse/discourse · GitHub некоторые значения currentRouteName не включены в проверку и не обрабатываются корректно, включая topic.fromParams, topic.fromParamsNear и userActivity.topics. Не совсем ясно, должны ли они быть частью isHighContextRoute.

1 лайк

Это исправлено в DEV: Refactor Horizon topic cards to use listContext instead of route checks by ZogStriP · Pull Request #37355 · discourse/discourse.

1 лайк