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

remove accessibility.force_disabled #1728

Closed
Thorin-Oakenpants opened this issue Sep 25, 2023 · 1 comment
Closed

remove accessibility.force_disabled #1728

Thorin-Oakenpants opened this issue Sep 25, 2023 · 1 comment

Comments

@Thorin-Oakenpants
Copy link
Contributor

Thorin-Oakenpants commented Sep 25, 2023

this can be removed now we have CtW (cache the world)

links

@Thorin-Oakenpants Thorin-Oakenpants changed the title reminder: accessibility.force_disabled reminder: accessibility.force_disabled / accessibility.force_disabled Sep 25, 2023
@Thorin-Oakenpants
Copy link
Contributor Author

man, I am not with it today - getting accessibility.cache.enabled mixed up

accessibility.force_disabled obviously exists - but, and I read a lot of bugzillas, I thought the rollout didn't take the force pref into account for some reason. That's not true AFAICT, and the pref was simply for the rollout (i.e to change from the old method to CtW)

So we have two options

  • check on some aria test page that force_disabled still works, make it inactive, move it (it has no privacy/security implications) - maybe to personal
  • remove it from the user.js and be done with it

I'm in favor of removing it. Class discuss

@Thorin-Oakenpants Thorin-Oakenpants changed the title reminder: accessibility.force_disabled / accessibility.force_disabled remove accessibility.force_disabled Sep 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

1 participant