Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feat] :Adding a move to top icon #608

Closed
wants to merge 0 commits into from

Conversation

rishabdev2997
Copy link

[Feat] : Adding a move to top icon

#597 - Added the feature.

Feature details-

This feature reduces the time and effort needed to manually scroll up, making navigation smoother.

How to see the change

  • Scroll Button Location: The "Move to Top" button is positioned at the bottom-right corner of the homepage.
  • Visibility Condition: The button becomes visible only when the user has scrolled down from the top of the page, enhancing usability by not cluttering the view initially.
  • Functionality: Clicking on the button will instantly scroll the user back to the top of the page.

This contributes to making the site more intuitive for users, allowing them to quickly navigate long pages, especially helpful on the homepage with multiple sections.

Here is the screenshot

go.to.top.with.pointer.2.mp4

@rishabdev2997
Copy link
Author

Hi @limzykenneth,
I’ve added the "scroll to top" feature. When you have a moment, could you please review it and let me know if any changes or improvements are needed? If everything looks okay, it would be great if you could merge it.
Thank you so much!

@rishabdev2997
Copy link
Author

Hi @limzykenneth,

I hope you're doing well! I just wanted to check in regarding the "scroll to top" feature I submitted recently. When you have a moment, could you kindly review it and let me know if any adjustments are needed? If everything looks good, it would be great if you could merge it.

Thank you so much for your time and consideration!

@limzykenneth
Copy link
Member

I'm not sure if this is the right way to implement this with inline style and scripts. There really should be some discussions in the issue to determine the implementation first.

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.

2 participants