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
@github-actions
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