Fix accessibility on new registration form #1288

Stef-Rousset · 2025-08-21T13:07:31Z

🎩 What? Why?

This PR improves accessibility on new registration form, by:

adding an indication of expected format for email with sr-only span
indicating that terms of service is required with sr-only span, and by linking the term of services text with its input with aria-describedby

This PR is issued from the audit of Angers city (pages 97 and 98), and refers to criterias 3.3.1 and 3.3.2 from WCAG.

📌 Related Issues

Related to https://github.com/orgs/OpenSourcePolitics/projects/26/views/1?pane=issue&itemId=122486011&issue=OpenSourcePolitics%7Cintern-tasks%7C89
https://github.com/orgs/OpenSourcePolitics/projects/26/views/1?pane=issue&itemId=122486764&issue=OpenSourcePolitics%7Cintern-tasks%7C90

Testing

As a non logged in user, go to the signup page
Activate your screenreader and navigate to the email field
Ensure that an example of format for email is read
Navigate to the terms of service field
Ensure that the indication of required field is read
When you navigate on the checkbox, ensure that the terms of service text is read

📷 Screenshots

♥️ Thank you!

…ssistance_indications

* New translations en.yml (Swedish) * New translations en.yml (Japanese) * New translations en.yml (Japanese) * New translations en.yml (Japanese) * New translations en.yml (Japanese) * New translations en.yml (Japanese) * New translations en.yml (Japanese) * New translations en.yml (Japanese) * New translations en.yml (Japanese) * New translations en.yml (Japanese) * New translations en.yml (Japanese) * New translations en.yml (Japanese) * New translations en.yml (Japanese) * New translations en.yml (Japanese) * New translations en.yml (Basque) * New translations en.yml (German) * New translations en.yml (Basque) * New translations en.yml (French) * New translations en.yml (Spanish) * New translations en.yml (Catalan) * New translations en.yml (Swedish) * New translations en.yml (Turkish) * New translations en.yml (Turkish) * New translations en.yml (Spanish, Mexico) * New translations en.yml (French, Canada) * New translations en.yml (Spanish, Paraguay) * New translations en.yml (Catalan, Algherese) * New translations en.yml (German) * New translations en.yml (Finnish) * New translations en.yml (Finnish) * New translations en.yml (Finnish) * New translations en.yml (German) * New translations en.yml (Basque) * New translations en.yml (Basque) * New translations en.yml (Basque) * New translations en.yml (Finnish) * New translations en.yml (Finnish) * New translations en.yml (Finnish) * New translations en.yml (Finnish) * New translations en.yml (Turkish) * New translations en.yml (Finnish (plain)) * New translations en.yml (Czech) * New translations en.yml (Czech) * New translations en.yml (Czech) * New translations en.yml (Czech) * New translations en.yml (Czech) * New translations en.yml (Basque) * New translations en.yml (Italian) * New translations en.yml (Finnish (plain)) * New translations en.yml (French) * New translations en.yml (Czech) * New translations en.yml (Czech) * New translations en.yml (French, Canada) * New translations en.yml (Finnish (plain)) * New translations en.yml (Finnish (plain)) * New translations en.yml (Finnish (plain)) * New translations en.yml (Finnish (plain)) * New translations en.yml (Finnish (plain)) * New translations en.yml (Spanish) * New translations en.yml (Catalan) * New translations en.yml (Basque) * New translations en.yml (Swedish) * New translations en.yml (Turkish) * New translations en.yml (Turkish) * New translations en.yml (Spanish, Mexico) * New translations en.yml (Spanish, Paraguay) * New translations en.yml (Catalan, Algherese) * New translations en.yml (Basque) * New translations en.yml (Korean) * New translations en.yml (Vietnamese) * New translations en.yml (Persian) * New translations en.yml (Maltese)

* Error handling of not found resources * Fix Page specs * Add I18n support for not found error * Fix i18n key * Fix locales * Fix failing specs * Add error code token * Rename context from graphQL hidden space to graphQL not found space * Address other review comments

