parse-dashboard
parse-dashboard copied to clipboard
Change row batch deletion confirmation entry
New Issue Checklist
- [x] I am not disclosing a vulnerability.
- [x] I am not just asking a question.
- [x] I have searched through existing issues.
- [x] I can reproduce the issue with the latest versions of Parse Server and Parse Dashboard.
Issue Description
This is a UX issue. The confirmation dialog to delete selected rows, to delete all rows and to delete the whole class require to enter the class name. All 3 dialogs look similar. The issue is that the dialogs may be mistaken for each other. Also the buttons in the menu to perform these actions are next to each other.




Steps to reproduce
- Delete selected rows
- Delete all rows
- Delete class
Actual Outcome
Described above
Expected Outcome
Make the dialogs more distinguishable, for example:
- The dialog to delete N rows should not request to enter the class name but the word "delete selected"
- The dialog to delete all rows should not request to enter the class name but the word "delete all".
- The dialog to delete the whole class should stay as is and request to enter the class name.
Environment
Dashboard
- Parse Dashboard version:
4.1.4
Logs
n/a
Thanks for opening this issue!
- 🚀 You can help us to fix this issue faster by opening a pull request with a failing test. See our Contribution Guide for how to make a pull request, or read our New Contributor's Guide if this is your first time contributing.
@mtrezza Can I please take a stab at this issue?
Please feel free, it doesn't seem that anyone is working on this as there's no comment
cc @Magdaraa
Thank you @mtrezza . Here's the pr https://github.com/parse-community/parse-dashboard/pull/2319. Please let me know about your thoughts.
Closing via #2319