Skip to content

Conversation

@devongovett
Copy link
Member

Sometimes the code examples don't display every file that is needed at once because there'd be way too many tabs and we want to focus on the relevant content. But users might still want to access other files to copy and paste them. This makes imported files in code examples into links to switch tabs. If you click one that isn't already in a tab, a tab will be added to display that file on demand.

@rspbot
Copy link

rspbot commented Nov 17, 2025

snowystinger
snowystinger previously approved these changes Nov 18, 2025
LFDanLu
LFDanLu previously approved these changes Nov 19, 2025
Copy link
Member

@LFDanLu LFDanLu left a comment

Choose a reason for hiding this comment

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

LGTM for the most part. I noticed that I couldn't expand the css file under Toolbar.tsx in the RAC toolbar docs even though it was a link. If you open many files, the quiet picker that it switches to is too long but I think we already noted this separately,

Image

Approving anyways if you wanna handle this in a followup, definitely a improvement in the experience IMO

@devongovett
Copy link
Member Author

I noticed that I couldn't expand the css file under Toolbar.tsx in the RAC toolbar docs even though it was a link

oh I think this is a bug we have in a few components where the code is actually the vanilla css example instead of the tailwind one. must be a copy paste issue. we can fix that separately.

@devongovett devongovett dismissed stale reviews from LFDanLu and snowystinger via 9ac075a November 22, 2025 01:52
@rspbot
Copy link

rspbot commented Nov 22, 2025

@devongovett devongovett merged commit f61e75c into main Nov 22, 2025
32 checks passed
@devongovett devongovett deleted the tab-links branch November 22, 2025 21:34
bsmiley1 pushed a commit to bsmiley1/react-spectrum that referenced this pull request Dec 5, 2025
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.

5 participants