Skip to content

Conversation

@biou
Copy link

@biou biou commented Nov 26, 2019

It might be interesting to add an option in AccessConfig to manage animations. This has been done in the accessibility parameters of other websites, for example sncf.fr.

Compared to other AccessConfig parameters, animations can often be managed by JavaScript, so we should find a way to notify this code of some changes in the settings. There are several ways to do this, I propose to use a callback function, mainly because it is simple to use and does not need a polyfill like custom events in old versions of IE.

What do you think, is it interesting?

@DuaelFr
Copy link

DuaelFr commented Jun 30, 2022

This MR looks good to me. It's working as expected and I like that there is a callback for JS to react to the change.
Thanks! I hope this could be merged officially!

@biou
Copy link
Author

biou commented Jul 6, 2022

This MR looks good to me. It's working as expected and I like that there is a callback for JS to react to the change.
Thanks! I hope this could be merged officially!

Thanks for your support!
I discussed this PR with @access42 a long time ago and I think that this approach has some limits, but I don't remember what it was. Maybe it is just too complex for front-end developers. To be double-checked with them. If you need this feature, feel free to fork this repo and integrate this PR. All feedback is of course welcome!

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.

2 participants