В окне чата активен скелетон-лоадер

Привет,

Я разместил это в ux, потому что это работает, но, похоже, активно только в боковой панели, и был коммит, показывающий скелетон только при первоначальной загрузке. PERF: applies optimisations on chat-live pane by jjaffeux · Pull Request #20532 · discourse/discourse · GitHub. Также, похоже, есть проблема с позиционированием, но это может быть вызвано загрузчиком.

Mar-16-2023 18-47-39

Спасибо :slightly_smiling_face:

2 лайка

Я немного не понимаю, какое именно ваше ожидание здесь не оправдывается.

Вы видите скелет, когда не ожидали его увидеть? Или вы ожидали увидеть скелет в каком-то случае, но не видите его?

2 лайка

На полной странице чата (на рабочем столе или мобильном устройстве) скелет не используется из-за соображений производительности.

Можно ли сделать такую же иконку загрузки скелета для обычных страниц?

О, хорошо. Я думаю, что понимаю.

Сам скелет — это «функция». Мы показываем его, пока сообщения ещё не загружены, чтобы улучшить опыт прокрутки.

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

Именно это вы имеете в виду?

Я считаю, что скелетон должен быть активен только при первоначальной загрузке при открытии канала. В коммите, на который я ссылался выше, в описании указано: «показывать скелетон только при первоначальной загрузке», что работает корректно как в полном чате на десктопе, так и на мобильных устройствах. Только панель (drawer) сохраняет скелетон для этой цели, и поэтому я считаю, что это не сделано намеренно. :slightly_smiling_face:

Сейчас он используется всегда, когда вы нажимаете, чтобы показать ответ или ссылку на предыдущее сообщение в том же канале, даже если сообщение уже загружено. Вот пример: отвеченное сообщение уже загружено.

Mar-17-2023 07-12-28

1 лайк

Я почти уверен, что исправил это на днях. Я только что проверил, и при нажатии на «Ответить» полной перезагрузки страницы не произошло.

РЕДАКТИРОВАНИЕ: apparently это работает на мобильных устройствах, но не на настольных компьютерах, странно. Разберусь.

4 лайка

Это должно было быть исправлено :+1:

2 лайка

Эта тема была автоматически закрыта через 2 дня после последнего ответа. Новые ответы больше не принимаются.