Skip to content

Add scroll up#112

Open
Diya-od wants to merge 6 commits into
codepvg:mainfrom
Diya-od:add-scroll-up
Open

Add scroll up#112
Diya-od wants to merge 6 commits into
codepvg:mainfrom
Diya-od:add-scroll-up

Conversation

@Diya-od
Copy link
Copy Markdown
Contributor

@Diya-od Diya-od commented Jun 5, 2026

Description

Added a Scroll-to-Top button on the leaderboard page to improve navigation and user experience. Users can now quickly return to the top of the page after scrolling through the leaderboard.

Linked Issue

Fixes #89

Changes Made

-Added a Scroll-to-Top button to the leaderboard page.
-Implemented smooth scrolling behavior when the button is clicked.
Configured the button to appear only after the user scrolls down a certain distance.
-Styled the button to match the existing UI design.

Type of Change

  • [ x] Bug fix
  • [ x] New feature
  • UI/Visual update
  • Documentation update
  • Refactor

Testing

  • [ x] Tested locally
  • [ x] Tested on mobile viewport (if applicable)
  • [ x] No console errors introduced

Checklist

  • [ x] My code follows the project's coding style
  • [ x] I have formatted my code locally using Prettier
  • [x ] I have performed a self-review of my code
  • [x ] My changes generate no new warnings or errors
  • I have updated documentation if required
  • [x ] I have linked the relevant issue

Screenshots / Screen Recording

Screenshot 2026-06-05 164003

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 5, 2026

Thank you for submitting a pull request.

Please ensure your changes comply with the project's contribution guidelines and that all workflow checks pass successfully.

A maintainer will review your submission shortly.

Copy link
Copy Markdown
Collaborator

@jagdish-15 jagdish-15 left a comment

Choose a reason for hiding this comment

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

I had originally intended this feature to remain primarily mobile-focused, as I had mentioned in the issue as well, since pagination already keeps the desktop page length relatively manageable.

Right now the scroll-to-top button is also visible on larger desktop layouts, which is not really needed.

Could you please update the implementation so the button is limited to smaller/mobile screen sizes?

@Diya-od
Copy link
Copy Markdown
Contributor Author

Diya-od commented Jun 7, 2026

Hello @jagdish-15 ,
Thanks for the feedback. I've updated the implementation so the scroll-to-top button is only shown on smaller/mobile screen sizes and remains hidden on desktop layouts, where pagination already keeps the page length manageable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Scroll-to-Top Arrow Button in Leaderboard Section

2 participants