#48927 closed defect (bug) (fixed)
Aesthetic Issues with Language Select on Install introduced in 5.3.1 RC1
| Reported by: | garrett-eclipse | Owned by: | audrasjb |
|---|---|---|---|
| Priority: | normal | Milestone: | 5.3.1 |
| Component: | Upgrade/Install | Version: | 5.4 |
| Severity: | normal | Keywords: | has-screenshots has-patch commit |
| Cc: | Focuses: | ui, accessibility, css, administration |
Description
Hello,
Testing 5.3.1 RC1 today I noticed on install the language expanded select has changed aesthetically and I believe it to be unintentional.
Original Slack Question - https://wordpress.slack.com/archives/C02RQBWTW/p1576000796409900
From my screen to be attached.
- Should an expanded select like this have the chevron (down arrow)? I don’t believe so as it’s not a dropdown select.
- When in focus all options get the blue focus colour which I found a little odd. Should they. Or would it be better to just have the border change and only change the options to blue when they’re hovered or selected.
Installing a clean 5.3 I can confirm this isn't present there so was introduced during the 5.3.1 cycle.
Cheers
Attachments (7)
Change History (13)
@
7 years ago
Remove the dropdown chevron by updating .language-chooser select to use background instead of background-color
@
7 years ago
Override the select focus on the language-chooser to avoid all entires changing colour
@
7 years ago
Combined patch removing the chevrons and moving focus to the options as well as adding hover to the options
#2
@
7 years ago
- Keywords has-patch needs-testing 2nd-opinion added
I've uploaded a few patches to address these issues.
- 48927.chevron.diff - Simply removes the chevron by updating the
.language-choose selectcss to usebackgroundinstead of justbackground-color. - 48927.focus.diff - This only disables the focus on the select so all options dont' change colour on select focus.
- 48927.focus.hover.diff - Similar to # 2 this disables the focus on the entire select, but it moves it down to the individual options and adds a hover to the individual options.
- 48927.everything.diff - This is a combination of # 1 and # 3 and removes the chevron, select focus colour and moves focus and a hover to the select options.
*At minimum I would like to see # 1 implemented.
#4
@
7 years ago
- Keywords commit added; needs-testing 2nd-opinion removed
- Status reviewing → accepted
Thanks @garrett-eclipse for the ticket and the few patch!
I tested 48927.everything.diff and it's a green light on my side (see screenshot below).
Adding commit keyword for 48927.everything.diff. This is a pretty simple patch, worth merging it to 5.3.1.
Thanks,
Jb
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Expanded Language select on 5.3.1 RC1 - See focus colour affects all options and select chevron (down arrow) is present.