WordPress.org

Make WordPress Core

Opened 2 months ago

Last modified 4 weeks ago

#46978 new defect (bug)

Remove title attributes from the Meta widget

Reported by: afercia Owned by:
Milestone: 5.3 Priority: normal
Severity: normal Version:
Component: Widgets Keywords: has-screenshots title-attribute has-patch
Focuses: accessibility, docs Cc:

Description

In the last releases, WordPress has been progressively removing many title attributes used in the admin screens for the reasons outlined in #24766.

So far, focus has been mainly on the admin part. I'd like to propose to start looking at the front end part and see which title attributes are good candidates for removal.

In the Meta widget, there are two title attributes:

http://cldup.com/mQWkmARw8s.png

1: Powered by WordPress, state-of-the-art semantic personal publishing platform.
This was discussed a while ago on Slack and there's consensus to remove it, as it doesn't add great value.

2: <abbr title="Really Simple Syndication">RSS</abbr>
Contrary to a common belief, title attributes used on abbreviations are not that great, as they're only available to a minority of users. Also, both the abbreviation and the expanded form:

  • RSS
  • Really Simple Syndication

are technical terms users are not supposed to know. In this case, I'd like to propose to remove the abbreviation altogether and use the term "feed" which is slightly more common also for non-tech-savvy users:

  • Entries feed
  • Comments feed

Attachments (3)

46978.patch (1.4 KB) - added by nishitlangaliya 2 months ago.
fixes #46978
46978-2.patch (1.7 KB) - added by nishitlangaliya 8 weeks ago.
updated patch doc block updated
46978-3.patch (1.7 KB) - added by nishitlangaliya 6 weeks ago.
doc version updated 5.3.0

Download all attachments as: .zip

Change History (16)

#1 @afercia
2 months ago

Forgot to mention the accessibility team recommendation for title attributes can be summarized as follows: don't rely on title attributes for relevant information. If the information is relevant, consider to put it in plain, visible, text on the page. If it's not relevant, consider to remove it.

@nishitlangaliya
2 months ago

fixes #46978

#2 @nishitlangaliya
2 months ago

@afercia , I have added patch can you please review it and let me know if any changes required.

Thanks

#3 @chetan200891
2 months ago

  • Focuses docs added

@afercia Also based on ticket and patch, we need to change inline doc for widget_meta_poweredby filter too. Also $title_text parameter need to use there?

#5 follow-up: @nishitlangaliya
2 months ago

@SergeyBiryukov , Do you mean it is already done and do not required patch ?

#6 @afercia
2 months ago

@chetan200891 yes the docblock should be updated. I guess it will also need a

@since x.x.x Removed the `$title_text` parameter.

#7 in reply to: ↑ 5 @SergeyBiryukov
2 months ago

Replying to nishitlangaliya:

Do you mean it is already done and do not required patch ?

No, I've just linked to a previous discussion on removing title attributes from default widgets, for better context.

#8 @nishitlangaliya
8 weeks ago

@afercia , Doc updated for widget_meta_poweredby filter. Currently I have put @sicne 5.2.0 let me know if we need to change @since version.

Thanks

@nishitlangaliya
8 weeks ago

updated patch doc block updated

#9 @afercia
7 weeks ago

  • Keywords has-patch added

This ticket was mentioned in Slack in #accessibility by afercia. View the logs.


6 weeks ago

#11 @afercia
6 weeks ago

  • Milestone changed from Awaiting Review to 5.3

Discussed during today's accessibility bug scrub and agreed to milestone for next 5.3 release.

@nishitlangaliya
6 weeks ago

doc version updated 5.3.0

#12 @nishitlangaliya
6 weeks ago

@afercia , I have updated the version to 5.3.0. uploaded final patch 46978-3.patch please review it and let me know if any changes required

#13 @afercia
4 weeks ago

@SergeyBiryukov when you have a chance: do you think the widget_meta_poweredby filter needs some love for backwards compatibility?

Note: See TracTickets for help on using tickets.