Skip to content

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

Merged
nfebe merged 1 commit into
stable8from
backport/8447/stable8
May 7, 2026
Merged

[stable8] feat(NcAppSidebarTab): Redesign active tab as rounded filled pill#8501
nfebe merged 1 commit into
stable8from
backport/8447/stable8

Conversation

@nfebe

@nfebe nfebe commented May 5, 2026

Copy link
Copy Markdown
Contributor

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.

@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

codecov Bot commented May 5, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 62.16216% with 14 lines in your changes missing coverage. Please review.
✅ Project coverage is 46.86%. Comparing base (a3a1c1d) to head (dbb8fcb).
⚠️ Report is 9 commits behind head on stable8.

Files with missing lines Patch % Lines
src/components/NcAppSidebar/NcAppSidebarTabs.vue 57.57% 14 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           stable8    #8501      +/-   ##
===========================================
+ Coverage    46.84%   46.86%   +0.02%     
===========================================
  Files          195      195              
  Lines         4921     4923       +2     
  Branches      1205     1205              
===========================================
+ Hits          2305     2307       +2     
  Misses        2530     2530              
  Partials        86       86              

☔ 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.

@susnux susnux 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.

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
@nfebe nfebe force-pushed the backport/8447/stable8 branch from a7cce51 to 9853a91 Compare May 6, 2026 09:55
@nfebe

nfebe commented May 6, 2026

Copy link
Copy Markdown
Contributor Author

where is the legacy fallback?

Forgot to update the local testing branch which is now the backport. Now done and pushed. Thank you.

The active sidebar tab now renders as a fully-rounded filled pill with
an inset, rounded blue indicator that grows into place when selected,
replacing the previous full-width bottom border treatment on Nextcloud
34+. Older Nextcloud versions keep the legacy border styling.

Backport of #8447 to stable8.

Signed-off-by: nfebe <fenn25.fn@gmail.com>
@nfebe nfebe force-pushed the backport/8447/stable8 branch from 9853a91 to dbb8fcb Compare May 6, 2026 10:00
@nfebe nfebe requested a review from susnux May 6, 2026 10:14
@ShGKme ShGKme added the enhancement New feature or request label May 7, 2026
@nfebe nfebe merged commit 697a3ee into stable8 May 7, 2026
19 checks passed
@nfebe nfebe deleted the backport/8447/stable8 branch May 7, 2026 11:15
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 enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants