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

Added back to top button #54

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

Conversation

shreyad2806
Copy link

#22
Created a "Back to Top" button that is fixed to the bottom right corner of the page. When clicked, it smoothly scrolls the user back to the top of the page.
Please assign the gssoc and hactoberfest labels.
Also please give level2 tag.
image

Copy link

netlify bot commented Oct 4, 2024

Deploy Preview for asc-hirehub ready!

Name Link
🔨 Latest commit 8faf3e8
🔍 Latest deploy log https://app.netlify.com/sites/asc-hirehub/deploys/67029638fb5f540007435ba5
😎 Deploy Preview https://deploy-preview-54--asc-hirehub.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@shreyad2806
Copy link
Author

#7
Added Dark mode toogle button in commit 2
Please give level3

Copy link
Owner

@ChandelAnish ChandelAnish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

implement dark mode for every elements there are many elements , images not fallowing the dark theme and also the footer text in not readable due to dark color
image

image

image

and also the dark mode toggle is not matching the theme make it some what like this :
image

@shreyad2806
Copy link
Author

Could you please first merge the #22 pr because it is creating confusion for me to update as its showing commit ahead.

@shreyad2806
Copy link
Author

fixed the dark mode

Copy link
Owner

@ChandelAnish ChandelAnish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just place the toggle button in the nav bar no need to make it sticky
image

the text is not readable , you should remove the background of the background image or use different image with same context
image

make the navbar in dark theme too
image

@shreyad2806
Copy link
Author

shreyad2806 commented Oct 6, 2024 via email

@shreyad2806
Copy link
Author

image
image
fixed the dark mode
please assign level3

@shreyad2806
Copy link
Author

Please assign with labels
Please give level2

Copy link
Owner

@ChandelAnish ChandelAnish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you removed the image from here
image

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