Skip to content

chore: fix React Aria home page "bring your styles" button in light mode#10242

Open
LFDanLu wants to merge 3 commits into
mainfrom
fix_aria_home_page
Open

chore: fix React Aria home page "bring your styles" button in light mode#10242
LFDanLu wants to merge 3 commits into
mainfrom
fix_aria_home_page

Conversation

@LFDanLu

@LFDanLu LFDanLu commented Jun 22, 2026

Copy link
Copy Markdown
Member

from testing

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

Test the React Aria home page in light and dark mode. The tabs under "Bring your own styles" should render properly in both

🧢 Your Project:

RSP

@rspbot

rspbot commented Jun 22, 2026

Copy link
Copy Markdown

);
}

export function RACSettingsProvider({children}: Pick<SettingsProviderProps, 'children'>) {

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The SettingsProvider renders an extra S2 Provider that adds a div with isolation: isolate which seems to break the tab styling. However, this actually didn't fix the "Explore Components" button hmm

Comment thread packages/dev/s2-docs/src/SettingsProvider.tsx
@rspbot

rspbot commented Jun 23, 2026

Copy link
Copy Markdown

@rspbot

rspbot commented Jun 23, 2026

Copy link
Copy Markdown

variant: {
primary: 'auto',
secondary: 'white',
secondary: {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

So now we need to do an actual release for this code change, it's not just docs? What happened here?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I messed up the styles and our chromatic didn't catch them. I'd originally gone this route, but ran into Error evaluating macro: transparent-overlay-1000 does not support states and wrongly decided that just going with a static value like the others would work. I didn't test it against enough backgrounds.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

specifically i messed it up prior to the last release, so this is pre-existing at the moment
73bc1db

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants