-
Notifications
You must be signed in to change notification settings - Fork 32
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:added scroll to top button #119
base: main
Are you sure you want to change the base?
Conversation
@himanshu1221 is attempting to deploy a commit to the git-opener-team 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.
Congratulations @himanshu1221 on creating your very first pull request in GitOpener! 🎉 Your hard work and dedication to this project is truly appreciated. Thank you for your valuable contribution and for helping us move one step closer to achieving our goals. Keep up the awesome work!
As we r using heroicons you can use up chevron. |
We are using tailwind css as our style library. Your can use tailwind css |
I am more comfortable with vanilla css If you still want me to remove css and try to make it work it tailwind i can give it a try |
4eb7d1b
to
f6620ce
Compare
add Tailwind CSS It would be better 🤟🏻 |
Ill work on it super soon i am bit occupied because of my exams soon gonna make a PR |
Yeah take your time |
Hey, @sboy99 I have added Tailwind CSS and removed vanilla CSS it would be great if you review this PR and suggest any further changes! |
@sboy99 please review it. |
I requested some changes that wasn't updated. So we can't proceed further |
Hello @sboy99, is this issue still active? |
Fixes Issue
closes #99
Changes proposed
Created a new file which enables the website to scroll to top
Screenshots