Improve unsaved changes modal
See original GitHub issueThere’s a few issues with the unsaved changes modal that we should clean up when we have a chance.
- Simply providing
YesNobuttons isn’t clear enough. We should provide more context in the button label, e.g.Yes, closeorNo, don't close. - The
Yes, closebutton should be on the right side as it implies forward progress, whileNo, don't closetakes you back. - I cleaned up the styling of the modal a bit (buttons are larger and headline text is smaller). It feels more balanced this way.
Current

Ideal

Issue Analytics
- State:
- Created 5 years ago
- Comments:16 (16 by maintainers)
Top Results From Across the Web
What is a good approach for warning user on leaving a modal ...
I would recommend having a pop-up modal asking the user that there is unsaved information that he will lose. It's weird to click...
Read more >Communicating unsaved changes - Cloudscape Design System
A confirmation modal prevents data loss when unsaved changes exist on the page. However, it's not necessary if a user hasn't made any...
Read more >How to prevent cancel modal if there is unsaved changes in ...
My Angular app creates a modal dialog to allow user to input and change some properties. It is requested to warn user that...
Read more >Modal Dialog Form: UX improvements relating to closing of ...
If/when a user fills in data in a Modal Form, and accidentally click outside the Modal, the form will be closed and any...
Read more >Unsaved changes modal UX pattern · FOLIO UX docs
Unsaved changes modal UX pattern. This modal is a confirmation modal that prevents users from leaving a create/edit record page without saving changes....
Read more >
Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free
Top Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found

@jashot7 sorry for the delay. Tested this PR today and overall it’s looking really great. Only suggestion I have is to flip the buttons on the
restartdialog. (Restart nowshould be on the right side)The
OpenBazaar Wallet Setupscreen will be going away soon so I’m fine with the HR changes there. Keeping onboarding as is works for me too.👍
Ok thanks @jashot7 I’ll catch up on this when I can. Hang tight