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

Save/cancel buttons for changeset save are very difficult to distinguish #3040

Closed
jancborchardt opened this issue Mar 18, 2016 · 3 comments
Closed

Comments

@jancborchardt
Copy link

This is how it currently looks:
capture du 2016-03-18 19-37-54

A few problems with the Save/Cancel buttons:

  • usually the confirm function is on the right. I nearly clicked cancel here, and was a bit afraid of losing my edits
  • they both have the same color
  • they both have the same visual importance

Here’s my proposal:
capture du 2016-03-18 19-42-13

  1. Switch them around so Cancel is left and Save is right.
  2. Use a different »empty« style for the cancel button since it’s not the primary action. This is similar to the user dropdown and the »empty« / inactive buttons up top (History etc)
  3. Change the color to grey to make it less present.

Here’s the CSS rules for the cancel button:

background: #fff;
border: 1px solid #999;
color: #999;
@bhousel
Copy link
Member

bhousel commented Mar 18, 2016

Thanks for the suggestions.. I agree with all of it, and I frequently hit the wrong button too.

  • As of 2016, Save on the right is more common, so we should switch the placement
  • We should use a secondary action style for cancel.
    (Note that Save is initially grey and disabled if there is no changeset comment)

@bhousel bhousel added the design label Mar 18, 2016
@jancborchardt
Copy link
Author

Yeah, I also thought about if the change button being grey is the best idea. An alternative is to use the blue color of the save button as border and text color for the cancel button.

@bhousel bhousel closed this as completed in bd57b81 Apr 7, 2016
@jancborchardt
Copy link
Author

@bhousel thanks a lot! Very cool improvements :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants