Skip to content

feat(#35): add responsive hamburger menu and fix sidebar visibility#88

Merged
hectorguillen-blip merged 1 commit into
7369-admin-angular-upgradefrom
feature/responsive-hamburger-menu
May 27, 2026
Merged

feat(#35): add responsive hamburger menu and fix sidebar visibility#88
hectorguillen-blip merged 1 commit into
7369-admin-angular-upgradefrom
feature/responsive-hamburger-menu

Conversation

@hectorguillen-blip

Copy link
Copy Markdown

Description

Adds a responsive hamburger menu to the admin tool header and fixes the sidebar navigation visibility on narrow viewports.

Closes #35

Changes

Hamburger menu in header

  • Added a hamburger button () to the header that appears on narrow viewports (mobile/tablet)
  • Clicking it toggles the app link and logout button, which are hidden by default on mobile
  • Toggle state is managed locally in HeaderComponent via menuOpen boolean
  • New header.component.less added with mobile-only hamburger styles

Sidebar always visible

  • Fixed the sidebar disappearing on narrow viewports — it now stays in place at all screen sizes
  • Sidebar shrinks naturally as the viewport narrows, allowing nav link text to wrap onto multiple lines
  • Removed the old position: fixed overlay behaviour from sidebar.component.less
  • Updated common.less with flex-wrap: nowrap and flex-shrink: 1 to keep sidebar inline

Checklist

  • Readable — follows style guide, ESLint passing
  • UI/UX backwards compatible — sidebar and header behaviour matches the original tool at all viewport sizes
Screenshot 2026-05-26 at 2 50 03 PM

@hectorguillen-blip hectorguillen-blip force-pushed the feature/responsive-hamburger-menu branch from 6cd7583 to 2d58650 Compare May 26, 2026 20:53

@jomcarvajal jomcarvajal left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@hectorguillen-blip hectorguillen-blip merged commit 122c494 into 7369-admin-angular-upgrade May 27, 2026
184 of 191 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.

2 participants