Richie
07.Июль.2023 06:55:01
1
Обновлено до версии 3.1.0.beta5 (3ea8203719 ) сегодня.
Сразу заметил, что при просмотре страницы /latest и прокрутке списка тем, нажатие на логотип в левом верхнем углу больше не возвращает меня к началу страницы
Вместо этого страница теперь просто перезагружается на текущей позиции прокрутки.
2 лайка
Richie
07.Июль.2023 07:18:39
2
Извините, забыл добавить: Safari на iOS 16.5.1
1 лайк
Richie
07.Июль.2023 08:39:59
6
Я также могу воспроизвести это на рабочем столе Windows в Firefox 114.x
Опять же, это происходит только на странице /latest, когда вы немного прокрутили вниз.
2 лайка
david
12.Июль.2023 12:18:30
10
У меня уже есть решение в работе:
main ← scroll-manager
merged 12:40PM - 13 Jul 23 UTC
Previously we were implementing scroll reset/memorization on a per-page basis. M… any of these approaches relied on the `didInsertElement` hook, which is no longer appropriate since Discourse changed to use the 'loading slider' strategy for page transitions.
This commit rips out all of our custom scroll resetting/memorizing, and implements those things in a generic service. There are two features:
1. After every route transition, scroll to the top of the page
2. When using browser back/forward buttons, restore the last known scroll position for those routes
Я обновлю информацию здесь, как только оно будет слито.
Редактирование: ветка теперь развернута на Meta для тестирования.
7 лайков
Richie
12.Июль.2023 19:23:42
11
Тесты здесь на Meta пройдены, спасибо за исправление бага @Дэвид
3 лайка
david
14.Июль.2023 07:00:48
13
Эта тема была автоматически закрыта через 17 часов. Новые ответы больше не принимаются.
