Skip to content

fix(site): underline links in running text - #260

Merged
accesswatch merged 1 commit into
mainfrom
fix/underline-content-links
Oct 28, 2026
Merged

accesswatch merged 1 commit into
mainfrom
fix/underline-content-links

Conversation

@accesswatch

Copy link
Copy Markdown
Collaborator

github-markdown-css underlines links only on hover. That left links inside paragraphs, lists, tables and quotes distinguished by color alone (WCAG 1.4.1, axe link-in-text-block). The site audit reported it as Community-Access/community-access.github.io#6. This adds the underline in scripts/build-html.js and rebuilds the HTML. Navigation, breadcrumbs and buttons keep their own styles.


Generated by Claude Code

github-markdown-css underlines links only on hover, so links inside
paragraphs, lists, tables and quotes were distinguished by color alone
(WCAG 1.4.1). Reported by the site audit as
Community-Access/community-access.github.io#6.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PmRBrUTuhzYnQBFxUCN6Vr
@accesswatch
accesswatch merged commit d052c83 into main Oct 28, 2026
3 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

Peer Review Assigned

Hi @accesswatch! Your PR has been automatically paired with @taylorarndt for peer review.

For @taylorarndt:

This is a great opportunity to practice code review skills! Here's what to look for:

Content Quality:

  • Does the change accomplish what the issue describes?
  • Is the writing clear and helpful?
  • Are there any typos or grammar issues?

Accessibility:

  • Proper heading hierarchy (H1 → H2 → H3, no skips)?
  • Descriptive link text (not "click here")?
  • Alt text on images?
  • [TODO] markers removed?

Documentation:

  • Code blocks are properly formatted?
  • Tables have headers?
  • References/links work correctly?

Review Guidelines:

  • Be kind and constructive
  • Suggest improvements, don't just point out problems
  • Ask questions if something is unclear
  • Approve when ready or request changes with explanation

Resources:


Pairing by Learning Room Grouping Engine

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