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: future Select and FilterSelect page jump on focus issue #5123

Merged

Conversation

mcwinter07
Copy link
Contributor

@mcwinter07 mcwinter07 commented Oct 3, 2024

Why

  • adds a fix for the recurring select jump issue that our consumers experience when opening the menu in the future Select and Filter select via keyboard / mouse click

What

  • Adds a new isClientReady hook to determine if window and document objects are available to our components
    • This can save us having to double up on useEffect's when using these or browser APIs
  • Update ListBox to manually set focus to circumvent the page jump issue
  • Add tests to FilterSelect to test arrow up and down open functionality

Copy link

changeset-bot bot commented Oct 3, 2024

🦋 Changeset detected

Latest commit: 42ceda4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@kaizen/components Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@mcwinter07 mcwinter07 force-pushed the KZN-2719/future-and-filter-select-page-jump-on-focus-fix branch from 7d6e76e to 6deb104 Compare October 3, 2024 06:56
@mcwinter07 mcwinter07 force-pushed the KZN-2719/future-and-filter-select-page-jump-on-focus-fix branch from 40dc332 to 5a21407 Compare October 3, 2024 23:10
@mcwinter07 mcwinter07 changed the title Kzn 2719/future and filter select page jump on focus fix Fix: future Select and FilterSelect page jump on focus issue Oct 3, 2024
@mcwinter07 mcwinter07 marked this pull request as ready for review October 3, 2024 23:21
@mcwinter07 mcwinter07 requested a review from a team as a code owner October 3, 2024 23:21
Copy link
Contributor

github-actions bot commented Oct 3, 2024

✨ Here is your branch preview! ✨

Last updated for commit 42ceda4: Update packages/components/src/future/Select/subcomponents/ListBox/ListBox.tsx

@mcwinter07 mcwinter07 merged commit cdabe86 into main Oct 8, 2024
19 checks passed
@mcwinter07 mcwinter07 deleted the KZN-2719/future-and-filter-select-page-jump-on-focus-fix branch October 8, 2024 03:20
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.

3 participants