Skip to content

Conversation

@snowystinger
Copy link
Member

Closes

✅ 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:

…into fix-mobile-sorting

# Conflicts:
#	packages/dev/s2-docs/src/ComponentCardView.tsx
@rspbot
Copy link

rspbot commented Nov 11, 2025

LFDanLu
LFDanLu previously approved these changes Nov 11, 2025
@rspbot
Copy link

rspbot commented Nov 11, 2025

@rspbot
Copy link

rspbot commented Nov 11, 2025

reidbarber
reidbarber previously approved these changes Nov 11, 2025
Comment on lines 26 to 27
<InternalCardViewContext.Provider value={{ElementType: ListBoxItem, layout: 'grid'}}>
<ListBox
Copy link
Member

Choose a reason for hiding this comment

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

I think we need to keep this as a ListBox for the virtual focus to work right? What was broken here that made this change necessary?

Copy link
Member Author

Choose a reason for hiding this comment

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

gah, how have so many merges gone wrong in this pr.... i didn't notice this locally it should only have the date in the type

@rspbot
Copy link

rspbot commented Nov 11, 2025

@snowystinger
Copy link
Member Author

closing in favour of #9159

@snowystinger snowystinger deleted the fix-mobile-sorting branch November 12, 2025 19:54
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.

5 participants