Skip to content

Conversation

Ovgodd
Copy link
Collaborator

@Ovgodd Ovgodd commented Sep 22, 2025

Purpose

Remove a redundant aria-label avoiding over-accessibility and prevent screen readers from duplicating content already present in a visible span

issue : 1401

image

Proposal

  • Remove unnecessary aria-label from the element

@Ovgodd Ovgodd requested a review from AntoLC September 22, 2025 12:24
@Ovgodd Ovgodd self-assigned this Sep 22, 2025
@Ovgodd Ovgodd force-pushed the fix/1401-avoid-over-a11y branch from 8c74049 to 07455ce Compare September 22, 2025 12:28
@Ovgodd Ovgodd marked this pull request as ready for review September 22, 2025 12:28
Copy link

github-actions bot commented Sep 22, 2025

Size Change: +6 B (0%)

Total Size: 3.65 MB

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

compressed-size-action

@Ovgodd Ovgodd force-pushed the fix/1401-avoid-over-a11y branch from 07455ce to 72a6f9b Compare September 22, 2025 12:31
@Ovgodd Ovgodd force-pushed the fix/1401-avoid-over-a11y branch 4 times, most recently from cbf28d1 to 0505619 Compare September 23, 2025 06:51
aria-label was removed because the visible span already provides the text

Signed-off-by: Cyril <c.gromoff@gmail.com>
@Ovgodd Ovgodd force-pushed the fix/1401-avoid-over-a11y branch from 0505619 to 3607faa Compare September 23, 2025 09:26
@Ovgodd Ovgodd merged commit 3607faa into main Sep 23, 2025
21 of 22 checks passed
@Ovgodd Ovgodd deleted the fix/1401-avoid-over-a11y branch September 23, 2025 10:06
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