Skip to content

feat(ui): share segmented item layout styles with Tabs - #837

Merged
pasqualevitiello merged 1 commit into
mainfrom
fix/segmented-control-item-layout
Aug 19, 2026
Merged

feat(ui): share segmented item layout styles with Tabs#837
pasqualevitiello merged 1 commit into
mainfrom
fix/segmented-control-item-layout

Conversation

@pasqualevitiello

Copy link
Copy Markdown
Contributor

Summary

  • Export segmentedControlItemLayoutClassName for the shared icon gap, size, and opacity treatment
  • Include that class in segmentedControlItemVariants and reuse it on TabsTab instead of inlined utilities
  • Document the new export in the changelog, segmented-control docs, and skill reference

Test plan

  • Check default, small, and large Tabs with icons (p-tabs-6 / p-tabs-8) for aligned gap, size, and opacity
  • Check segmented Radio Group and navigation particles still look correct
  • Confirm underline Tabs still use their own indicator and focus styles
  • Spot-check that existing Tabs JSX does not need a size or class change

Made with Cursor

Extract icon gap, size, and opacity into segmentedControlItemLayoutClassName so radio, navigation, and Tabs stay optically aligned.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel

vercel Bot commented Aug 19, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
coss-com Ready Ready Preview Aug 19, 2026 7:29am
coss-com-ui Ready Ready Preview Aug 19, 2026 7:29am
coss-examples-calcom Ready Ready Preview Aug 19, 2026 7:29am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
coss-com-origin Skipped Skipped Aug 19, 2026 7:29am

Request Review

@pasqualevitiello
pasqualevitiello merged commit f27d668 into main Aug 19, 2026
13 checks passed
@pasqualevitiello
pasqualevitiello deleted the fix/segmented-control-item-layout branch August 19, 2026 07:35
@linear-code

linear-code Bot commented Aug 19, 2026

Copy link
Copy Markdown

ENG-3729

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.

3 participants