Navigation Link, Navigation Submenu: Remove the title attribute controls#69689
Conversation
|
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. |
|
The failing tests don’t seem to be related to the changes in this PR. |
t-hamano
left a comment
There was a problem hiding this comment.
Thanks for the PR!
I think it would be good to focus only on two blocks, core/navigation-link and core/navigation-submenu in this PR.
If we plan to remove the title attribute from the Image block, we need to address the following tasks:
- If we update the save function, we will need to add a block deprecation.
- The
titleattribute is allowed to be used as Block Binding API (See this core code). This means that we may need to prepare a patch that removes thetitleattribute from the supported attributes to the core along with this PR.
I think It's too big to tackle all the tasks in one PR.
|
Hi @t-hamano, |
t-hamano
left a comment
There was a problem hiding this comment.
I think we need to completely remove code related to the title attribute. For example:
gutenberg/packages/block-library/src/navigation-link/index.php
Lines 252 to 254 in d8545b2
It's a good idea to go through the code in both blocks and look for anything that needs to be removed.
|
Hi @t-hamano, Thank You! |
t-hamano
left a comment
There was a problem hiding this comment.
LGTM! I think all code related to the title attribute has been removed.
Since I think this change is very important for site builders, I'm adding a Needs dev note label to announce it in the next major release.
…ping attribute support (#70234) * Revert the changes from PR #69689 * Remove title attribute from Controls in Navigation Link and Submenu blocks Co-authored-by: Infinite-Null <[email protected]> Co-authored-by: t-hamano <[email protected]>
…ols (WordPress#69689) * Remove Title attribute controls * Remove Title controls from Image component * Remove 'title' attribute from navigation link and submenu blocks * Remove title attribute checks from custom sources tests * Revert image changes * Remove `title` attribute from navigation link and submenu blocks Co-authored-by: Infinite-Null <[email protected]> Co-authored-by: t-hamano <[email protected]> Co-authored-by: carolinan <[email protected]> Co-authored-by: joedolson <[email protected]>
…ping attribute support (WordPress#70234) * Revert the changes from PR WordPress#69689 * Remove title attribute from Controls in Navigation Link and Submenu blocks Co-authored-by: Infinite-Null <[email protected]> Co-authored-by: t-hamano <[email protected]>
|
@t-hamano do you think it makes sense to combine this PR's dev note with the other deprecations? |
Closes: #69663
What?
Removes the Title attribute UI option from Navigation Submenu block and Custom Link button to align with accessibility best practices.
Testing Instructions
Screenshots or screencast