Проблема с `topic-progress-wrapper` в RTL

Внести вклад Ошибка
1

Привет, dear discourse,

В режиме RTL (справа налево), когда вы переходите в конец страницы темы, класс topic-progress-wrapper создаёт загромождение на странице. Пожалуйста, проверьте этот вопрос.

Если закомментировать следующие значения, проблема будет решена:

@media screen and (min-width: 925px)
.container.posts .topic-navigation {
    grid-area: timeline;
    align-self: start;
    /* position: -webkit-sticky;
       position: sticky; */
    top: 6em;
    margin-right: 1em;
    z-index: 100;
}
2

Привет, @Nildarar

Не могли бы вы прислать скриншот проблемы?

3

На скриншоте не показано. Позвольте мне объяснить подробнее. Чтобы увидеть проблему, выполните следующие действия:

  1. Установите для шаблона направление справа налево (RTL).
  2. Перейдите в мобильный режим просмотра.
  3. Увеличьте ширину экрана до значения более 925 пикселей.
  4. Прокрутите страницу до конца темы.
  5. Модуль Прогресс темы исчезает с экрана, а страница начинает прокручиваться по горизонтали.

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

1 лайк
4

Хорошо, это симптом более серьёзной проблемы. Корень проблемы в том, что Discourse должен отображаться в режиме для настольных компьютеров, а не в мобильном режиме, когда ширина экрана ≥ 925 пикселей. Скажите, на каких устройствах вы наблюдаете эту проблему?

Стоит упомянуть, что удаление position: sticky, возможно, решит эту конкретную проблему, но это сломает таймлайн темы (он не будет прокручиваться вниз при прокрутке темы).

Косвенно не связано с этим, но я заметил, что индикатор прогресса темы должен располагаться слева, а не справа в макете RTL. Я отправил PR по этому поводу здесь: FIX: Topic progress bar should be on the left in RTL layouts by OsamaSayegh · Pull Request #11999 · discourse/discourse · GitHub.

7 лайков
5

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

Спасибо за это исправление :+1:

2 лайка
6

Объединено, большое спасибо за сообщение. Надеюсь, это исправит проблему.

3 лайка