#42647 closed defect (bug) (fixed)
Underline links in the plugin description for installed plugins
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 5.1 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Plugins | Keywords: | has-screenshots has-patch |
Focuses: | ui, accessibility | Cc: |
Description
At the moment the links in the plugin description for installed plugins are not underlined.
In my eyes this is fine for the lower links like "details", "support" and so on because there are separated with "|" and stand alone. But I see a problem for links in the plugin description above, cause color blind people could miss the links in the text.
I would suggest to underline the links in the plugin excerption so its easier for everyone to spot the links.
.plugin-description a {
text-decoration: underline;
}
Attachments (10)
Change History (24)
#3
@
8 years ago
- Keywords has-patch added; needs-patch removed
@Travel_girl +1
I have attached patch 42647.diff
There was already class ".plugins .desc p" so I have used it and added CSS "plugins .desc p a" for make underline of links in plugin description.
#4
@
8 years ago
Sorry. I am new to core contribution. Added wrong 426471.diff which has other patches also included. So I have added latest patch 426471.1.diff
#5
@
8 years ago
Should the links within the update notices be underlined too? I guess so, as the contrast on the yellow background is even lower than on a white background.
Plugins page:
On the Dashboard the background is white but there's no underline:
This would also improve consistency with similar links used for Themes, that are underlined:
#6
@
8 years ago
I have created patch 426471.3.diff for link underline on plugin update notice and dashboard of updates. Suggested by @afercia
#7
@
8 years ago
@chetan200891 thanks for the patches and welcome to contributing! :-)
@afercia yes there should be underlined too because it is also surrounded by p, so color blind people could miss the links.
@
8 years ago
added class in common.css ".plugin-version-author-uri a, .plugin-update p a, .plugin-title p a"
This ticket was mentioned in Slack in #accessibility by afercia. View the logs.
8 years ago
#9
@
8 years ago
- Component changed from General to Plugins
- Milestone changed from Awaiting Review to 5.0
This ticket was mentioned in Slack in #design by melchoyce. View the logs.
7 years ago
#11
@
7 years ago
42647.3.diff refreshes the patch. As mentioned in the original ticket description, better to not touch the links in the line below the plugins description because they're separated with "|" and the context helps clarifying they're links.
See in the screenshot below an example of the links that are going to be underlined (plus the one in the Updates screen):
#12
@
7 years ago
- Owner set to afercia
- Resolution set to fixed
- Status changed from new to closed
In 42810:
Current view