-
-
Notifications
You must be signed in to change notification settings - Fork 219
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
Resolves #687 Add skip to content button #1235
Conversation
|
built with Refined Cloudflare Pages Action⚡ Cloudflare Pages Deployment
|
9d2efe0
to
c54d68f
Compare
Hey, thank you for the PR! It seems the button isn't visible when focusing it. I can see in the console that it is focused, but nonetheless there is no visual indication. At least in the cloudflare preview! |
I noticed that, too, I had a look at the preview. It needed a higher z-index, I already pushed that change |
Hmm, the new preview still fails. I don't think z-1000 is a valid tailwind class, you may need to do |
Strange, it works on my machine 🤔 |
Awesome, it works now! Thank you so much for the contribution, and for putting up with the back and forth 😄 |
No problem, happy to help! |
Skip to content button added. It does not work so well on the first page - it is the first item in the navigation order, but the focus is set to the popover button. Let me know what you think, and if there's something in the code that needs updating.
Resolves issue #687