Skip to content

[stable8] feat(NcAppSidebarTab): Redesign active tab as rounded filled pill#8501

Open
nfebe wants to merge 1 commit intostable8from
backport/8447/stable8
Open

[stable8] feat(NcAppSidebarTab): Redesign active tab as rounded filled pill#8501
nfebe wants to merge 1 commit intostable8from
backport/8447/stable8

Conversation

@nfebe
Copy link
Copy Markdown
Contributor

@nfebe nfebe commented May 5, 2026

The active sidebar tab now renders as a rounded surface backed by a neutral hover-tone fill, with a wide rounded indicator that grows into place when selected. The icon slot now receives the active state so consumers can render filled/outlined icon variants.

Backport of #8447 to stable8.

The active sidebar tab now renders as a rounded surface backed by a
neutral hover-tone fill, with a wide rounded indicator that grows into
place when selected. The icon slot now receives the active state so
consumers can render filled/outlined icon variants.

Backport of #8447 to stable8.

Signed-off-by: nfebe <fenn25.fn@gmail.com>
@nfebe nfebe added 3. to review Waiting for reviews design Design, UX, interface and interaction design labels May 5, 2026
@nfebe nfebe requested review from ShGKme and susnux May 5, 2026 15:24
@codecov
Copy link
Copy Markdown

codecov Bot commented May 5, 2026

Codecov Report

❌ Patch coverage is 35.71429% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 46.84%. Comparing base (1010fb5) to head (a7cce51).
⚠️ Report is 28 commits behind head on stable8.

Files with missing lines Patch % Lines
src/components/NcAppSidebar/NcAppSidebarTabs.vue 35.71% 9 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           stable8    #8501      +/-   ##
===========================================
+ Coverage    46.53%   46.84%   +0.30%     
===========================================
  Files          194      195       +1     
  Lines         4891     4921      +30     
  Branches      1201     1268      +67     
===========================================
+ Hits          2276     2305      +29     
+ Misses        2529     2528       -1     
- Partials        86       88       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Copy Markdown
Contributor

@susnux susnux left a comment

Choose a reason for hiding this comment

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

where is the legacy fallback?

@ShGKme ShGKme changed the title feat(sidebar-tabs): Redesign active tab as rounded filled pill [stable8] feat(NcAppSidebarTab): Redesign active tab as rounded filled pill May 5, 2026
@ShGKme ShGKme added this to the 8.39.0 milestone May 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews design Design, UX, interface and interaction design

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants