-
Notifications
You must be signed in to change notification settings - Fork 102
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
Close all button for notifications #17
Comments
Regarding UX, maybe it could be cmd-click? I see there aren't any key combos in the project yet. https://package.elm-lang.org/packages/ohanhi/keyboard/latest/Keyboard could be an option. |
Hi, in fact there is a close button but it's on the right, quite small and sometimes not really visible 😞 An cmd-click option could be fine also (but not sure how to tell it to users ^^). Will be happy to review a PR from you 👍 |
Hi @hansvb Thanks for your comment. My current take is to move away from this "quick and dirty" UI and open a report modal at the end of the import with some logs inside, so it's clearer for you to see what happened. What do you think about this? |
Parsing my postgres schema generated a large number of errors. None of them were important so I was happy to ignore them all but I was left with a large number of notifications to close. A close all button would be really handy.
(happy to open a PR if we agree the design)
The text was updated successfully, but these errors were encountered: