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

Website: Fixed anchortags #8003

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

CronKz
Copy link
Contributor

@CronKz CronKz commented Feb 7, 2025

  • Fixed visibilty of anchors by removing display: none and adding overflow: visible

Additional fixes

  • Fixed anchor color variable (defaulted to black before)
- fill="var(--heading-text-color)"
+ fill="var(--cc-heading-text-color)"
  • Added scroll margins considering "sticky" headers to prevent overlapping
  • Adjusted the responsive article menu position. It was not blurring the edges. (Visible if a full-width "warning" box was below)

Closes #7908

@CronKz CronKz changed the title Website: Fix anchortags Website: Fixed anchortags Feb 8, 2025
@michaelstaib michaelstaib requested a review from rstaib February 17, 2025 23:05
@rstaib
Copy link
Member

rstaib commented Feb 18, 2025

@CronKz First of all thanks for your contribution. I will go through it and test it probably today.

@CronKz
Copy link
Contributor Author

CronKz commented Feb 19, 2025

@rstaib Thanks, take any time you need. Let me know if there is anything I can help you with.

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.

Website Anchors are broken
2 participants