Skip to content

Improve accessibility vol 2#6124

Closed
ahukkanen wants to merge 369 commits intodecidim:developfrom
mainio:accessibility/final
Closed

Improve accessibility vol 2#6124
ahukkanen wants to merge 369 commits intodecidim:developfrom
mainio:accessibility/final

Conversation

@ahukkanen
Copy link
Copy Markdown
Contributor

🎩 What? Why?

The great work that got started by MoretS continues.

This pull request is a big one but fixes many accessibility issues in Decidim. More information about the fixed issues is available from:
https://trello.com/b/gu7Ztm3C/decidim-accessibility

These issues have been reported by accessibility experts for two cities in Finland.

There are still some issues remaining after this but this addresses all the "extremely important" and "important" level issues, which should be enough to pass most accessibility reviews.

A thing to note is also that the accessibility reviews have been done on customized Decidim themes, so the color contrasts of the default theme have not been reviewed by any experts. That is left for further evaluation.

📌 Related Issues

📋 Subtasks

  • Add CHANGELOG entry
  • Add tests

ahukkanen added 30 commits May 17, 2020 23:54
- Fix the expected titles and button texts after the ALLCAPS
  change [TAC042]
- Fix expected error states when passing too short texts and
  texts with too many uppercase characters [TAC061]

Original commit: bd53650
Original commit: c00b687
- Promote to proposal button for amendments
- Collaborative drafts edit and request access buttons

Original commit: 3ae8d9d
@microstudi
Copy link
Copy Markdown
Contributor

microstudi commented Jul 24, 2020

This PR has been superseded by #6253 into develop and #6336 for the 0.22 branch

@microstudi microstudi removed the release: v0.22 Issues that need to be tackled for v0.22 label Jul 24, 2020
@tramuntanal
Copy link
Copy Markdown
Contributor

Closing as it has been superseded by #6253

@ahukkanen ahukkanen deleted the accessibility/final branch June 30, 2021 10:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants