这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@llovvoll
Copy link
Contributor

@llovvoll llovvoll commented Jan 4, 2022

The current version use "opacity" to control the display state of the button, but when it is hidden, the mouse can still click on the button in the lower right corner

@vercel
Copy link

vercel bot commented Jan 4, 2022

Someone is attempting to deploy a commit to a Personal Account owned by @timlrx on Vercel.

@timlrx first needs to authorize it.

@vercel
Copy link

vercel bot commented Jan 4, 2022

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/timlrx/tailwind-nextjs-starter-blog/CmxCBWD4aYKvzQ3CK3AfQ5wxKChD
✅ Preview: https://tailwind-nextjs-starter-blog-git-fork-llovvoll-fi-2b9018-timlrx.vercel.app

@timlrx
Copy link
Owner

timlrx commented Jan 4, 2022

Thanks, I think we can use display rather than visibility and use the tailwind classes rather than style?

@11006281 11006281 force-pushed the fix/scroll-top-button branch from 4bc98c5 to 759f33d Compare January 4, 2022 02:10
@llovvoll
Copy link
Contributor Author

llovvoll commented Jan 4, 2022

Thanks, I think we can use display rather than visibility and use the tailwind classes rather than style?

ok~

@timlrx timlrx merged commit 5e617c6 into timlrx:master Jan 4, 2022
@llovvoll llovvoll changed the title fix: should use visibility to control display state of scroll top button fix: should use display to control display state of scroll top button Jan 4, 2022
@11006281 11006281 deleted the fix/scroll-top-button branch January 13, 2022 09:54
Meez25 pushed a commit to Meez25/Blog that referenced this pull request Jun 17, 2024
fix: should use visibility to control display state of scroll top button
bhiwagade-rahul pushed a commit to bhiwagade-rahul/tailwind-nextjs-starter-blog that referenced this pull request Sep 22, 2025
fix: should use visibility to control display state of scroll top button
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