In the Sprite Editor, the Revert and Apply buttons are directly next to each other and relatively small overall. This makes it possible for users to accidentally click revert or apply when they want to perform the opposite action. I vote for a more forgiving UI design which places these buttons slightly apart from one another so that it prevents accidentally reverting or applying changes.
My team experienced this small issue a couple of times in the last weeks and it can be frustrating, especially when using the Sprite Editor for more complex tasks such as bone editing for 2D animations, especially because undo does not work after reverting accidentally.
If “Revert” can’t be undone, how about a confirmation dialog when pressing “Revert”? Then it should be impossible to accidentally revert, even if the would be UI is further apart.
A confirmation dialog sounds like an option as well. Personally, I’m not sure if this would be needed, since clicking the wrong button is still mainly user error and a confirmation dialog can be come annoying as well. So for me, I would simply try to avoid the mistake of clicking the wrong button by moving the buttons further apart, but I’m also no UX designer, I simply look at all other operation system dialogs etc where yes/no/cancel are clearly spaced out.
Yes, this is something we definitely should look into. I’ve also recently come across another user who also lost quite a bit of work due to the accidental clicking of the revert button. It can be quite a heart-stopping moment, isn’t it?
Both of these suggestions sound like possible solutions. And I’m also wondering if we could enable undo by pressing Ctrl+ z even after clicking “revert” i.e. bringing back all the changes we have made in the Sprite Editor to the state right before we accidentally hit “revert”. I wonder if there are any technical limitations and would love to discuss this with the devs back at the office.
Interestingly, we already have this (optional) confirmation dialog modal made available in the Unity > Preferences settings under the 2D > Sprite Editor Window. Silly me for not realising this and sharing with you guys earlier.
So if we enable that checkbox, every time we hit Revert in the Sprite Editor (in any of these Custom Outline, Custom Physics Shape, Secondary Textures module, Skinning Editor modules), we’ll be greeted with this secondary confirmation dialog.
The is the same for Apply too if we enable that checkbox in the preferences.
So the question is, would you guys prefer if we have the Sprite Editor’s Apply/Revert confirmation dialogs enabledby default in the Preferences? Would that be more helpful? (It’s disabled by default at the moment.)
This looks like an useful change, thank you and thumbs up! My preference would be to enable it by default to be “on the safe side”.
Here are a few suggestions for the UI/UX…
Don’t display just Yes/No on the button, display the “action verb” instead, like it’s done in most dialogs in Unity:
You could also add a note to the dialog that tells the user where to find the option to toggle the confirmation. You could also add another button to disable the dialog like in the dialog. Here is an example of this when you try to debug without having debug mode enabled:
Thank you @Peter77 ! These are great, great suggestions! I’m going to have to feedback these to the team and see if we could have them updated. Thanks a lot!
Silly me for not realising this and sharing with you guys earlier.
