Don
13.Ноябрь.2024 06:23:46
1
Привет
Я обнаружил несколько мест, где, по моему мнению, также следует скрывать онлайн-статус.
Статус в чате: когда вы пишете сообщение, он отображается внизу, даже если вы скрыли свой статус.
Думаю, в сводке профиля пользователя в разделе статистики секция «Последний раз был(а)» также должна скрываться для пользователей, если активирована функция скрытия статуса.
Было бы хорошо, если бы это работало так же, как в пункте 2, с компонентом темы «Последний раз был(а)» User Card .
Спасибо
4 лайка
lindsey
21.Ноябрь.2024 15:35:08
2
Спасибо за эти идеи! Я перенесу это в #feature , так как они предполагают расширение нашей текущей функциональности за пределы корректировок UX.
1 лайк
Don:
Присутствие в чате: когда вы пишете сообщение, оно отображается внизу, даже если вы скрыли своё присутствие.
Это будет исправлено в
main ← fix-is-replying-when-offline
closed 02:00AM - 17 Feb 25 UTC
While trying to reproduce an issue reported internally where a user saw the "is … replying..." from another user who had disabled presence (aka. was "offline"), I noticed something odd about the way we "notify" presence via `notifyState`. The way it is setup up, it's being called on every "input" event in the composer/editor, due to it tracking the "reply" property of the composer model. That's fine, but the way the `helperFn` helper works, make it so we're calling the `on.cleanup` callback also on **every** "input" events in the composer/editor. For every character we enter in the composer, we're "entering" the presence channel and immediatelly "leaving" it...
If I remove the checks on `reply` (and thus the auto-tracking), then the `notifyState` function is called much less frequently. But we have some specs that expects us not to "publish" our presence when we open the composer and the reply is "empty".
Note: I switched to using `replyDirty` instead to re-use the checks the composer model is already doing to ensure we actually have changed something in the reply.
The best I came up with was not to rely on the `helperFn` but now, there's no way (that I could find) to call `composerPresenceManager.leave()` when the composer is closed. I tried a lot of different ways, but none worked. I'm sure it's something stupid easy, but I can't figure it out just now.
Internal ref - t/127490/74
(как только мне удастся заставить это работать должным образом
2 лайка
chat
committed 11:21AM - 04 Dec 24 UTC
...since it was mostly duplicating the work the "ComposerPresenceManager" was do… ing.
So now the #chat composer uses the same "presence manager" as the composer, benefiting from the "hide presence" checks, with the only difference that the "keep alive" timeout is 5s for chat and 10s for topics/posts.
4 лайка
)