-
Notifications
You must be signed in to change notification settings - Fork 362
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
Combobox examples: Visual design review from PR #1255 #1263
Comments
I think it is very important to have a visual indication for the role. A combobox should be visually different from an input field. A combobox with text input should also be visually distinguishable from a combobox without text input. Just like a |
The focus indicator shows insufficient contrasts and thus violates WCAG SC 1.4.11:
Minimum contrast is 3:1. If the contrasts are adjusted, it should be noted that the selected element should also be visible in High Contrast Mode, i.e. the focus should not be represented exclusively by a color change. In addition, WCAG 2.2 requires a minimum area for the contrast indicator (length of the element times 2px). I am unsure whether the green frame meets this requirement. |
@carmacleod wrote
This is because the |
I have made changes to examples 1-3 to try to address the design issues. It should work with Windows High Contrast Mode in Edge as well. I haven't changed example 4. |
I liked the rounder corners... :) |
Yeah though @isaacdurazo said rounded corners are a bit passe. 🙂 |
I also think the usage of rounded corners as well as some colors were inconsistent with the design of the rest of the page |
Thanks @carmacleod! The labels in 2 and 3 should be moved outside of the combobox container in the markup. |
Trends come and go... but science is here to stay. 😉 Can we compromise and go with very slightly rounded corners like the middle button in this snap? 😄 |
Visual design review of combobox PR #1255
There are 4 examples:
The first 3 have the same visual design, so I'll only review the first one of those ("example 1"). The grid popup example ("example 4") has different visual design.
Example 1
First, let's compare the default state of the example side by side in Safari (left), Chrome (middle), and Firefox (right).
The following screenshots are taken with Firefox Nightly on macOS.
Default state
Hover on the combobox field
Hover on the button
Focus on the combobox
Popup open
After typing "Al" and focusing Alabama
Example 4
Default state in Safari (top), Chrome (middle), Firefox (bottom)
There is no provided styling for hover or focus. In Safari and Chrome, the default focus ring is shown. In Firefox, it is not. Removing the
border-radius
makes Firefox show its default focus ring. This might be a bug in Firefox? (Specifying-webkit-appearance: none
has the same effect.)After focusing and typing the letter "a"
Moving focus to the first cell of the first row
Moving focus to the second cell of the first row
Typical modern implementations of the widget
paper-dropdown-menu
demopfe-autocomplete
Suggest
Omnibar
Analysis
Our examples are not on par with modern design systems/component frameworks.
Example 1:
Example 4:
The text was updated successfully, but these errors were encountered: