-
Notifications
You must be signed in to change notification settings - Fork 6
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
Success Alerts in Record App #2524
Labels
discussion required
requires a discussion before moving forward
help wanted
Intended for internal use for student workers
Comments
Here are the solutions we discussed and how they address the two main concerns: (1) avoiding content shift, and (2) not reducing the workspace. 1. Grouping alerts into a collapsible alert:
2. Adding a "Delete All" option:
3. Fade-away stacked alerts:
4. Having a 'Donot notify me' option to suppress alerts:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
discussion required
requires a discussion before moving forward
help wanted
Intended for internal use for student workers
Background
The current implementation of record app, shows success alerts when a pure and binary table has records added to it ("Link records" was used). Other actions on record app that modify the contents of the page don't show an alert. Those actions include:
This behavior to show an alert is removed in issue #2273 to be consistent with what we are currently doing for these other actions.
Task
This issue is for improving the "alerts" on record app to communicate changes more accurately when a user uses the actions on record page that "refresh" content for the user. The content that is refreshed happens "silently" with spinners on the sections that are updated but could easily be missed.
The alert should include information about:
reference.displayname
that was used in the success alert for P&B beforeAlerts should be added when the page is focused again and we are "fetching" data. This happens for
Other UX changes
If we have multiple alerts on the page, this can eventually change the main content area to be very small making it hard to interact with the app unless the user clears the alerts. We need to explore the following:
other pages/places
We have other pages where content can be added/updated where the main page opens a tab and then refreshes then content when focused again. These places include:
The text was updated successfully, but these errors were encountered: