feat/mobile header + home page UI - #92
Merged
Merged
Conversation
Header collapses to a mobile layout below lg (avatar, centred logo, hamburger) with a slide-in drawer for navigation. Between lg and xl the desktop pill nav renders at a compact size so it fits 1024px - previously it needed ~1110px minimum and would overflow anywhere below xl. The drawer closes on link tap, backdrop click, X and Escape. Its scroll lock is scoped to the drawer's own breakpoint so widening past lg always releases the page, even if the drawer's open state has not caught up. Profile/Admin/Logout move into the drawer on mobile, since the desktop profile dropdown is hidden below lg. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
harrywu23
changed the title
…general alignment and spacing of elements on the page aswl as font size.
…and alignment with what we do card.
RLee64
approved these changes
Sep 7, 2026
RLee64
left a comment
RLee64
left a comment
Collaborator
There was a problem hiding this comment.
Lgtm, we prolly wanna get this merged in asap so others can have a functioning header to work with. That said, there are some small issues that still need fixing:
Join Us button size on home page
issue with gallery display (ipad pro)
trying to drag the images in the bg (instead of fg) on the slider makes them disappear
Aside from that, everything else looks great!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes. I think maybe the only thing im unsure is the sizing of the "discover more" and "more events" buttons. They look different to what is on the figma but also this is probably just the styling that was implemented by whoever made the homepage originally so idk im just following it. Also one minor difference is the spacing is slightly off from the "more events button" downwards so like if thats a concern you can raise it too and ill change it lmfao.
Edit: i noticed the design of the events gallery styling is slightly diff if that needs to be changed lemme know.