Fix mobile navigation overflow
What kind of change does this PR introduce? Bugfix - Fixed mobile navigation overflow issue and improved accordion behavior
Issue Number:
- Closes #1797
Screenshots/videos:
- Before fix: Navigation dropdowns were overflowing on mobile and multiple sections could be open simultaneously, causing layout issues.
https://github.com/user-attachments/assets/c9767316-a9cf-4812-b853-d680745211c6
- After fix: Only one accordion section can be open at a time, preventing overflow and improving mobile UX.
https://github.com/user-attachments/assets/c2a3450c-7d67-4f32-b6a3-6bed063b96fb
If relevant, did you update the documentation?
- No
Summary This PR fixes the mobile responsiveness issue reported in #1797 where the left navigation drawer's dropdown menus were overflowing from their parent container on mobile devices, breaking the layout and negatively impacting user experience.
Key changes made:
- Fixed typo: Corrected
pathWtihoutFragmenttopathWithoutFragmentthroughout the component - Improved accordion behavior: Implemented
handleAccordionfunction to ensure only one navigation section can be open at a time - Enhanced mobile UX: Prevents content overflow by closing other sections when a new one is opened
Does this PR introduce a breaking change? No, this is a backward-compatible fix that only improves the existing navigation behavior without changing any APIs or interfaces.
Checklist
Please ensure the following tasks are completed before submitting this pull request.
- [x] Read, understood, and followed the contributing guidelines.
built with Refined Cloudflare Pages Action
⚡ Cloudflare Pages Deployment
| Name | Status | Preview | Last Commit |
|---|---|---|---|
| website | ✅ Ready (View Log) | Visit Preview | 09e65a7b8aadfe616e400bbb0364079ff48b7292 |
Hii @benjagm & @Utkarsh-123github, can you please review this PR. Thanks
Hi @vtushar06 , before I review this PR, it would be great if you please fix this failing check. Thank you!
yes @Utkarsh-123github, I tried fixing it, and can you help me out in fixing that.
Codecov Report
:white_check_mark: All modified and coverable lines are covered by tests.
:warning: Please upload report for BASE (main@e97ceb0). Learn more about missing BASE report.
Additional details and impacted files
@@ Coverage Diff @@
## main #1811 +/- ##
========================================
Coverage ? 100.00%
========================================
Files ? 30
Lines ? 633
Branches ? 196
========================================
Hits ? 633
Misses ? 0
Partials ? 0
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
:rocket: New features to boost your workflow:
- :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
- :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
@Utkarsh-123github can you take a look into this, if in case any more changes required.
great job, LGTM👍
Thanks @idanidan29👍