Skip to content

fix:Hamburger opening issue - #1150

Open
Ash-Misty wants to merge 3 commits into
komalharshita:mainfrom
Ash-Misty:main
Open

fix:Hamburger opening issue#1150
Ash-Misty wants to merge 3 commits into
komalharshita:mainfrom
Ash-Misty:main

Conversation

@Ash-Misty

Copy link
Copy Markdown

Pull Request

Summary [required]

Improved the mobile navigation experience by fixing the misbehavior of hamburger menu drawer. Added a modern slide-in menu with smoother animations, better spacing, improved accessibility, and a dedicated overlay backdrop. The updated design provides a cleaner and more responsive user experience on mobile devices.

Related Issue [required]

Closes #1127

Type of Change [required]

  • Bug fix — resolves a broken behaviour
  • Feature — adds new functionality
  • Data — adds new projects to data/projects.json
  • Documentation — updates docs, README, or code comments only
  • Style — CSS or visual changes only, no logic change
  • Refactor — restructures code without changing behaviour
  • Test — adds or updates tests

What Was Changed [required]

File Change Made

| style.css | Improved spacing and typography for mobile navigation links and made the hamburger menu working |
| index.html | form tag mispplaced |
| script.js | Enhanced hover and focus states for accessibility |

How to Test This PR [required]

  1. Checkout this branch:

    git checkout <branch-name>
  2. Run the project locally.

  3. Open the website in a mobile viewport (375px width recommended).

  4. Click the hamburger menu icon.

  5. Verify the drawer slides in smoothly from the right.

  6. Verify the overlay appears behind the menu.

  7. Verify all navigation links remain functional.

  8. Close the menu using the toggle button and by clicking outside the drawer.

  9. Verify the desktop navigation remains unchanged.

Expected Result

  • Hamburger icon displays correctly.
  • Mobile drawer opens and closes smoothly.
  • Overlay covers the background when the drawer is open.
  • Navigation links remain accessible and responsive.
  • No layout issues occur on mobile screens.

Test Results [required]

Tested manually on:
✓ Mobile View (375px)
✓ Tablet View (768px)
✓ Desktop View (1280px)

No visual regressions observed.

Screenshots (if UI change)

Before After
Old hamburger menu Redesigned mobile drawer

Self-Review Checklist [required]

  • I have read CONTRIBUTING.md and followed all guidelines
  • My branch name follows the convention: style/mobile-menu-redesign
  • I tested the changes manually
  • I have not introduced any debug statements
  • I have not modified files outside the scope of the linked issue
  • I tested the UI at 375px (mobile) and 1280px (desktop)

Notes for Reviewer

This PR focuses exclusively on visual and UX improvements for the mobile navigation drawer. No business logic or navigation functionality was modified.

@vercel

vercel Bot commented Jun 24, 2026

Copy link
Copy Markdown

@Ash-Misty is attempting to deploy a commit to the komalsony234-1530's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot 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.

Thank you for submitting your first pull request to DevPath.

Before review:

  • Complete the PR template fully
  • Ensure all tests pass
  • Link your PR to an issue
  • Keep changes scoped to the issue

A maintainer will review your contribution soon.

@komalharshita

Copy link
Copy Markdown
Owner

@Ash-Misty kindly resolve the merge conflicts

@komalharshita komalharshita added the need review Further information is requested label Jul 16, 2026
@Ash-Misty

Copy link
Copy Markdown
Author

solved the conflicts

@komalharshita komalharshita removed the need review Further information is requested label Jul 25, 2026
@komalharshita

Copy link
Copy Markdown
Owner

@Ash-Misty kindly check again

@komalharshita komalharshita added type:bug Something isn't working need review Further information is requested type:design level:beginner quality:clean bug labels Jul 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Mobile Navigation Menu Displays Blank Screen Before Opening

2 participants