[wp-trac] [WordPress Trac] #35136: Remove title attributes: the Media Library screen

[WordPress Trac]

#35136: Remove title attributes: the Media Library screen
---------------------------------------+--------------------------------
 Reporter:  afercia                    |       Owner:  afercia
     Type:  defect (bug)               |      Status:  assigned
 Priority:  normal                     |   Milestone:  4.5
Component:  Media                      |     Version:  4.4
 Severity:  normal                     |  Resolution:
 Keywords:  has-screenshots has-patch  |     Focuses:  ui, accessibility
---------------------------------------+--------------------------------
Description changed by afercia:

Old description:

> See related #24766 and all the following tickets about title attributes.
>
> Please refer to the screenshot below:
>
> [[Image(https://cldup.com/dJhVEeoZR7.png)]]
>
> '''The title attribute on the "Comments" table header'''
> I'd suggest to simply remove it, the link already has a hidden text.
>
> '''The title attributes on the row action links'''
> there are a couple of things to notice:
> - the only link with a title attribute is the "View" link, this is a bit
> inconsistent (also visually)
> - the title attribute is different depending on the attachment being
> attached or not attached to a post
>
> When it's attached, the attachment permalink will contain also the parent
> post title but it still points to the attachment page so the title
> attribute saying "View {post parent title}" doesn't look 100% accurate to
> me. I'd propose to simplify and just say "View {attachment title}".
>
> The attachment main "Edit" link and all the row action links should have
> some hidden text or `aria-label` to give more context or to reference the
> attachment title. See for example the Media Library screen and the
> Plugins screen.
>
> '''The "Attach" link'''
> it opens the "Find Posts or Pages" modal, should inform that is going to
> open a modal dialog.
>
> '''The link to the post parent in the "Uploaded to" column'''
> it points to the "Edit" screen. It's not clear at first sight if it's a
> link to "View" or "Edit", by the way this is out of the scope of this
> ticket and should go in a separate one.
>
> Working on a patch but would like to have some feedback from the
> accessibility team before.

New description:

 See related #24766 and all the following tickets about title attributes.

 Please refer to the screenshot below:

 [[Image(https://cldup.com/dJhVEeoZR7.png)]]

 '''The title attribute on the "Comments" table header'''
 I'd suggest to simply remove it, the link already has a hidden text.

 '''The title attributes on the row action links'''
 there are a couple of things to notice:
 - the only link with a title attribute is the "View" link, this is a bit
 inconsistent (also visually)
 - the title attribute is different depending on the attachment being
 attached or not attached to a post

 When it's attached, the attachment permalink will contain also the parent
 post title but it still points to the attachment page so the title
 attribute saying "View {post parent title}" doesn't look 100% accurate to
 me. I'd propose to simplify and just say "View {attachment title}".

 The attachment main "Edit" link and all the row action links should have
 some hidden text or `aria-label` to give more context or to reference the
 attachment title. See for example the Plugins screen.

 '''The "Attach" link'''
 it opens the "Find Posts or Pages" modal, should inform that is going to
 open a modal dialog.

 '''The link to the post parent in the "Uploaded to" column'''
 it points to the "Edit" screen. It's not clear at first sight if it's a
 link to "View" or "Edit", by the way this is out of the scope of this
 ticket and should go in a separate one.

 Working on a patch but would like to have some feedback from the
 accessibility team before.

--

--
Ticket URL: <https://core.trac.wordpress.org/ticket/35136#comment:9>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform