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.

Sign up for GitHub

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 bottom

Fix Skip to content #2152

Closed
BorghildSelle opened this issue Feb 23, 2024 · 0 comments
Closed

Fix Skip to content #2152

BorghildSelle opened this issue Feb 23, 2024 · 0 comments
Assignees
@millianapia
Labels
🚀 ready to deploy Use this if issue is ready to be deployed

Comments

@BorghildSelle
Copy link
Contributor

BorghildSelle commented Feb 23, 2024 •
edited
Loading

The skip to content now points to the container div that holds header and main.
The skip to content should go to the h1 in the main
@padms padms self-assigned this Feb 28, 2024
millianapia added a commit that referenced this issue Mar 4, 2024
@millianapia
♿️ Use our own skip to content component #2152
eaed9ef
@millianapia millianapia assigned millianapia and unassigned padms Mar 4, 2024
millianapia added a commit that referenced this issue Mar 12, 2024
@millianapia
🐛 Change so only h1 headers get id #2152
eebf611
millianapia added a commit that referenced this issue Apr 10, 2024
@millianapia
♿️ Use our own skip to content component #2152 (#2168)
eefbb1f 
* ♿️ Use our own skip to content component #2152

* 🐛 Change so only h1 headers get id #2152
@millianapia millianapia added the 🚀 ready to deploy Use this if issue is ready to be deployed label Apr 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@millianapia millianapia

Labels
🚀 ready to deploy Use this if issue is ready to be deployed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@millianapia @padms @BorghildSelle