#35135 closed defect (bug) (fixed)
Remove title attributes: the Link Manager screen
Reported by: | afercia | Owned by: | afercia |
---|---|---|---|
Milestone: | 4.5 | Priority: | normal |
Severity: | normal | Version: | 4.4 |
Component: | Administration | Keywords: | has-screenshots, has-patch, commit, title-attribute |
Focuses: | ui, accessibility | Cc: |
Description
See related #24766 and all the following tickets about title attributes.
The Link Manager is disabled by default since WordPress 3.5 and it can be re-enabled with the Link Manager plugin.
The first title attribute should expand the link text using screen-reader-text
, as already done for example in the Media Library list table (please notice this should be done for all similar links in the List Tables).
I'd suggest to simply remove the second title attribute. since it's a link in a "URL" column and it's clear enough.
Attachments (3)
Change History (13)
#2
@
9 years ago
Discussed on Slack with the accessibility team because we noticed a bit of inconsistency in the way we're using .screen-reader-text
and aria-label
. Decided to stick on aria-label
when the intent is to completely replace the visible text and .screen-reader-text
if adding additional text.
New patch incoming.
#3
@
9 years ago
- Keywords commit added
Refreshed patch to use aria-label
instead of .screen-reader-text
.
This ticket was mentioned in Slack in #core by dossy. View the logs.
9 years ago
#6
@
9 years ago
- Keywords commit removed
- Resolution fixed deleted
- Status changed from closed to reopened
The use of esc_attr_e()
in [35998] is wrong, should be esc_attr()
.
The proposed patch fixes also some typos in the DocBlocks.