psa-systems/mokosh-apps
Watch
9
0
Fork
You've already forked mokosh-apps
 0
Code Issues Pull requests 1 Projects Releases 14 Packages Wiki Activity Actions

MAPPS-346: consistent even spacing across collapsed sidebar icons #409

Merged
vas2000-work merged 1 commit from feat/MAPPS-346-collapsed-nav-even into main 2026-07-02 01:25:27 +02:00
Conversation 0 Commits 1 Files changed 1 +5 -3
vas2000-work commented 2026-07-02 01:22:55 +02:00
Owner
Copy link

Follow-up to #408 (merged). Makes the collapsed rail spacing consistent across every icon instead of spacing the section groups apart. Sidebar scope only.

Background

#408 made the collapsed nav a flex column with justify-between, but each section wraps its items in a block, so the nav's direct flex children were the section groups. justify-between therefore distributed the spare height between groups, giving uneven (by-section) spacing.

What changed (collapsed rail only; expanded unchanged)

  • The collapsed NavSection wrapper is now display: contents (Tailwind contents) instead of a pt-1 space-y-0.5 block. Its icons become direct flex children of the nav.
  • The nav keeps flex flex-col justify-between gap-0.5 from #408, so now every icon (Dashboard plus all section items) is distributed with one consistent gap; gap-0.5 remains the minimum and it still degrades to flex-start + scroll on overflow.

Net: consistent even spacing across all collapsed icons.

#MAPPS-346

Follow-up to #408 (merged). Makes the collapsed rail spacing consistent across every icon instead of spacing the section groups apart. Sidebar scope only. ## Background #408 made the collapsed nav a flex column with `justify-between`, but each section wraps its items in a block, so the nav's direct flex children were the section groups. `justify-between` therefore distributed the spare height between groups, giving uneven (by-section) spacing. ## What changed (collapsed rail only; expanded unchanged) - The collapsed `NavSection` wrapper is now `display: contents` (Tailwind `contents`) instead of a `pt-1 space-y-0.5` block. Its icons become direct flex children of the nav. - The nav keeps `flex flex-col justify-between gap-0.5` from #408, so now every icon (Dashboard plus all section items) is distributed with one consistent gap; `gap-0.5` remains the minimum and it still degrades to `flex-start` + scroll on overflow. Net: consistent even spacing across all collapsed icons. #MAPPS-346
feat(sidebar): even icon spacing on the collapsed rail
All checks were successful
Check / fmt + clippy + tests (pull_request) Successful in 2m28s
Details
Create release / Create release from merged PR (pull_request) Has been skipped
Details
c39157fc5d 
The collapsed spread was distributing space between section groups, not individual icons, because each section wrapped its items in a block. Set the collapsed section wrapper to display: contents so its icons become direct flex children of the nav. The nav's justify-between + gap-0.5 now spreads every icon with one consistent gap instead of clustering by section. Expanded rail is unchanged.

#MAPPS-346

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
vas2000-work scheduled this pull request to auto merge when all checks succeed 2026-07-02 01:23:26 +02:00
vas2000-work deleted branch feat/MAPPS-346-collapsed-nav-even 2026-07-02 01:25:27 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
psa-systems/mokosh-apps!409
No description provided.