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

fix: (CXSPA-9028) - Search dropdown first option visible focus #19741

Merged
merged 7 commits into from
Jan 8, 2025

Conversation

Pio-Bar
Copy link
Contributor

@Pio-Bar Pio-Bar commented Dec 9, 2024

Closes: CXSPA-9028

Arrow key presses are expected for inputs and normally do not trigger the visible focus. An exception needs to be made for search dropdowns where the down arrow moves the focus to the first option.

I've fixed this issue for the header search bar as well.

@Pio-Bar Pio-Bar requested a review from a team as a code owner December 9, 2024 12:33
@github-actions github-actions bot marked this pull request as draft December 9, 2024 12:34
@Pio-Bar Pio-Bar marked this pull request as ready for review December 9, 2024 12:34
Copy link

cypress bot commented Dec 9, 2024

spartacus    Run #46569

Run Properties:  status check passed Passed #46569  •  git commit 233752deca ℹ️: Merge 09ce7ab0f170c16f9124dc76fff4c6e517e7379b into 609570de95299566fda9354484ff...
Project spartacus
Branch Review feature/CXSPA-9028
Run status status check passed Passed #46569
Run duration 12m 08s
Commit git commit 233752deca ℹ️: Merge 09ce7ab0f170c16f9124dc76fff4c6e517e7379b into 609570de95299566fda9354484ff...
Committer PioBar
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 3
Tests that did not run due to a developer annotating a test with .skip  Pending 2
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 125
⚠️ You've recorded test results over your free plan limit.
Upgrade your plan to view test results.
View all changes introduced in this branch ↗︎

Zeyber
Zeyber previously approved these changes Dec 10, 2024
developpeurweb
developpeurweb previously approved these changes Dec 11, 2024
@Pio-Bar Pio-Bar dismissed stale reviews from developpeurweb and Zeyber via 22fa783 December 23, 2024 08:54
@github-actions github-actions bot marked this pull request as draft December 23, 2024 08:54
@Pio-Bar Pio-Bar marked this pull request as ready for review December 23, 2024 08:55
Copy link
Contributor

Merge Checks Failed

Please push a commit to re-trigger the build. 
To push an empty commit you can use `git commit --allow-empty -m "Trigger Build"`

@github-actions github-actions bot marked this pull request as draft December 23, 2024 08:59
@Pio-Bar Pio-Bar marked this pull request as ready for review December 23, 2024 09:28
@StanislavSukhanov StanislavSukhanov self-requested a review December 23, 2024 09:59
@github-actions github-actions bot marked this pull request as draft January 8, 2025 09:03
@Pio-Bar Pio-Bar marked this pull request as ready for review January 8, 2025 09:03
@github-actions github-actions bot marked this pull request as draft January 8, 2025 12:23
@Pio-Bar Pio-Bar marked this pull request as ready for review January 8, 2025 12:25
@Pio-Bar Pio-Bar merged commit 1016f2d into develop Jan 8, 2025
28 checks passed
@Pio-Bar Pio-Bar deleted the feature/CXSPA-9028 branch January 8, 2025 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants