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

#fixed: issue No. 458 #470

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

AnkitLuhar
Copy link

###Description

A clear and concise description of what the PR does.

This PR does the following:

  1. Adds a scroll-to-top button in the footer component to enhance user interaction.

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

Copy link

vercel bot commented Jul 7, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
spring-music-player ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 10, 2024 6:43am

Copy link

vercel bot commented Jul 7, 2024

@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.

Copy link

@github-actions github-actions bot left a 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

@Satyam1923
Copy link
Owner

fine just make it look good,

@AnkitLuhar
Copy link
Author

@Satyam1923 ohk

@AnkitLuhar
Copy link
Author

@Satyam1923 changes are made in the scroll-to-top button and also gives a little bit animation,
Attaching the file go through it....
screen-capture (5).webm

@Satyam1923
Copy link
Owner

@AnkitLuhar change that arrow logo use something good and why yellow color? use some design which matches with ui.

@AnkitLuhar
Copy link
Author

@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...
Screenshot (867)

@Satyam1923
Copy link
Owner

Reduce size a little bit rest looks fine.

@AnkitLuhar
Copy link
Author

@Satyam1923 done,

@AnkitLuhar AnkitLuhar closed this Jul 10, 2024
@AnkitLuhar AnkitLuhar reopened this Jul 10, 2024
@AnkitLuhar
Copy link
Author

@Satyam1923 kindly merge the PR

@AnkitLuhar
Copy link
Author

@saurabhsingh720 please merge this PR

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

Successfully merging this pull request may close these issues.

Feat:Adding a scroll-to-top button
2 participants