-
Notifications
You must be signed in to change notification settings - Fork 126
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
#fixed: issue No. 458 #470
#fixed: issue No. 458 #470
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@AnkitLuhar is attempting to deploy a commit to the satyam1923's projects Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job, @AnkitLuhar! 🎉 Thank you for submitting your first pull request. Your contribution is valuable and we appreciate your efforts to improve our project.
We'd also like to invite you to our Discord server! Join us here.
We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines
…' into feature/add-scroll-to-top-button imp
fixed issue no. 458
fine just make it look good, |
@Satyam1923 ohk |
@Satyam1923 changes are made in the scroll-to-top button and also gives a little bit animation, |
@AnkitLuhar change that arrow logo use something good and why yellow color? use some design which matches with ui. |
@Satyam1923 attaching the screenshot you can check it out then if it is okay then i will commit the changes,if the logo or the color combination you don't like it just provide me the color combination then i will do the changes according to your preferences... |
Reduce size a little bit rest looks fine. |
@Satyam1923 done, |
@Satyam1923 kindly merge the PR |
@saurabhsingh720 please merge this PR |
@Satyam1923 kindly merge the PR! |
This PR has been automatically closed due to inactivity from the owner for 15 days. |
###Description
A clear and concise description of what the PR does.
This PR does the following:
2.Fixes the visibility issue by making the button only appear when the user scrolls down the page.
3.Updates the footer component to include the new feature.
###Related Issues:
Fixes #458
###Changes:
List the detailed changes made in this PR.
1.Added a new feature to show a scroll-to-top button.
2.Refactored the footer component to include the new button.
3.Implemented the functionality to show/hide the button based on the scroll position.
###Testing Instructions:
Detailed instructions on how to test the changes. Include any setup needed and specific test cases.
1.Pull this branch.
2.Run npm install to install dependencies.
3.Run npm start to launch the application.
4.setup the frontend and backend according to it.
5.Verify that the scroll-to-top button appears when scrolling down and hides when scrolling up.
6.Click the button to ensure it scrolls the page to the top smoothly.
###Screenshots (if applicable)
screen-capture (4).webm
###Additional Context
This PR is based on the need to improve user interaction by providing an easy way to navigate to the top of the page