Opened 4 years ago
Closed 4 years ago
#50697 closed defect (bug) (fixed)
Remove underline in Theme "Enable auto-updates".
Reported by: | chetan200891 | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 5.5 | Priority: | normal |
Severity: | normal | Version: | 5.5 |
Component: | Themes | Keywords: | has-screenshots has-patch |
Focuses: | ui, accessibility, css, administration | Cc: |
Description
In theme overlay there is underline in "Enable auto-updates". It should be removed to make it consistent with other.
Attachments (5)
Change History (10)
#2
follow-up:
↓ 4
@
4 years ago
Thanks @chetan200891 !
50697.1.diff changes the selector so the !important
is no longer necessary to remove the underline from the button (as intended when it was a link).
However, I highly recommend making it consistent the other way by underlining both the author link and the button text (without underlining the spinner icon). 50697.underline-author.diff clearly distinguishes the blue link from the gray text.
#3
@
4 years ago
- Focuses accessibility added
The underline shouldn't be removed. Otherwise, this link (actually it's a button that looks like a link) would be distinguishable as an interactive controls only by its color.
Color alone is not sufficient. As in the case of links, the underline is necessary when the context doesn't clarify it's an interactive control.
When links can be identified as such by the context, for example because they’re part of a menu, or a set of links clearly identified as user interface controls, they don’t necessarily need to be underlined. In all the other cases, especially for links surrounded by other text (in a line or block of text), links need to be always underlined.
#4
in reply to:
↑ 2
@
4 years ago
- Milestone changed from Awaiting Review to 5.5
Replying to sabernhardt:
However, I highly recommend making it consistent the other way by underlining both the author link and the button text (without underlining the spinner icon). 50697.underline-author.diff clearly distinguishes the blue link from the gray text.
I agree with that.
Before