* Refactor API location * Add Proposal answer mutation example * Add link to decidim docs website * Process page announcement bug fix with system test (decidim#15616) Co-authored-by: Leo <leo.storey@mainiotech.fi> * Add debug permissions * Fix exception * Error handling of not found resources * Fix Page specs * Add I18n support for not found error * Fix i18n key * Fix locales * Fix failing specs * Adjust some of the existing mutations * Add error code token * Rename context from graphQL hidden space to graphQL not found space * Address other review comments * Fix typo * Fix some types * Fix failing specs * Fix failing specs * Apply suggestions from code review Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com> * Fix NuValidator image * Apply review recommendations * Update decidim-api/lib/decidim/api/test/component_context.rb Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com> --------- Co-authored-by: Leo Storey <123873192+Ginger-Leo@users.noreply.github.com> Co-authored-by: Leo <leo.storey@mainiotech.fi> Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

…5768) * Refactor QueryType to API * Remove bullet error * Remove unneded autoload * Fix typo * Disable bullet * Fix bullet error

* Extract each one spec in own test * Fix budgets specs * Add propsoals specs * Fix failing specs * Update decidim-proposals/spec/system/proposals_breadcrumbs_spec.rb Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Apply suggestions from code review Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update decidim-accountability/spec/system/explore_versions_spec.rb Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Extract results * Extract surveys specs * Extract surveys specs part 2 * Fully extract proposals * Fully extract proposals * Fix typo * Extract meeetings * Extract elections * Extract debates * Extract collaborative texts * Extract blogs * Create uniform specs * Fix filename * Apply review recommendations * Standardize tests * Fix specs --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Co-authored-by: Alexandru Emil Lupu <contact@alecslupu.ro>

* Add AWS Bucket configuration for public assets * Fix typo * Apply suggestions from code review Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Apply suggestions from code review Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update docs/modules/services/pages/activestorage.adoc * Apply suggestions from code review Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com> * Update docs/modules/services/pages/activestorage.adoc Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com> --------- Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Co-authored-by: Alexandru Emil Lupu <contact@alecslupu.ro> Co-authored-by: Leo <leo.storey@mainiotech.fi>

…decidim#15787) Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* condition to display responses in dropdown * i18n transalation for responses * Update decidim-surveys/app/views/decidim/surveys/admin/surveys/index.html.erb Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com> * Update decidim-surveys/config/locales/en.yml Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com> * Update decidim-surveys/config/locales/en.yml Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com> * updated the icon for responses tab * route for admin added in spec, specs for responses and no responses added * fixed translation conflict from suggestion --------- Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

Co-authored-by: Leo <leo.storey@mainiotech.fi>

* Refactor API documentation * Fix typos * Apply suggestions from code review Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Apply suggestions from code review * Apply suggestions from code review Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Apply suggestions from code review * Apply suggestions from code review * Apply suggestions from code review Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com> * Apply suggestions from code review Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com> * Apply more review recommendations * Apply more review recommendations - error pages * Apply suggestions from code review Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com> --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* Move FormFactory to be accessible from API * Add missing method * Apply review recommendation

* Refactor API to raise and catch Decidim::PermissionAction::PermissionNotSetError exceptions * Add Decidim::Api::Errors::MutationNotAuthorizedError * Add Decidim::Api::Errors::ValidationError * Add Decidim::Api::Errors::AttributeValidationError * Apply suggestions from code review Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Fix spec * Add test example * Refactor file name * Apply suggestions from code review Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Apply review recommendation * Fix spec * Apply suggestions from code review Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * change the variable name * Add additional I18n related exceptions * Refactor error layer * Fix error class tokens * Fix the initiatives specs * Apply suggestions from code review Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Apply suggestions from code review Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Running linters * Apply suggestions from code review Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com> --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

…dim#15822)

* Refactor API to raise and catch Decidim::PermissionAction::PermissionNotSetError exceptions * Add Decidim::Api::Errors::MutationNotAuthorizedError * Add Decidim::Api::Errors::ValidationError * Add Decidim::Api::Errors::AttributeValidationError * Apply suggestions from code review Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Fix spec * Add test example * Refactor file name * Apply suggestions from code review Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Apply review recommendation * Fix spec * Add mutation to vote Proposals via API * Apply suggestions from code review Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * change the variable name * Add additional I18n related exceptions * Refactor error layer * Fix error class tokens * Fix the initiatives specs * Apply suggestions from code review Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Apply suggestions from code review Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Running linters * Refactor API documentation * Fix typos * Apply suggestions from code review Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Apply suggestions from code review * Apply suggestions from code review Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Apply suggestions from code review * Apply suggestions from code review * Refactor documentation * Apply review recommendations * Apply suggestions from code review Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com> * Removed comment --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

