website icon indicating copy to clipboard operation
website copied to clipboard

fix: sidebar overflow on mobile #1949

Open DivyanshuVortex opened this issue 1 month ago • 2 comments

What kind of change does this PR introduce?
Bugfix + Minor UI improvement

Issue Number:

  • Closes #1949

Screenshots/videos:
Add screenshots or screen recordings showing the sidebar overflow before/after. Add before/after for homepage link styling.

If relevant, did you update the documentation?
Not required.


Summary

This PR includes two related UI improvements:


1. Fix: Mobile sidebar overflow (Issue #1949)

On screens below 1024px, opening the sidebar caused the page to become wider than the viewport — triggering horizontal scrolling and shifting the layout.

The fix ensures:

  • Sidebar stays within viewport width
  • No horizontal overflow on open/close
  • Main content remains stable
  • Correct overflow-x-hidden and layout constraints applied

2. UI improvement: Simplified homepage email and link styling

The “Email us” link on the homepage previously looked inconsistent with other links and required hover to appear interactive.

Updated styling:

  • Uses standard underline styling
  • Adds a hover effect
  • Cleaner and more consistent across breakpoints
  • No layout shift, works better on small screens

This improves clarity and accessibility without major design changes.


Does this PR introduce a breaking change?

No.


Checklist

Before - Screenshot (282)

Screenshot (283)

After - Screenshot (281) Screenshot (284)

DivyanshuVortex avatar Nov 23 '25 18:11 DivyanshuVortex

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name Status Preview Last Commit
website ✅ Ready (View Log) Visit Preview d7076e42ff05db4342557d1b3f3a36f8a6c5de49

github-actions[bot] avatar Nov 23 '25 18:11 github-actions[bot]

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 100.00%. Comparing base (dcf4ea4) to head (d7076e4). :warning: Report is 26 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main     #1950   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           30        30           
  Lines          633       633           
  Branches       196       196           
=========================================
  Hits           633       633           

: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.

codecov[bot] avatar Nov 23 '25 18:11 codecov[bot]

Hey @Utkarsh-123github please review it

DivyanshuVortex avatar Dec 13 '25 18:12 DivyanshuVortex

Sir @Utkarsh-123github , I make all the required changes in the Tailwind classes pls have a look .

DivyanshuVortex avatar Dec 16 '25 07:12 DivyanshuVortex