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
feat: show delete confirmation dialog when clicking "Delete & re-draft" menu #2296
feat: show delete confirmation dialog when clicking "Delete & re-draft" menu #2296
Conversation
Run & review this pull request in StackBlitz Codeflow. |
✅ Deploy Preview for elk-zone ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for elk-docs canceled.
|
Ah, it seems that the CI error is not related to this change. It started from this commit: e6172ad in the |
What's the default button focused when shown (if any)? We should focus the cancel button. Can you add Maybe the dialog component requires a review. |
@userquin Yeah I agree we should revise the design of the confirm dialog. It's not blocking this PR tho. |
Thank you both! I also agree on the above suggestions will improve the dialog UI. I'm going to try to make a separate PR to adjust the layout and add a title to the dialog later. 🙂 |
resolve #2291
This change ensures to show the same dialog message as the "Delete" menu. I'm wondering if we'd better change the message but basically, both actions are warning about the same delete action so I assume the same message can be acceptable.