Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Accessibility fixes #112

Open
wants to merge 29 commits into
base: v3
Choose a base branch
from
Open

Accessibility fixes #112

wants to merge 29 commits into from

Conversation

felixgirault
Copy link
Contributor

No description provided.

Using swc cuts down the execution time by a factor of 10.
This allows using the lib within a bundler, while still providing an
autoloading mechanism for standard usage.
Translations are migrated from yaml to ts to simplify the build and
import process, as there is no real advantage to using yaml here.
The preloading mechanism is removed, as there is a 99% chance that the
UI will be shown on a first visit.
Using tsup simplifies and speeds up the process.
`tsup` was generating a custom type for each and every translation
object, with all of their properties, resulting in a huge declaration.
It turns out that `satisfies` does not impact the output, whereas `as`
forces the `Translation` type to be used.
We're still keeping the `satisfies` part, as it provides better
typechecking (i.e. warning about missing or unknown properties).
* Folded the annotated configuration by default
* Added details on theming
* fixed page markup
* added missing styles
* added a section in the doc
When running a dev server, renaming chunks doesn't work as expected.
HTML elements require dataset props to be camelCased.
When using React 18+, the app must be rendered using createRoot.
Events added by micromodal weren't properly unregistered, causing errors
after closing a modal.
Also, hiding the banner with `display: none` was preventing focus to be
moved back to it after closing the modal.
* harmonized color names
* fixed issues when closing modals by clicking the backdrop
* fixed focus issue on toggle buttons (using `aria-disabled` instead of
`disabled` prevents losing focus when the state changes)
The link's title wasn't valid according to the RGAA.
Instead of fixing it, we're removing the link altogether as it is of no
real value.
@see RGAA 6.1
The differenciation was made by color only, there is now also a change
in shape.
@see RGAA 3.1
We're deviating slightly from the official component styles so that
buttons appear in the right order.
@see
https://www.systeme-de-design.gouv.fr/composants-et-modeles/composants/gestionnaire-de-consentement
@see RGAA 12.8
Most texts were translated automatically from english, pending a better
translation.
@see RGAA 7.1
@felixgirault felixgirault self-assigned this Oct 11, 2024
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.

1 participant