Skip to content

[Website] Preserve example tabs across the RSC boundary - #10136

Merged
tobias-tengler merged 2 commits into
mainfrom
tte/fix-tabs-on-website
Jul 22, 2026
Merged

tobias-tengler merged 2 commits into
mainfrom
tte/fix-tabs-on-website

Conversation

@tobias-tengler

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings July 22, 2026 09:58

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes ExampleTabs marker detection across the Next.js RSC → client boundary by switching from component-identity checks (child.type) to a serializable discriminator (exampleTab) passed on the marker elements.

Changes:

  • Reworked ExampleTabs to group tab content by a string union (ExampleTabKind) read from marker props rather than relying on child.type.
  • Added an exampleTab discriminator to marker props and updated tab ordering/labeling to use the new kind.
  • Updated mdx-components.tsx to wrap MDX marker components and inject the correct exampleTab value.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
website/src/components/ExampleTabs.tsx Switches marker detection from element type identity to a stable exampleTab discriminator.
website/mdx-components.tsx Wraps MDX marker components to inject exampleTab values for client-side inspection.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@tobias-tengler
tobias-tengler merged commit 86231bf into main Jul 22, 2026
17 checks passed
@tobias-tengler
tobias-tengler deleted the tte/fix-tabs-on-website branch July 22, 2026 10:17
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.

2 participants