processing / p5.js-website

New p5.js website!
http://p5js.org
MIT License
18 stars 90 forks source link

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

Closed rishabdev2997 closed 2 weeks ago

rishabdev2997 commented 1 month ago

[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

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

https://github.com/user-attachments/assets/708d5c3e-a335-4aa4-8232-ed384cf866a0

rishabdev2997 commented 1 month ago

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 commented 1 month ago

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 commented 1 month ago

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.