github-actions bot added type: fix module: core labels Aug 21, 2025

github-actions bot approved these changes Aug 21, 2025

View reviewed changes

Stef-Rousset force-pushed the fix/input_assistance_indications branch from 00ef22d to 84be711 Compare August 22, 2025 07:08

Stef-Rousset force-pushed the fix/input_assistance_indications branch from 77a15c8 to af26a64 Compare September 1, 2025 10:09

Stef-Rousset added 3 commits September 8, 2025 17:20

feat: add sr-only with email example in registration new

75a4e81

feat: improve accessibility on terms of service in registration

3d3f1d7

Trigger CI

4a75eea

Stef-Rousset force-pushed the fix/input_assistance_indications branch from af26a64 to 4a75eea Compare September 8, 2025 15:21

test: fix failing proposal system test

2717d0e

github-actions bot added the module: proposals label Sep 9, 2025

Merge branch 'develop' of github.com:decidim/decidim into fix/input_a…

c852f97

…ssistance_indications

github-actions bot added module: collaborative_texts module: demographics labels Nov 28, 2025

decidim-bot and others added 28 commits December 9, 2025 17:30

Fix NuValidator image (decidim#15773)

a0841c2

Change Vote button to Edit vote when user has already voted (decidim#…

f72c523

…15761)

Add total votes count to election results in admin and public views (d…

4ef4a70

…ecidim#15763)

Allow editing last question on receipt page for per-question elections (

15dbb71

decidim#15742)

Prevent server error on proposal page when the user is not logged in (d…

00b96d3

…ecidim#15776)

Refactor QueryExtensions to move the implementation to API (decidim#1…

085ede4

…5768) * Refactor QueryType to API * Remove bullet error * Remove unneded autoload * Fix typo * Disable bullet * Fix bullet error

Rename private participants to members (decidim#15749)

1a80b16

Add short_name field to Organizations for PWA (decidim#15729)

370ebe4

Co-authored-by: Alexandru Emil Lupu <contact@alecslupu.ro>

Extract documentation from /api to docs.decidim.org (decidim#15804)

0623de4

Fix search result comment with link (decidim#15774)

c4aa90c

Co-authored-by: Alexandru Emil Lupu <contact@alecslupu.ro> Co-authored-by: Leo <leo.storey@mainiotech.fi>

Improve handling of PermissionNotSetError exceptions in GraphQL API (…

4c1c6a0

…decidim#15787) Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Fix process and assembly admin members action logs (decidim#15788)

94b7039

Sorting via created_at attribute within spaces (decidim#15810)

9b079c2

Fix focus trap in modal dialog for sharing (decidim#15807)

243d573

Co-authored-by: Leo <leo.storey@mainiotech.fi>

Move FormFactory to be accessible from API (decidim#15819)

b72051b

* Move FormFactory to be accessible from API * Add missing method * Apply review recommendation

Update docs for GitHub Actions configuration (decidim#15824)

8361425

Enable EnforcedShorthandSyntax: always on rubocop configuration (deci…

e2e272b

…dim#15822)

Fix details in ProposalAnswerType spec file (decidim#15826)

0f9074e

Merge branch 'develop' into fix/input_assistance_indications

97f1461

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Fix accessibility on new registration form #1288

Fix accessibility on new registration form #1288

Uh oh!

Stef-Rousset commented Aug 21, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

9 participants

Fix accessibility on new registration form #1288

Are you sure you want to change the base?

Fix accessibility on new registration form #1288

Uh oh!

Conversation

Stef-Rousset commented Aug 21, 2025

🎩 What? Why?

📌 Related Issues

Testing

📷 Screenshots

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

9 participants