Skip to content

Feat/mobile nav menu 13#75

Merged
codebestia merged 5 commits into
ShadeProtocol:mainfrom
manueldezman:feat/mobile-nav-menu-13
Jun 3, 2026
Merged

Feat/mobile nav menu 13#75
codebestia merged 5 commits into
ShadeProtocol:mainfrom
manueldezman:feat/mobile-nav-menu-13

Conversation

@manueldezman
Copy link
Copy Markdown
Contributor

@manueldezman manueldezman commented May 31, 2026

Closes #13


Summary

Adds a mobile navigation drawer for small screens and moves merchant-facing pages into the shared dashboard shell so the standard sidebar/topbar appears consistently across merchant routes.

Changes

  • Added a mobile-only hamburger menu in the topbar with a slide-out navigation drawer.
  • Shared navigation item definitions between desktop sidebar and mobile drawer.
  • Hid the desktop sidebar on small screens and removed mobile left-offset spacing.
  • Moved merchant routes into the (dashboard) route group while preserving public URLs.
  • Added a placeholder /customers page for the existing Customers nav item.
  • Fixed a JSX unescaped entities lint error in payment instructions copy.

Verification

  • npm run typecheck
  • npm run format:check
  • npm run lint

Screenshot

Screencast.from.2026-05-31.14-21-01.mp4

@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented May 31, 2026

@manueldezman 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

@manueldezman manueldezman force-pushed the feat/mobile-nav-menu-13 branch from b8a28b9 to e852c84 Compare June 3, 2026 11:18
@codebestia
Copy link
Copy Markdown
Contributor

GM @manueldezman
Thanks for the implementation and the video showing the implementation.
Please resolve the merge conflicts.

@manueldezman manueldezman force-pushed the feat/mobile-nav-menu-13 branch from e852c84 to 530e4cb Compare June 3, 2026 17:22
Copy link
Copy Markdown
Contributor

@codebestia codebestia left a comment

Choose a reason for hiding this comment

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

LGTM!
Thanks

@codebestia codebestia merged commit 9c101be into ShadeProtocol:main Jun 3, 2026
1 check failed
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.

Create Mobile Navigation Menu

2 participants