Skip to content

Conversation

Ovgodd
Copy link
Collaborator

@Ovgodd Ovgodd commented Sep 24, 2025

Purpose

This PR improves the semantic structure and accessibility of the left panel ( TargetFilters and Favorites too) .

leftpaneltonav

issue : 1429

Proposal

  • Wrap main navigation areas in nav with appropriate aria-labels
  • Replace nav with section for grouped pinned content
  • Use aria-labelledby and element id for better labeling

@Ovgodd Ovgodd requested a review from AntoLC September 24, 2025 08:01
@Ovgodd Ovgodd self-assigned this Sep 24, 2025
@Ovgodd Ovgodd force-pushed the fix/1429-left-panel-semantic branch from a15d892 to ee12135 Compare September 24, 2025 08:02
@Ovgodd Ovgodd marked this pull request as ready for review September 24, 2025 08:02
Copy link

github-actions bot commented Sep 24, 2025

Size Change: +16 B (0%)

Total Size: 3.65 MB

Filename Size Change
apps/impress/out/_next/static/ac0a45b3/_buildManifest.js 0 B -866 B (removed) 🏆
apps/impress/out/_next/static/08be57c0/_buildManifest.js 865 B +865 B (new file) 🆕

compressed-size-action

Copy link
Collaborator

@AntoLC AntoLC left a comment

Choose a reason for hiding this comment

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

Seems good ✅

@Ovgodd Ovgodd force-pushed the fix/1429-left-panel-semantic branch from ee12135 to f382533 Compare September 24, 2025 10:14
use nav and appropriate aria attributes to enhance accessibility

Signed-off-by: Cyril <c.gromoff@gmail.com>
@Ovgodd Ovgodd force-pushed the fix/1429-left-panel-semantic branch from f382533 to 8ab21ef Compare September 24, 2025 10:19
@Ovgodd Ovgodd merged commit 8ab21ef into main Sep 24, 2025
21 of 22 checks passed
@Ovgodd Ovgodd deleted the fix/1429-left-panel-semantic branch September 24, 2025 11:18
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.

2 participants