Le bouton Retour dans Chat oublie si la conversation est étoilée

zogstrip · Mai 21, 2026, 11:25

Ça devrait être mieux une fois que cela sera fusionné

github.com/discourse/discourse

FIX: Chat back button loses starred context after drawer reopen (#40219)

main ← fix-chat-back-button-starred

opened 11:24AM - 21 May 26 UTC

ZogStriP

+66 -0

When the chat drawer is closed and reopened on a starred channel, the back butto…n took the user to the default channels list instead of the starred-channels list they had originally come from. Root cause: `chatHistory.visit(route)` always appended to history. When the drawer reopens to `lastKnownChatURL` (the channel itself), the same route is appended a second time, shifting `previousRoute` off the original `chat.starred-channels` entry. The back-button fallback in `drawer-routes/channel.gjs` then routed to `chat.channels`. Fix: dedupe consecutive identical route visits in `chatHistory.visit` by comparing name and params (via `deepEqual`). Reopening to the same URL no longer clobbers the user's real navigation context. Adds a unit test for the dedup behavior, a system spec covering the close/reopen scenario, and a view-agnostic `open_channel_row` helper on the drawer page object (the existing `open_channel` is scoped to the channels-list view only and doesn't work from the starred view). https://meta.discourse.org/t/403431

Sujet		Réponses	Vues
Suggestion: add a button to switch back from fullscreen to small chat UX chat	17	1470	Mai 25, 2022
Chat opens to private frontpage of channel UX chat	4	387	Janvier 17, 2024
Star your favorite chat channels for easy access Announcements chat	1	219	Juin 21, 2026
Open chat by default or remember if it was open Feature chat	1	403	Décembre 17, 2022
When chat is expanded there's no "<" back button Support chat	2	71	Janvier 16, 2026

Le bouton Retour dans Chat oublie si la conversation est étoilée

Sujets connexes