SLP_Software/slp-modular-cms
1
0
Fork 0
Code Issues Pull Requests Actions Projects Releases Wiki Activity

Fixes mobile sidebar collapsing to content height instead of filling the overlay #3

Merged
Sluijsens merged 1 commits from fix/mobile-sidebar-height into master 2026-07-31 00:47:19 +02:00
Conversation 0 Commits 1 Files Changed 1
+1 -1
Sluijsens commented 2026-07-31 00:44:06 +02:00
Owner
Copy Link
Copy Source

The aside had no explicit height, so it stretched to full height on
desktop only because its flex-row parent applies align-items:stretch.
In the mobile SidebarOverlay it sits in a plain (non-flex) h-full div,
so it shrank to fit its nav items, pushing settings/profile up under
the nav instead of staying pinned to the bottom.

The aside had no explicit height, so it stretched to full height on desktop only because its flex-row parent applies align-items:stretch. In the mobile SidebarOverlay it sits in a plain (non-flex) h-full div, so it shrank to fit its nav items, pushing settings/profile up under the nav instead of staying pinned to the bottom.
Sluijsens added 1 commit 2026-07-31 00:44:07 +02:00
Fixes mobile sidebar collapsing to content height instead of filling the overlay
Continuous Integration / config (pull_request) Successful in 9s
Details
Continuous Integration / backend-build (pull_request) Successful in 5m9s
Details
Continuous Integration / vulnerability-scan (pull_request) Successful in 4m28s
Details
Continuous Integration / frontend-prepare (pull_request) Successful in 1m53s
Details
Continuous Integration / backend-test (pull_request) Successful in 5m29s
Details
Continuous Integration / frontend-build (pull_request) Successful in 2m16s
Details
Continuous Integration / frontend-test (pull_request) Successful in 4m42s
Details
Continuous Integration / frontend-lint (pull_request) Successful in 1m54s
Details
Continuous Integration / publish-production (pull_request) Skipped
Details
Continuous Integration / deploy-production (pull_request) Skipped
Details
Continuous Integration / publish-test (pull_request) Successful in 6m37s
Details
Continuous Integration / deploy-test (pull_request) Skipped
Details
bc4de27b97 
The aside had no explicit height, so it stretched to full height on
desktop only because its flex-row parent applies align-items:stretch.
In the mobile SidebarOverlay it sits in a plain (non-flex) h-full div,
so it shrank to fit its nav items, pushing settings/profile up under
the nav instead of staying pinned to the bottom.
Sluijsens marked the pull request as ready for review 2026-07-31 00:46:46 +02:00
Sluijsens merged commit 66c0d768cf into master 2026-07-31 00:47:19 +02:00
Sluijsens deleted branch fix/mobile-sidebar-height 2026-07-31 00:47:19 +02:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
Sluijsens
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SLP_Software/slp-modular-cms#3