-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Try to make link UI in the button block consistent with other link interfaces #16495
Comments
Seems to me one of the inconsistencies (and an actual problem for accessibility) is that when a link suggestion is highlighted, tabbing away from the input field sets the link. This is a problem because the Tab key must be reserved for keyboard navigation. Triggering an action (setting the link) while users just want to navigate away would be completely unexpected. |
Yep, though just to clarify, the link is updated on every keypress not when the input blurs. That's not something new, but the way the button block worked before #10128 as well. Introducing a submit/apply button would make it more consistent and probably help in the implementation of #16494 as well. Could even be done as part of fixing that issue. |
Closing as the link UI has recently been updated. |
See also #2216, #6392
Description
#10128 involved quite a bit of iteration on the link UI in the button block, which was perhaps out of scope for the related issue. Eventually, a simplified design was settled on in order to move the PR forwards, but there's was acknowledgement that we're still left with inconsistencies across link user interfaces in Gutenberg.
This issue summarises some of the explorations on that PR and captures that further exploration may be required.
Attempted in #10128:
Using the link popover in the button block (#10128)
For:
Against:
Using a button to open the link popover (#10128 (comment))
For:
Against:
Other info:
An inline version of the link popover (#10128 (comment))
For:
Against:
Other info:
A simplified version of the link UI (screen capture is similar to what was implemented)
For:
Against:
The text was updated successfully, but these errors were encountered: