Skip to content

Fix TOC active link spacing - #67

Open
Ashishchoudhary07 wants to merge 1 commit into
OWASP:websitefrom
Ashishchoudhary07:fix/toc-active-link-spacing
Open

Ashishchoudhary07 wants to merge 1 commit into
OWASP:websitefrom
Ashishchoudhary07:fix/toc-active-link-spacing

Conversation

@Ashishchoudhary07

Copy link
Copy Markdown

Summary

  • Add horizontal padding to level 2 Table of Contents links to prevent the active indicator border from touching the heading text.
  • Remove the obsolete commented-out padding rule.
  • Preserve the existing indentation for level 3 links.

Testing

  • git diff --check — passed
  • npx tsc --noEmit — passed
  • npm run lint — passed
  • npx next build --webpack — passed
  • py -3 scripts/validate_website_export.py — passed (47 HTML pages and 2,955 local references validated)

AI Disclosure

  • AI assistance: Yes
  • Tool or model: ChatGPT (GPT-6)
  • Provider: OpenAI
  • Contribution: Assisted with identifying the spacing issue and reviewing the proposed fix.
  • Human validation: Reviewed the code change and ran the listed validation commands.

Fixes #62

@github-actions

Copy link
Copy Markdown
Contributor

✅ No duplicate payload entries detected

Checked 0 normalized entries from added/modified payload files on the latest push.

Posted automatically by the Duplicate Payload Check workflow.

This branch has not been deployed

No deployments
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.

Table of Contents active link indicator accent border collides into text with 0px padding

1 participant