Skip to content

♿️(frontend) add nb accesses in share button aria-label#2017

Merged
Ovgodd merged 1 commit intomainfrom
fix/share-button-aria-label-nb-accesses
Mar 18, 2026
Merged

♿️(frontend) add nb accesses in share button aria-label#2017
Ovgodd merged 1 commit intomainfrom
fix/share-button-aria-label-nb-accesses

Conversation

@Ovgodd
Copy link
Collaborator

@Ovgodd Ovgodd commented Mar 16, 2026

Purpose

The share button shows the number of users with access, but screen readers don't announce it.

Proposal

  • Add nb_accesses_direct to the share button aria-label when the count is visible

@Ovgodd Ovgodd requested a review from AntoLC March 16, 2026 15:06
@Ovgodd Ovgodd self-assigned this Mar 16, 2026
@Ovgodd Ovgodd force-pushed the fix/share-button-aria-label-nb-accesses branch from 10657c3 to 362ff07 Compare March 16, 2026 15:07
@Ovgodd Ovgodd marked this pull request as ready for review March 16, 2026 15:07
@Ovgodd Ovgodd moved this from Backlog to In review in LaSuite Docs A11y Mar 16, 2026
@Ovgodd Ovgodd linked an issue Mar 16, 2026 that may be closed by this pull request
@github-actions
Copy link

github-actions bot commented Mar 16, 2026

Size Change: +24 B (0%)

Total Size: 4.23 MB

Filename Size Change
apps/impress/out/_next/static/1d215153/_buildManifest.js 904 B +904 B (new file) 🆕
apps/impress/out/_next/static/063e302d/_buildManifest.js 0 B -905 B (removed) 🏆

compressed-size-action

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
- ♿️(frontend) add nb accesses in share button aria-label #2017
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/share-button-aria-label-nb-accesses branch 3 times, most recently from 3579660 to 23a455d Compare March 18, 2026 12:38
Expose nb_accesses_direct to screen readers when share button shows count.
@Ovgodd Ovgodd force-pushed the fix/share-button-aria-label-nb-accesses branch from 23a455d to 1172fbe Compare March 18, 2026 13:21
@Ovgodd Ovgodd merged commit 1172fbe into main Mar 18, 2026
26 of 27 checks passed
@Ovgodd Ovgodd deleted the fix/share-button-aria-label-nb-accesses branch March 18, 2026 14:28
@github-project-automation github-project-automation bot moved this from In review to Done in LaSuite Docs A11y Mar 18, 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.

Number of people who have access to sharing this document

2 participants