Opened 4 weeks ago
Closed 4 weeks ago
#62838 closed defect (bug) (fixed)
Remove title attribute from shortlink
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | 6.8 | Priority: | normal |
Severity: | normal | Version: | 3.0 |
Component: | Posts, Post Types | Keywords: | title-attribute has-patch |
Focuses: | accessibility | Cc: |
Description
Related: #24766
Since [13683], the_shortlink()
output has included a title
attribute. By default, that gives the sanitized post title, and it does not sanitize custom text. Given the low value of this attribute, it probably would be better to remove it than to try making it more accessible.
The shortlink has very limited use in directory plugins and themes.
Change History (8)
This ticket was mentioned in PR #8159 on WordPress/wordpress-develop by @sabernhardt.
4 weeks ago
#1
- Keywords has-patch added
This ticket was mentioned in PR #8160 on WordPress/wordpress-develop by @sabernhardt.
4 weeks ago
#2
Option 2: Removes only the default title
attribute from the shortlink.
#3
@
4 weeks ago
- Milestone changed from Awaiting Review to 6.8
I vote in favor of complete removal. It would still be possible to retain support in custom contexts through the filter, and given the real world usage, I think that's adequate.
#4
@
4 weeks ago
Yes, let's remove it entirely.
I proposed to set the $title
param to null
as in other deprecated params in various functions (but we're not 100% actually consistent in core by the way).
@sabernhardt commented on PR #8160:
4 weeks ago
#5
Closing in favor of option 1 (PR 8159)
This ticket was mentioned in Slack in #core-test by oglekler. View the logs.
4 weeks ago
@audrasjb commented on PR #8159:
4 weeks ago
#7
committed in https://core.trac.wordpress.org/changeset/59703
Option 1: Removes any
title
attribute from the shortlink output, and updates documentation.Trac 62838