Skip to content

Conversation

@Gourangi4
Copy link

Issue Fixed: #764

🔧 What Changed

  • Added CSS rule .drawer-open .back-to-top to hide the button whenever the mobile drawer is open.
  • Adjusted z-index layering so:
    • Drawer (.mobile-nav) sits above overlay.
    • Overlay covers background but not the drawer.
    • Back-to-Top stays below both.
  • Updated JavaScript toggle to:
    • Add/remove drawer-open on <body>.
    • Ensure Back-to-Top hides while drawer is active.
  • Footer interaction preserved → button hides when footer enters view.

✅ Behavior After Fix

  • Desktop: Back-to-Top works as before (appears on scroll, hides near footer).
  • Mobile:
    • Opening the hamburger menu hides Back-to-Top.
    • Closing the menu restores it (if scroll conditions are met).
    • Button no longer overlaps footer links or drawer UI.

This PR resolves Issue #764 🚀
image

image

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@OpArcade OpArcade added bug Something isn't working level 1 Beginner-level issue labels Aug 26, 2025
@Gourangi4
Copy link
Author

hey @adityai0 , kindly check the pr I have submitted.

@adityai0 adityai0 merged commit 44f44c8 into opensource-society:main Aug 30, 2025
1 check passed
@adityai0 adityai0 added the gssoc2025 This issue is part of GirlScript Summer of Code 2025 label Aug 30, 2025
@Gourangi4 Gourangi4 deleted the issue-764-back-to-top-fix branch August 30, 2025 20:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working gssoc2025 This issue is part of GirlScript Summer of Code 2025 level 1 Beginner-level issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants