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

If accessibility is disabled dialogs cannot be opened #616

Open
chris-steele opened this issue Jan 16, 2025 · 6 comments · May be fixed by #617
Open

If accessibility is disabled dialogs cannot be opened #616

chris-steele opened this issue Jan 16, 2025 · 6 comments · May be fixed by #617
Labels
bug Something isn't working high priority

Comments

@chris-steele
Copy link
Contributor

chris-steele commented Jan 16, 2025

Observations:

  • unopened dialogs are styled display:none by default
  • showModal is not called if accessibility has been disabled
  • problem began with core v6.60.3
@chris-steele
Copy link
Contributor Author

@oliverfoster don't suppose you could have a look at this, could you?

@oliverfoster
Copy link
Member

I don't understand what you mean by 'if accessibility is disabled". Could you clarify plz?

@chris-steele
Copy link
Contributor Author

Hi @oliverfoster, sorry for not being clearer, by disabling accessibility I meant setting the following:

"_accessibility": {
    "_isEnabled": false

ref: https://github.com/adaptlearning/adapt_framework/blob/master/src/course/config.json#L14

@oliverfoster
Copy link
Member

I thought we'd removed that option. Why would you set this to false?

@chris-steele
Copy link
Contributor Author

I thought we'd removed that option. Why would you set this to false?

it's a jolly good question, I don't think it would give you much advantage in any arena?

Thank you so much for that PR 👍

@oliverfoster
Copy link
Member

Why are you setting it to _isEnabled: false? How did you find this bug? I need to know to tell you if the fix will work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working high priority
Development

Successfully merging a pull request may close this issue.

2 participants