Skip to content
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

Confirmation for reset #4117

Closed
fqdb opened this issue Jun 25, 2017 · 4 comments
Closed

Confirmation for reset #4117

fqdb opened this issue Jun 25, 2017 · 4 comments
Labels
considering Not Actionable - still considering if this is something we want usability An issue with ease-of-use or design

Comments

@fqdb
Copy link

fqdb commented Jun 25, 2017

Could there be a confirmation dialog for the reset button?

Especially since 'reset' and 'restore' are the same size, it's easy to click another out of habit.

@bhousel
Copy link
Member

bhousel commented Jun 25, 2017

Can you share a screenshot? I'm not sure which buttons you mean..

@fqdb
Copy link
Author

fqdb commented Jun 25, 2017

image

Maybe the buttons could also be redisigned or renamed, because the left symbol is also a common symbol to mean 'reset', and right one could seem like a 'cancel', as in 'do nothing and keep previous'.

@bhousel
Copy link
Member

bhousel commented Jun 26, 2017

I think changing the icons would probably confuse people who are already very familiar with this dialog.

But I think the text labels could be improved. What if they said "Restore my changes" / "Discard my changes" ?

@bhousel bhousel added considering Not Actionable - still considering if this is something we want usability An issue with ease-of-use or design labels Jun 26, 2017
@fqdb
Copy link
Author

fqdb commented Jun 26, 2017

Yes, that would be an improvement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
considering Not Actionable - still considering if this is something we want usability An issue with ease-of-use or design
Projects
None yet
Development

No branches or pull requests

2 participants