Skip to content

Poweruser search, use / to focus on searchinput - #797

Closed
tswfi wants to merge 329 commits into
PrestaShop:developfrom
tswfi:poweruser_search
Closed

Poweruser search, use / to focus on searchinput#797
tswfi wants to merge 329 commits into
PrestaShop:developfrom
tswfi:poweruser_search

Conversation

@tswfi

@tswfi tswfi commented Oct 25, 2025

Copy link
Copy Markdown
Contributor
Questions Answers
Description? Poweruser search. Use / to focus on the searchInput field
Type? new feature
BC breaks? no
Deprecations? no
Fixed ticket? -
Sponsor company
How to test? Open a page and type / , check that search input is focused (and on mobile the offCanvas is open) and ready for searching. Check that the / can still be typed on input fields elsewhere (the search focus should only happen if you are not in a input field already)

@github-project-automation github-project-automation Bot moved this to Ready for review in PR Dashboard Oct 25, 2025
@ga-devfront

Copy link
Copy Markdown
Contributor

A large pull request is currently being merged (#803). It includes many fixes and improvements for version 2.0 of Hummingbird.
You have two options regarding this pull request:

Option 1:

  • Wait for the pull request to be merged.
  • Verify that your pull request is still relevant.
  • If it's the case rebase it so that it can be considered by our teams.

Option 2:

  • Target the 1.x branch if you consider this to be an important pull request for version 1 of Hummingbird.

If you have any questions or concerns, please feel free to let us know. Have a great day, and may the power of the bird 🐦 be with you ❤️

@ga-devfront ga-devfront added the Blocked Waiting on another task or dependency. label Nov 4, 2025
@tswfi

tswfi commented Nov 4, 2025

Copy link
Copy Markdown
Contributor Author

@ga-devfront you did notice that this pr is againts the improvements branch already? :)

@ga-devfront

Copy link
Copy Markdown
Contributor

Sorry, in any case, the pull requests are on hold while we merge the large pull request mentioned earlier. We will be happy to review and test everything as soon as our initial work is complete.

@ga-devfront
ga-devfront changed the base branch from improvements to develop November 10, 2025 13:32
@ga-devfront

Copy link
Copy Markdown
Contributor

@tswfi can you rebase please. I change the target to merge into the right branch.
Thank you for your contribution.

@ga-devfront ga-devfront added Waiting for Rebase and removed Blocked Waiting on another task or dependency. labels Nov 10, 2025
@tswfi

tswfi commented Nov 10, 2025

Copy link
Copy Markdown
Contributor Author

rebaseing nightmare. opened a new pr for this: #806

@tswfi tswfi closed this Nov 10, 2025
@github-project-automation github-project-automation Bot moved this from Ready for review to Closed in PR Dashboard Nov 10, 2025
@tswfi
tswfi deleted the poweruser_search branch November 10, 2025 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

6 participants