-
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
Search Block: Button Inside and Outside options behave similarly #69318
Comments
Hopefully needless to say, but a button element can not be placed inside an input field. Even if it is only moved with CSS, it would take a large effort to make it accessible, as seen in the screenshot. |
Thanks @carolinan for sharing your views,
Agree with this point, that's why I shared the caveats as well with this approach 🙂 |
Can you elaborate a bit on why this feels intuitive to you? Additionally, I don't think we should tolerate approaches that create accessibility issues. |
I think we "shot ourselves in the foot" with the decision to make the color options only apply to the button. That is also one of the reasons why the design in the screenshot can not be made with only the search block settings. It can come closer by wrapping it in a group block and adding the colors and border, but not all the way. |
Sure @t-hamano, so from a user’s perspective, the term “Button Inside” naturally suggests that the button is placed within the input field. This interpretation is further reinforced by the icon associated with it. ![]() Additionally, when exploring widely available design patterns on the internet for buttons inside input fields, the “Suggested Implementation” aligns more closely with common design practices compared to the “Current Implementation.” This makes it a more intuitive and expected approach for users.
In complete support of this, we can definitely close the issue if we feel it would be causing more problem than solving some minor UI/UX changes, I just wanted to bring this up to share some insights on the current implementation Thanks for understanding! |
This will be interpreted differently by different people:
Do you have any concrete examples of this? I just want to understand what benefits would be gained by placing a button inside an input element 🤔 |
Description
Currently, the ‘Button Inside’ and ‘Button Outside’ options in the search block function the same way, with the only difference being the border. We should reconsider the ‘Button Inside’ implementation, as it should place the button inside the input field. However, it is currently still positioned outside. Additionally, if the border is removed from the ‘Button Inside’ option, it becomes identical to the ‘Button Outside’ option.
Step-by-step reproduction instructions
Screenshots, screen recording, code snippet
Screen.Recording.2025-02-26.at.11.07.53.AM.mov
Environment info
Please confirm that you have searched existing issues in the repo.
Please confirm that you have tested with all plugins deactivated except Gutenberg.
Please confirm which theme type you used for testing.
The text was updated successfully, but these errors were encountered: