fleet/changes/22661-improve-flash-message-ux at c797fb73c2618ececa1a46bfac44cb9ea6a646c0 - Elgato_dark/fleet - Forgejo: Beyond coding. We Forge.

Elgato_dark/fleet

mirror of https://github.com/fleetdm/fleet synced 2026-05-23 08:58:41 +00:00

jacobshandling d3ccb51755

UI - Improve UX of Flash messages (#22836 )

## #22661 


![ezgif-6-71e48912ae](https://github.com/user-attachments/assets/01144620-0eba-48f0-9254-cc4795fde9fd)

- Update `FlashMessage` behavior to, by default, hide itself when the
user performs any URL-changing navigation
- Add `persistOnPageChange` option to `renderFlash` API and associated
notification context and reducer logic, allowing override of this
behavior on a per-call basis
- Ensure proper order of evaluation of URL changes and render flash
action dispatches on the event loop
- Clean up legacy unused "undo"-related arguments and logic
- Allow the user to click in the same horizontal dimension as a flash
message
- Other misc. cleanup and refactoring

[Demo - messages hidden on page (any URL)
change](https://www.loom.com/share/1e884b6ba11c4b59bc74f51df3690131?sid=9b53e78b-6535-4541-b676-377760366cf4)

- [x] Changes file added for user-visible changes in `changes/`,
- [x] Manual QA for all new/changed functionality

---------

Co-authored-by: Jacob Shandling <jacob@fleetdm.com>

2024-10-22 10:52:20 -07:00

4 lines

No EOL

336 B

Text

Raw Blame History

 * Allow clicking in the horizontal dimension of a flash message, outside of the message itself, and
 always hide flash messages when changing routes.
 * Refactor `renderFlash` and associated logic to by default dismiss the rendered message on page
   change. Provide optional `persistOnPageChange` option to allow overriding this behavior.