Skip to content

♿️(frontend) fix waffle aria-label spacing for new-window links#2030

Merged
Ovgodd merged 1 commit intomainfrom
fix/waffle-new-window-key
Mar 19, 2026
Merged

♿️(frontend) fix waffle aria-label spacing for new-window links#2030
Ovgodd merged 1 commit intomainfrom
fix/waffle-new-window-key

Conversation

@Ovgodd
Copy link
Collaborator

@Ovgodd Ovgodd commented Mar 17, 2026

Purpose

Fix the aria-label for waffle links that open in a new window. Screen readers previously announced "Visionouvelle fenêtre" instead of "Visio (nouvelle fenêtre)".

Proposal

  • Include the space and parentheses in the key

@Ovgodd Ovgodd requested a review from AntoLC March 17, 2026 09:19
@Ovgodd Ovgodd self-assigned this Mar 17, 2026
@Ovgodd Ovgodd moved this from Backlog to In progress in LaSuite Docs A11y Mar 17, 2026
@Ovgodd Ovgodd linked an issue Mar 17, 2026 that may be closed by this pull request
@Ovgodd Ovgodd changed the title ♿️(i18n) fix waffle aria-label spacing for new-window links 🌐(i18n) fix waffle aria-label spacing for new-window links Mar 17, 2026
@Ovgodd Ovgodd force-pushed the fix/waffle-new-window-key branch from caf4b0b to 35abbf4 Compare March 17, 2026 09:20
@Ovgodd Ovgodd marked this pull request as ready for review March 17, 2026 09:21
@github-actions
Copy link

github-actions bot commented Mar 17, 2026

Size Change: +14 B (0%)

Total Size: 4.23 MB

Filename Size Change
apps/impress/out/_next/static/3fbaad80/_buildManifest.js 903 B +903 B (new file) 🆕
apps/impress/out/_next/static/eb845de6/_buildManifest.js 0 B -904 B (removed) 🏆

compressed-size-action

@Ovgodd Ovgodd moved this from In progress to In review in LaSuite Docs A11y Mar 17, 2026
CHANGELOG.md Outdated
- ⬆️(frontend) upgrade Next.js to v16 #1980
- ♿️(frontend) fix aria-label and landmark on document banner state #1986
- 🌐(i18n) add "new window" translation key for waffle aria-label #1984
- 🌐(i18n) fix waffle aria-label spacing for new-window links #2030
Copy link
Collaborator

Choose a reason for hiding this comment

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

Under "Unreleased"

@Ovgodd Ovgodd force-pushed the fix/waffle-new-window-key branch 7 times, most recently from bca1037 to e4bf348 Compare March 18, 2026 14:32
@Ovgodd Ovgodd changed the title 🌐(i18n) fix waffle aria-label spacing for new-window links ♿️(frontend) fix waffle aria-label spacing for new-window links Mar 18, 2026
@Ovgodd Ovgodd force-pushed the fix/waffle-new-window-key branch 2 times, most recently from 2a9bf83 to 51ee114 Compare March 18, 2026 15:35
Include space and parentheses in translation value for proper aria-label.
@Ovgodd Ovgodd force-pushed the fix/waffle-new-window-key branch from 51ee114 to 35fba02 Compare March 19, 2026 08:15
@Ovgodd Ovgodd merged commit 35fba02 into main Mar 19, 2026
26 of 27 checks passed
@Ovgodd Ovgodd deleted the fix/waffle-new-window-key branch March 19, 2026 09:12
@github-project-automation github-project-automation bot moved this from In review to Done in LaSuite Docs A11y Mar 19, 2026
lunika added a commit that referenced this pull request Mar 19, 2026
Changed

- ♿️(frontend) ensure doc title is h1 for accessibility #2006
- ♿️(frontend) add nb accesses in share button aria-label #2017

Fixed

- 🐛(frontend) fix image resizing when caption #2045
- 🙈(docker) add \*\*/.next to .dockerignore #2034
- ♿️(frontend) fix share modal heading hierarchy #2007
- ♿️(frontend) fix Copy link toast accessibility for screen readers #2029
- ♿️(frontend) fix modal aria-label and name #2014
- ♿️(frontend) fix language dropdown ARIA for screen readers #2020
- ♿️(frontend) fix waffle aria-label spacing for new-window links #2030
- 🐛(backend) stop using add_sibling method to create sandbox document #2084
- 🐛(backend) duplicate a document as last-sibling #2084
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Wrong translation key in aria-label="Visionouvelle fenêtre"

2 participants