Skip to content

Feature/183 responsive navigation#216

Merged
llinsss merged 2 commits into
DogStark:mainfrom
rejoicetukura-blip:feature/183-responsive-navigation
Mar 26, 2026
Merged

Feature/183 responsive navigation#216
llinsss merged 2 commits into
DogStark:mainfrom
rejoicetukura-blip:feature/183-responsive-navigation

Conversation

@rejoicetukura-blip

Copy link
Copy Markdown
Contributor

close #183

- Add AppLayout shell with desktop sidebar + mobile bottom nav
- Sidebar: collapsible (w-64/w-16), nested menu items with expand/collapse,
  persists collapsed state in localStorage, search integrated
- BottomNav: 5-tab mobile nav (Home, Pets, Appointments, Search, Profile),
  active indicator, auth-aware tab visibility
- MobileHeader: sticky top bar with hamburger drawer, inline search toggle,
  full drawer with nested nav items and user info
- Breadcrumbs: auto-generated from route segments, shown on deep pages
- NavSearch: debounced autocomplete, keyboard nav (Enter/Escape), router push
- NavIcon: self-contained SVG icon map, zero extra dependencies
- navConfig: single source of truth for all nav items and auth requirements
- AppLayout: skips shell for auth/landing pages, wraps all other pages
- ARIA labels, aria-current, aria-expanded, focus-visible rings throughout
- iOS safe-area-inset-bottom support for bottom nav
- Smooth CSS transitions on sidebar collapse and drawer slide

Closes DogStark#183
@drips-wave

drips-wave Bot commented Mar 25, 2026

Copy link
Copy Markdown

@rejoicetukura-blip Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@llinsss llinsss merged commit 1b93906 into DogStark:main Mar 26, 2026
1 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Mobile] Responsive Navigation System

2 participants