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

Jump to top button #653

Merged
merged 2 commits into from
Oct 1, 2024
Merged

Jump to top button #653

merged 2 commits into from
Oct 1, 2024

Conversation

edent
Copy link
Contributor

@edent edent commented Oct 1, 2024

Fixes #647

@jmhobbs
Copy link
Owner

jmhobbs commented Oct 1, 2024

That it does! Thanks!

Could you put a style on there to remove the underline?
Looks a little off with the emoji rendering right against the baseline.
Screenshot 2024-10-01 at 10 12 28 AM

@jmhobbs
Copy link
Owner

jmhobbs commented Oct 1, 2024

Looks like I need to update the Go bits 😅
I'll merge through that when it's ready.

@edent
Copy link
Contributor Author

edent commented Oct 1, 2024

I've removed the underline - you have better eyesight than me 😂

@jmhobbs jmhobbs merged commit 47529e7 into jmhobbs:main Oct 1, 2024
7 of 8 checks passed
@jmhobbs
Copy link
Owner

jmhobbs commented Oct 1, 2024

Thanks again!

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.

Adding a Scroll to the Top Button to the website
2 participants