Skip to content

fix: Fix Table disabled state + Safari loadMore and scroll Combobox selected item into view when opening via click #8224

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

LFDanLu
Copy link
Member

@LFDanLu LFDanLu commented May 12, 2025

caught by chromatic

✅ Pull Request Checklist:

  • Included link to corresponding React Spectrum GitHub Issue.
  • Added/updated unit tests and storybook for this change (for new code or code which already has tests).
  • Filled out test instructions.
  • Updated documentation (if it already exists for this component).
  • Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Project:

RSP

@rspbot
Copy link

rspbot commented May 12, 2025

snowystinger
snowystinger previously approved these changes May 12, 2025
…fari

position absolute/relative is not really well supported in Safari making the loading sentinel not scroll into view when scrolling the table
@LFDanLu
Copy link
Member Author

LFDanLu commented May 13, 2025

https://www.chromatic.com/build?appId=5f0dd5ad2b5fc10022a2e320&number=948, can ignore the one failure, that is known and unrelated to this PR

@rspbot
Copy link

rspbot commented May 13, 2025

@LFDanLu LFDanLu changed the title fix: Fix select all checkbox disabled state and other empty state issues when loading sentinel is provided fix: Fix select all checkbox disabled state when loading sentinel is provided and Safari loadMore when non-virtualized May 13, 2025
@LFDanLu LFDanLu changed the title fix: Fix select all checkbox disabled state when loading sentinel is provided and Safari loadMore when non-virtualized fix: Fix Table disabled state + Safari loadMore and scroll Combobox selected item into view when opening via click May 13, 2025
Copy link
Member Author

@LFDanLu LFDanLu May 13, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For now I'm skipping these tests until we do more testing and confirm if we want to focus + scroll the ComboBox selected item into view when opening the drop down via click. These tests originate from #1014 but I don't remember/can't find any reason why we don't want this behavior other than a214902. The NVDA problem should apply since we still clear focus when the input field is modified, but will test

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One potential weird point may be if you have a selected key and allowsCustomValue, where then if you backspace once the dropdown will open and an item will be focused, preventing a subsequent ENTER keypress from submitting your custom value

@rspbot
Copy link

rspbot commented May 13, 2025

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants