Skip to content

docs(codemods): add migration entry for Tabs' navigation-usage move to TabNavigation - #3062

Merged
mfal merged 2 commits into
mainfrom
claude/tabs-component-migration-4f237b
Sep 1, 2026
Merged

docs(codemods): add migration entry for Tabs' navigation-usage move to TabNavigation#3062
mfal merged 2 commits into
mainfrom
claude/tabs-component-migration-4f237b

Conversation

@mfal

@mfal mfal commented Sep 1, 2026

Copy link
Copy Markdown
Member

Summary

  • Adds a migration catalogue entry documenting that Tabs' rendering changed in 0.2.0-alpha.977 (the same release that introduced TabNavigation): full-width equal tabs with a shared sliding indicator replaced the previous fit-content tabs with per-tab static hover/pressed/selected backgrounds.
  • No code change is needed for Tabs used to switch content within a page.
  • Consumers who used Tabs/TabTitle's inherited React Aria routing props (href/target/routerOptions) to fake real navigation should move that usage to TabNavigation, which was built for exactly this and deliberately kept the old Tabs look and feel.
  • No codemod: whether a given Tabs usage was real navigation or a genuine content switcher can't be derived from the source alone, same reasoning as the existing SegmentedControl deprecation entry.

Files

  • packages/codemods/src/migrations/tabs-navigation-usage-to-tab-navigation/entry.md (new catalogue entry)
  • packages/codemods/src/tests/remoteScope.test.ts (registers the entry's remote-package targets)
  • packages/codemods/src/migrations.generated.ts, packages/components/MIGRATION.md (regenerated via pnpm nx build codemods)

Test plan

  • pnpm nx build codemods — regenerates migrations.generated.ts and MIGRATION.md
  • pnpm nx test:unit codemods — 308/308 tests pass, including the catalogue/remote-scope consistency checks
  • prettier --check on touched files

🤖 Generated with Claude Code

…o TabNavigation

Tabs' rendering changed in 0.2.0-alpha.977 (the same release that introduced
TabNavigation): full-width equal tabs with a shared sliding indicator replaced
the old fit-content, individually-highlighted tabs. Consumers who exploited
Tabs' inherited React Aria routing props (href/target/routerOptions on
TabTitle) to fake real navigation should move that usage to TabNavigation,
which was built for it and deliberately kept the old Tabs look.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@mfal
mfal requested a review from a team September 1, 2026 13:56
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Coverage Report for ./packages/components/

Status Category Percentage Covered / Total
🔵 Lines 78.69% 746 / 948
🔵 Statements 78.57% 763 / 971
🔵 Functions 80.09% 165 / 206
🔵 Branches 70.33% 377 / 536
File CoverageNo changed files found.
Generated in workflow #6509 for commit bcb4f8c by the Vitest Coverage Report Action

@mfal
mfal enabled auto-merge (squash) September 1, 2026 13:59
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

🚀 Preview Deployment

Preview environments are ready:

Type URL
docs pr-3062.docs.review.flow-components.de
storybook pr-3062.storybook.review.flow-components.de

Images:

  • docs: ghcr.io/mittwald/flow/docs:pr-3062
  • storybook: ghcr.io/mittwald/flow/storybook:pr-3062

Comment thread packages/codemods/src/migrations/tabs-navigation-usage-to-tab-navigation/entry.md Outdated
Comment thread packages/components/MIGRATION.md Outdated
…igration entry

Tabs collapses all-at-once (the whole tab list turns into a single button
opening a context menu with every tab), while TabNavigation collapses
incrementally (items hide one by one into a "More" menu as space runs out).
The entry previously claimed both collapse the same way.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@mfal
mfal merged commit c1d45d4 into main Sep 1, 2026
21 of 22 checks passed
@mfal
mfal deleted the claude/tabs-component-migration-4f237b branch September 1, 2026 14:29
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.

2 participants