Skip to content

Conversation

@Ovgodd
Copy link
Collaborator

@Ovgodd Ovgodd commented Oct 29, 2025

Purpose

Improve accessibility and styling of the Table of Contents component.

summaryheadings.mp4

issue : 1529

Proposal

  • Add semantic structure (nav, ul, li)
  • Add ARIA attributes for screen reader support
  • Replace hardcoded styles with Cunningham tokens
  • Apply consistent custom focus styling
  • Enable full keyboard navigation: Tab to navigate, Enter to focus and edit a heading (same behavior as mouse click)

@Ovgodd Ovgodd requested a review from AntoLC October 29, 2025 09:54
@Ovgodd Ovgodd self-assigned this Oct 29, 2025
@Ovgodd Ovgodd marked this pull request as ready for review October 29, 2025 09:55
@github-actions
Copy link

github-actions bot commented Oct 29, 2025

Size Change: +208 B (+0.01%)

Total Size: 3.68 MB

Filename Size Change
apps/impress/out/_next/static/06b47090/_buildManifest.js 0 B -884 B (removed) 🏆
apps/impress/out/_next/static/e8430d1c/_buildManifest.js 884 B +884 B (new file) 🆕

compressed-size-action

@Ovgodd Ovgodd force-pushed the fix/a11y-summary branch 2 times, most recently from 7f5f6fd to d8d1be8 Compare November 10, 2025 11:23
@Ovgodd Ovgodd requested a review from AntoLC November 10, 2025 11:23
CHANGELOG.md Outdated
- 🐛(backend) fix trashbin list
- ♿(frontend) improve accessibility:
- ♿(frontend) remove empty alt on logo due to Axe a11y error #1516
- ♿(frontend) improve accessibility and styling of summary table #1528
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We did a release, when you will rebase with main, think of moving this part under the Unrelease part.

adds semantic structure, aria attributes, and token-based focus styling

Signed-off-by: Cyril <c.gromoff@gmail.com>
@Ovgodd Ovgodd merged commit 3d45c7c into main Nov 12, 2025
21 of 22 checks passed
@Ovgodd Ovgodd deleted the fix/a11y-summary branch November 12, 2025 09:41
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.

3 participants