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

[2] Dropdown Improvements #2149

Closed
startailcoon opened this issue Nov 11, 2018 · 14 comments
Closed

[2] Dropdown Improvements #2149

startailcoon opened this issue Nov 11, 2018 · 14 comments
Assignees
Labels
[1b] User Story The User Story details a requirement. It may ref a parent Project (Epic). It may ref child Task(s) [3] Enhancement Classification indicating a change to the functionality of the existing imlementation [5b] Small Indicates size of task. Est. between one and two hours [6] Style Guide Impact flag identifying related to a Style Guide [6] UX Impact flag identifying the application User Experience [7] Estimated Administration flag, notifies hours are estimated and requires revisiting
Milestone

Comments

@startailcoon
Copy link
Contributor

Issue extracted from #2020

By @wmbutler

The color of the selected item in the dropdown is a little ambiguous. I'd typically expect highest contrast to be the selected item but this is clearly not possible on the Midnight view. See this screenshot to illustrate my point. Maybe we use the blue universally? This is a tough one.

bild

@startailcoon startailcoon added [3] Enhancement Classification indicating a change to the functionality of the existing imlementation [1b] User Story The User Story details a requirement. It may ref a parent Project (Epic). It may ref child Task(s) [5b] Small Indicates size of task. Est. between one and two hours labels Nov 11, 2018
@startailcoon startailcoon added this to the 181128 milestone Nov 11, 2018
@startailcoon
Copy link
Contributor Author

Here is a licecap that visualizes it a bit more

bitshares-dropdown

@wmbutler wmbutler added the [6] UX Impact flag identifying the application User Experience label Nov 13, 2018
@startailcoon startailcoon added the [7] Estimated Administration flag, notifies hours are estimated and requires revisiting label Nov 20, 2018
@startailcoon startailcoon changed the title [Style-Guide] Dropdown Improvements [1.5] Dropdown Improvements Nov 20, 2018
@startailcoon startailcoon added the [6] Style Guide Impact flag identifying related to a Style Guide label Nov 20, 2018
@gibbsfromncis
Copy link
Contributor

gibbsfromncis commented Nov 22, 2018

@startailcoon to be honest I'm not following the idea... can you clarify please?

Do you mean that currently selected item has the same background as hovered items?

@sschiessl-bcp
Copy link
Contributor

Please check that you can also select with keyboard (up and down moves the selection, enter confirms it)

@startailcoon
Copy link
Contributor Author

@wmbutler, could you clarify your thoughts on this matter?

@wmbutler
Copy link
Contributor

wmbutler commented Dec 1, 2018

I'll claim this.

@wmbutler wmbutler assigned wmbutler and unassigned gibbsfromncis Dec 1, 2018
@wmbutler wmbutler modified the milestones: 181128, 181212 Dec 1, 2018
@wmbutler
Copy link
Contributor

wmbutler commented Dec 1, 2018

Here is my proposed change. I think what was most confusing was the hover color. Typically, I'd expect a higher contrast for the selected item. The midnight view makes higher contrast virtually impossible with a background color, so I updated the style guide to reflect only a blue color for active/hover states on the text menu items. I think it looks pretty good.

It took me two hours to hunt through all the variations.

select

@wmbutler wmbutler changed the title [1.5] Dropdown Improvements [2] Dropdown Improvements Dec 1, 2018
wmbutler added a commit to wmbutler/bitshares-ui-style-guide that referenced this issue Dec 1, 2018
@wmbutler wmbutler modified the milestones: 181212, 181128 Dec 1, 2018
@wmbutler
Copy link
Contributor

wmbutler commented Dec 1, 2018

@wmbutler wmbutler closed this as completed Dec 1, 2018
@gibbsfromncis
Copy link
Contributor

@wmbutler Unfortunately your changes will work but only till next update from my side, because you changed source file instead of config file which uses to build themes.

There is instruction how to update styles on the right way - https://github.com/bitshares/bitshares-ui-style-guide/blob/master/INSTRUCTIONS.md#how-to-manage-styles-of-style-guide-components-

I'll open new issue to fix it

@wmbutler
Copy link
Contributor

wmbutler commented Dec 3, 2018

Oops. Sorry. I'll review.

@wmbutler
Copy link
Contributor

wmbutler commented Dec 3, 2018

I just read your instructions and saw no mention of a config file.

@wmbutler
Copy link
Contributor

wmbutler commented Dec 3, 2018

Your instructions are a bit ambiguous. Every directory is /dark/ ?

screen shot 2018-12-03 at 11 08 43 am

@gibbsfromncis
Copy link
Contributor

@gibbsfromncis
Copy link
Contributor

@wmbutler There are few ways to get your task done.

  1. Update specific ant's variable which is responsible for that.
  2. If ant has no variable to handle your situation you should do the next:
  • Provide your own variable for Select (ant calls it Select)
  • Add override of Select for specific state you need

@wmbutler Lets make it more faster - give me color you need, I'll update it and we will keep your hours and I'll put my hours and we finally will close this task.

@gibbsfromncis
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[1b] User Story The User Story details a requirement. It may ref a parent Project (Epic). It may ref child Task(s) [3] Enhancement Classification indicating a change to the functionality of the existing imlementation [5b] Small Indicates size of task. Est. between one and two hours [6] Style Guide Impact flag identifying related to a Style Guide [6] UX Impact flag identifying the application User Experience [7] Estimated Administration flag, notifies hours are estimated and requires revisiting
Projects
None yet
Development

No branches or pull requests

4 participants