Skip to content
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: Remove 'Button only' option. #27329

Closed
bph opened this issue Nov 27, 2020 · 6 comments · Fixed by #27379
Closed

Search block: Remove 'Button only' option. #27329

bph opened this issue Nov 27, 2020 · 6 comments · Fixed by #27379
Assignees
Labels
[Block] Search Affects the Search Block - used to display a search field [Status] In Progress Tracking issues with work in progress

Comments

@bph
Copy link
Contributor

bph commented Nov 27, 2020

Describe the bug
When testing the Search block, I was going through all the options and found that the button only option doesn't have any action tied to it. It doesn't do anything as it doesn't have a search term to look for. It just reloads the same page.

See also comment #24666 (comment)

To reproduce
Steps to reproduce the behavior:

  1. Add a search block
  2. Select the "Button only" option
  3. Save & preview
  4. click on the Search button.

Expected behavior
I don't think there is an expected behavior. The "Button only" option feels foreign to the search functionality, unless an additional action is tied to it.

I can imagine for instance, a form field to slide out on button click for a visitor to enter a search term and on the second click it initiates the search. Describing it still feels a bit awkward, though.

Screenshots
Screen Shot 2020-11-27 at 9 36 37 AM
Screen Shot 2020-11-27 at 9 36 45 AM

Editor version (please complete the following information):

  • WordPress version: WordPress 5.6-RC1-49694

Desktop (please complete the following information):

  • OS: 19.15.7
  • Browser: Chrome
  • Version 87.0.4280.67
@bph bph added [Block] Search Affects the Search Block - used to display a search field Backport to WP 6.7 Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta labels Nov 27, 2020
@scruffian
Copy link
Contributor

I'd prefer to update the behaviour so that in the button only version, when the user clicks the button the search field shows.

@cguntur
Copy link

cguntur commented Nov 27, 2020

I like the option to just display the button. Showing the field when the button is clicked gives a better user experience.

@apeatling
Copy link
Contributor

Left a comment here with the intention #24666 (comment)

@bph
Copy link
Contributor Author

bph commented Nov 27, 2020

At this late stage in the release removal seems to be best way to handle this, to avoid more bug reports about a feature not working. Confusions are to be avoided 😋😂

@tellthemachines
Copy link
Contributor

I'm removing the Backport label as we only use it for PRs, but adding this issue to the 5.6 must haves board. Agree that this late in the release, the best fix for now is to remove that option altogether.

If anyone would like to do a PR for that, I'm happy to backport it 🙂 but if we're aiming for RC2 then it'll have to be done in the next 12 hours or so.

@tellthemachines tellthemachines removed the Backport to WP 6.7 Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta label Nov 30, 2020
@apeatling
Copy link
Contributor

Working on a PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Search Affects the Search Block - used to display a search field [Status] In Progress Tracking issues with work in progress
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants