-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix: Add rel="tag"
to individual anchor links in editor markup of the post terms block
#68722
base: trunk
Are you sure you want to change the base?
Fix: Add rel="tag"
to individual anchor links in editor markup of the post terms block
#68722
Conversation
…he post terms block
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.
To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
Size Change: +8 B (0%) Total Size: 1.84 MB
ℹ️ View Unchanged
|
This was a really minor change I noticed whilst working on something else, so I quickly created this PR without adding an issue first. Happy to create one if this is more controversial
What?
Adds the
rel="tag"
attribute to thea
elements of thepost-terms
block in the editorWhy?
Because the frontend markup renders them that way. On the server we use the `` function to produce the markup which includes the
rel="tag"
attribute.gutenberg/packages/block-library/src/post-terms/index.php
Lines 49 to 55 in 5688333
https://github.com/WordPress/wordpress-develop/blob/d2a7bee9a6201f0276492ff29afd8c509480d266/src/wp-includes/category-template.php#L1356
How?
Adding the attribute in the edit.js file
Testing Instructions
rel="tag"
attribute present