Make WordPress Core

Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#46978 closed defect (bug) (fixed)

Remove title attributes from the Meta widget

Reported by: afercia's profile afercia Owned by: audrasjb's profile audrasjb
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 (5)

46978.patch (1.4 KB) - added by nishitlangaliya 5 years ago.
fixes #46978
46978-2.patch (1.7 KB) - added by nishitlangaliya 5 years ago.
updated patch doc block updated
46978-3.patch (1.7 KB) - added by nishitlangaliya 5 years ago.
doc version updated 5.3.0
46978.4.diff (1.7 KB) - added by audrasjb 5 years ago.
Add %1$s and %2$s placeholders to sprintf function & refresh patch.
46978.diff (1.8 KB) - added by afercia 5 years ago.

Download all attachments as: .zip

Change History (25)

#1 @afercia
5 years 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
5 years ago

fixes #46978

#2 @nishitlangaliya
5 years ago

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

Thanks

#3 @chetan200891
5 years 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
5 years ago

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

#6 @afercia
5 years 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
5 years 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
5 years 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
5 years ago

updated patch doc block updated

#9 @afercia
5 years ago

  • Keywords has-patch added

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


5 years ago

#11 @afercia
5 years 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
5 years ago

doc version updated 5.3.0

#12 @nishitlangaliya
5 years 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
5 years ago

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

#14 @audrasjb
5 years ago

  • Owner set to audrasjb
  • Status changed from new to assigned

Hi there,

The previous patch didn't apply properly on my side, due to a _x() function used in the patch, instead of __() in current trunk (for reference: changeset [45436]).

I also replaced %s with %1$s and %2$s numbered placeholders.

@afercia I wouldn't say we need anything for backward compatibility here, since the filter still accepts a single bit of HTML code :)

Last edited 5 years ago by audrasjb (previous) (diff)

@audrasjb
5 years ago

Add %1$s and %2$s placeholders to sprintf function & refresh patch.

#15 @audrasjb
5 years ago

  • Keywords commit added

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


5 years ago

#17 @afercia
5 years ago

  • Keywords needs-refresh added; commit removed

See also https://core.trac.wordpress.org/ticket/14358#comment:1 (9 years ago):

I think we should also review the title text, as it feels a little dated. "Powered by WordPress, state-of-the-art semantic personal publishing platform." Most of that used to appear on wordpress.org IIRC but it has since been modified.

As per the patch, I think the change to the filter documentation should be reviewed.

* @param string $title_text Default title text for the WordPress.org link.

It appears to be not 100% correct. Since the filter was introduced in #14358 it filters the whole list item HTML content and not a $title_text (there's no variable for a title text, whatever that means). Also, this patch doesn't remove the param :)

On it.

@afercia
5 years ago

#18 @afercia
5 years ago

  • Keywords needs-refresh removed

46978.diff refreshes the patch to improve the filter documentation.

#19 @afercia
5 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 45728:

Accessibility: Remove title attributes from the Meta widget.

  • removes the Powered by WordPress, state-of-the-art semantic personal publishing platform. title attribute
  • removes the title attribute from the abbreviations: contrary to a common belief, title attributes on abbreviations are only available to a minority of users, see https://developer.paciellogroup.com/blog/2019/03/short-note-the-abbreviation-appreciation-society/
  • changes the term RSS to feed, as RSS is a technical term not all users are supposed to be familiar with
  • improves the widget_meta_poweredby filter documentation

Props nishitlangaliya, chetan200891, audrasjb.
See #46980, #24766.
Fixes #46978.

This ticket was mentioned in Slack in #core by jeffpaul. View the logs.


5 years ago

Note: See TracTickets for help on using tickets.