Skip to content

fix: improve mobile navigation drawer responsiveness#40

Open
pragya0129 wants to merge 6 commits into
Areeb-coder:mainfrom
pragya0129:fix/mobile-nav-overlay
Open

fix: improve mobile navigation drawer responsiveness#40
pragya0129 wants to merge 6 commits into
Areeb-coder:mainfrom
pragya0129:fix/mobile-nav-overlay

Conversation

@pragya0129
Copy link
Copy Markdown
Contributor

This PR fixes the issues where the updated mobile navbar was not responsive and not functional.

Checks

  • All nav links are working now
  • While nav menu is open you cannot interact with the website at the back
  • Added blurr overlay behind the open menu
  • Fixed overlapping and click through issues
  • Clicking outside the menu now closes the menu
  • Preserved existing desktop behaviour

Areeb-coder and others added 5 commits May 18, 2026 01:41
This fix resolves a critical UX issue where mobile users were missing core navigation links in the header, leaving only the "Sign In" button. It introduces a responsive, fully animated hamburger menu dropdown that houses all section links and neatly integrates the "Sign In" CTA, restoring full site navigation for smaller viewports while preserving the platform's literary aesthetic.
- Integrated password visibility toggle button into the login and signup forms.
- Used Lucide React Eye and EyeOff icons for dynamic state indicators.
- Added accessibility labels (`aria-label`) for screen reader compatibility.
- Maintained existing responsive UI styling and layout continuity.
Fixes password strength validation on signup.
@vercel
Copy link
Copy Markdown

vercel Bot commented May 19, 2026

@pragya0129 is attempting to deploy a commit to the areeb-coder's projects Team on Vercel.

A member of the Team first needs to authorize it.

@pragya0129
Copy link
Copy Markdown
Contributor Author

Hey @Areeb-coder I have done the required changes. Please Review them!.

One small request. Kindly please have a look at the official gssoc website screenshot below, with the gssoc:appropriate labels you are adding level and type directly but it requires you to add it like the way it is mentioned. Kindly please add proper labels to this PR and the other two PRs that you merged before . Thanks, Sorry for the inconvenience.

image

@pragya0129
Copy link
Copy Markdown
Contributor Author

Hey @Areeb-coder kindly please review and merge this PR. Thanks!

@pragya0129
Copy link
Copy Markdown
Contributor Author

Hey @Areeb-coder I have resolved the conflicts. Now you can merge these. It is in reference to the issues you told in previous PR comment. Thanks!

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