Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revise notification dismissal on client-side navigation #82

Open
tracy-french opened this issue Apr 4, 2023 · 0 comments
Open

Revise notification dismissal on client-side navigation #82

tracy-french opened this issue Apr 4, 2023 · 0 comments
Labels
debt Something we need to pay back low effort Can be completed in a few hours or less med priority Medium priority papercut Minor UX change requested

Comments

@tracy-french
Copy link
Contributor

Notifications are expected to have the UX behavior (defined by Cloudscape)[https://cloudscape.design/components/flashbar/?tabId=usage].

There are two issues with our implementation:

  1. A side-effect occurs. Users of the useApplication hook must understand that navigate dismisses notifications. This is problematic when creating a dashboard, as a navigation needs to occur and then a success notification needs to appear. If the emission of the notification occurs before navigation, the notification will be dismissed. (note: this behavior is documented in the useApplication hook.
  2. navigate dismisses all notifications, but the UX guidance is to persist notifications relevant to pages across navigation. This will require adding context to the notification structure about which pages it should be visible on.
@tracy-french tracy-french mentioned this issue Apr 4, 2023
9 tasks
@tracy-french tracy-french added papercut Minor UX change requested debt Something we need to pay back low effort Can be completed in a few hours or less high priority High priority labels Apr 6, 2023
@tracy-french tracy-french added med priority Medium priority and removed high priority High priority labels May 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
debt Something we need to pay back low effort Can be completed in a few hours or less med priority Medium priority papercut Minor UX change requested
Projects
None yet
Development

No branches or pull requests

1 participant