LinkControl: Fix visually Duplicative label and placeholder#69620
Conversation
|
Size Change: -1 B (0%) Total Size: 1.84 MB
ℹ️ View Unchanged
|
7e68432 to
3d894a7
Compare
|
+1. I would like to avoid introducing new public APIs ( |
3bc7d7d to
0dcc98d
Compare
|
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. |
|
@t-hamano, I think we should re-open #65458 or create a new issue and try to address the original a11y concerns. Here are some problems with the original fix:
|
Mamaduka
left a comment
There was a problem hiding this comment.
Thanks, @t-hamano!
This looks good to me, but let's ensure we follow up on #69620 (comment).
|
There was a conflict while trying to cherry-pick the commit to the wp/6.8 branch. Please resolve the conflict manually and create a PR to the wp/6.8 branch. PRs to wp/6.8 are similar to PRs to trunk, but you should base your PR on the wp/6.8 branch instead of trunk. |
I would like to submit another PR for the wp/6.8 branch. |
* LinkControl: Fix visually Duplicative label and placeholder * Try to fix e2e tests * Try to fix e2e tests Co-authored-by: t-hamano <wildworks@git.wordpress.org> Co-authored-by: Mamaduka <mamaduka@git.wordpress.org>
|
The backport PR fo the wp/6.8 branch: #69653 |
|
Thanks, @t-hamano! Feel free to merge backport PR as soon as CI checks are green 🙌 |
…69653) * LinkControl: Fix visually Duplicative label and placeholder * Try to fix e2e tests * Try to fix e2e tests Co-authored-by: t-hamano <wildworks@git.wordpress.org> Co-authored-by: Mamaduka <mamaduka@git.wordpress.org>
…s#69620) * LinkControl: Fix visually Duplicative label and placeholder * Try to fix e2e tests * Try to fix e2e tests Co-authored-by: t-hamano <wildworks@git.wordpress.org> Co-authored-by: Mamaduka <mamaduka@git.wordpress.org>
What?
Alternatives to #66329
Why?
#65458 changed many UIs to have matching labels and placeholder text. However, this only seems to work if the label is visually hidden.
In the
LinkControlcomponent, once a URL is submitted, the label and placeholder text visually overlap. I think this is not the intended result of #65458.How?
Reverting to the previous spec.
Ideally, we would move forward with #66329, but I am cautious about adding new props to the
LinkControlcomponent in the 6.8 RC1 phase.Personally, I would recommend backporting this PR to 6.8 RC1 and then re-reviewing #66329 after that.
Testing Instructions
Screenshots or screencast
The UI is the same as before when the text isn't linked yet:
When the text is linked;