-
Notifications
You must be signed in to change notification settings - Fork 6
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
design / selected color isn't clear #14
Comments
Yes, this is not clear and was the first thing I saw after this update. On 22 August 2015 at 16:02, ofZach notifications@github.com wrote:
|
Btw I think I fixed this in 002c075 (makes it white on gray) |
if it helps, it was also unclear to me which one was "selected". a color might not be enough, but adding a border to the selected ones would feel very clear to me. |
I also think an "x" image, like the PG roadmap mockup and the adddons multiselect would be useful, ie, signaling an unselect... I tried to do this via css but I think it has to be done with javascript. |
Places the console as relative and fixes some of it’s CSS Removes css from index.html ( into app.css ) Fixes PG error messages not being sent to console Console setting and advanced setting are independent now(#7)
I did a new try, might be better but I'm still not convinced. |
I think if the x was bigger (like the PG mockup) and it was visibility:none on unselected (so there wasn't a gap) it might help. Obviously things will reflow as you select them, but it feels like it would be more obvious. |
I made the gap on purpose so the buttons don't move on enabling/disabling them. (because adding the "x" adds more width to the buttons) |
I think it's fine if they reflow as you select them... |
- Reflow stuff (removed whitespace padding) - No pointer nor "x" when only 1 platform is selected Also Fixes default platform selection
Should be better now :) |
this could be closed i think, as the design is much clearer now. |
here, in one case grey = selected, while in another case, white = selected. needs maybe some consistency?
The text was updated successfully, but these errors were encountered: