Skip to content

Fix issues with a11y specs#9929

Merged
andreslucena merged 2 commits intodecidim:developfrom
mainio:fix/a11y-specs
Oct 25, 2022
Merged

Fix issues with a11y specs#9929
andreslucena merged 2 commits intodecidim:developfrom
mainio:fix/a11y-specs

Conversation

@ahukkanen
Copy link
Copy Markdown
Contributor

🎩 What? Why?

Recently we have few accessibility tests broken because axe-core was updated:

  1. The axe matchers specs are bound to the previous axe mainline version, after this it will be read dynamically from the package
  2. The new version added stricter checks for link color contrasts which causes actual a11y violations

Testing

See that CI is green.

Copy link
Copy Markdown
Member

@andreslucena andreslucena left a comment

Choose a reason for hiding this comment

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

That was quick, thanks for the PR and the fix!!
👍🏽

@andreslucena andreslucena merged commit 0ee7c76 into decidim:develop Oct 25, 2022
@ahukkanen ahukkanen deleted the fix/a11y-specs branch October 25, 2022 13:31
@ahukkanen ahukkanen added type: internal PRs that aren't necessary to add to the CHANGELOG for implementers and removed type: fix PRs that implement a fix for a bug labels Oct 25, 2022
entantoencuanto added a commit that referenced this pull request Oct 26, 2022
* develop: (35 commits)
  Install turbo-rails (#9881)
  Fix conference invitations (#9664)
  Fix invalid rendering of meeting and proposal body texts (#9764)
  Make documentation site work with multiple versions (#9917)
  Bump versions on install docs (#9916)
  Standardize CSV import formats and fix private users CSV import with invalid file (#9627)
  Fix: The i18n locales selector is showing a dropdown with 3 languages (#9902)
  Make Scopes field in debates translatable (#9903)
  Make ToS agreement translatable (#9909)
  Fix issues with a11y specs (#9929)
  Remove invitations badge (#9906)
  Make initiatives order translatable (#9905)
  Add missing active actions on admin navigation menu (#9904)
  Fix user sign up with invalid name (#9896)
  Remove duplication of LastActivity queries (#9895)
  Rename IgnoredMethods to AllowedMethods in Rubocop configuration (#9893)
  Exclude malformed file from codeclimate configuration (#9910)
  Fix correct resource linking for amendments (#9887)
  Fix superposition in admin's error forms (#9871)
  Add missing i18n key in Initiatives (#9892)
  ...
entantoencuanto added a commit that referenced this pull request Oct 31, 2022
* develop: (36 commits)
  Fix proposal etiquette and length validator with base64 images (#9639)
  Install turbo-rails (#9881)
  Fix conference invitations (#9664)
  Fix invalid rendering of meeting and proposal body texts (#9764)
  Make documentation site work with multiple versions (#9917)
  Bump versions on install docs (#9916)
  Standardize CSV import formats and fix private users CSV import with invalid file (#9627)
  Fix: The i18n locales selector is showing a dropdown with 3 languages (#9902)
  Make Scopes field in debates translatable (#9903)
  Make ToS agreement translatable (#9909)
  Fix issues with a11y specs (#9929)
  Remove invitations badge (#9906)
  Make initiatives order translatable (#9905)
  Add missing active actions on admin navigation menu (#9904)
  Fix user sign up with invalid name (#9896)
  Remove duplication of LastActivity queries (#9895)
  Rename IgnoredMethods to AllowedMethods in Rubocop configuration (#9893)
  Exclude malformed file from codeclimate configuration (#9910)
  Fix correct resource linking for amendments (#9887)
  Fix superposition in admin's error forms (#9871)
  ...
Quentinchampenois pushed a commit to Quentinchampenois/decidim that referenced this pull request Nov 23, 2022
* Fix the axe matchers spec with the updated axe version

* Fix accessibility issue with links within callout elements
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module: core module: dev type: internal PRs that aren't necessary to add to the CHANGELOG for implementers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants