Optimize mobile navigation sidebar responsiveness#1784
Conversation
Added media query for mobile navigation sidebar optimization.
|
@nancy-verma780 is attempting to deploy a commit to the recode Team on Vercel. A member of the Team first needs to authorize it. |
|
Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. The estimated time for response is 5–8 hrs. In the meantime, please provide all necessary screenshots and make sure you run - npm build run , command and provide a screenshot, a video recording, or an image of the update you made below, which helps speed up the review and assignment. If you have questions, reach out to LinkedIn. Your contributions are highly appreciated!😊 Note: I maintain the repo issue every day twice at 8:00 AM IST and 9:00 PM IST. If your PR goes stale for more than one day, you can tag and comment on this same issue by tagging @sanjay-kv. We are here to help you on this journey of open source. Consistent 20 contributions are eligible for sponsorship 💰 🎁 check our list of amazing people we sponsored so far: GitHub Sponsorship. ✨ 📚Your perks for contribution to this community 👇🏻
If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊 |
|
Hi @sanjay-kv, I have successfully updated my fork and submitted the Pull Request to fix the mobile sidebar responsive rules under 480px! You can find the PR here: #1784. Please review and merge it. Thank you! |
| @media screen and (max-width: 480px) { | ||
| .sidebar_src-theme-Navbar-ResponsiveSidebar-styles-module, | ||
| .navbar-sidebar, | ||
| .menu { |
There was a problem hiding this comment.
Hi @nancy-verma780 this issue is not assigned yet to raise PR, infact we asked for more details in the issue
|
Hi @sanjay-kv, my sincere apologies for opening the Pull Request before receiving formal assignment confirmation! I got excited about fixing the mobile responsive layout bug. To answer your request for more details, here is the structural breakdown:
Could you please assign this issue to me so we can officially move forward with reviewing the open PR (#1784)? Thank you for your guidance! |
|
thats alright, thanks for understanding, i will run the vercel CI and assign @Adez017 to review it. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Hi @sanjay-kv and @Adez017, I have successfully pushed the layout fixes directly to this branch (Commit: 46ec816). The code resolves the navbar placeholder height calculation issues and cleans up the mobile menu positioning boundaries. The lint checks have passed successfully! However, the Vercel preview deployment is currently waiting for maintainer authorization. Could you please click "Approve and run" on the Vercel check so the final layout can compile and render for review? Thank you! |
Added media query for mobile navigation sidebar optimization.
Description
Fixes #1784
This PR resolves a mobile layout bug where the responsive navigation sidebar menu items and layout placeholders clashed on screens below 480px. The updated style sheet handles viewport safety boundaries dynamically.
Type of Change
Changes Made
@media screen and (max-width: 1024px)breakpoint to clean up.theme-doc-navbar-placeholderlayout heights.@media screen and (max-width: 480px)for.navbar-sidebar .menu__list-itemto fix content overlap errors.Dependencies
Checklist
npm run buildand verified the build succeeds cleanly.