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.

Sign up for GitHub

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

Jump to bottom

ariaQuestion content in example.json #211

Closed
tomallpress opened this issue Nov 13, 2023 · 1 comment · Fixed by #212
Closed

ariaQuestion content in example.json #211

tomallpress opened this issue Nov 13, 2023 · 1 comment · Fixed by #212
Assignees
@kirsty-hames
Labels
accessibility bug

Comments

@tomallpress
Copy link
Contributor

Subject of the issue/enhancement/features

Could the text 'Question text specifically for screen readers.' be removed from the ariaQuestion json element please. Just so there isn't a risk of this being added in redundantly when it isn't needed, or it being left in accidently on accessible courses where ariaQuestion text isn't required.
@kirsty-hames
Copy link
Contributor

I agree with removing the ariaQuestion placeholder text for the same reasons.

This applies to all question components: Slider, confidenceSlider, textInput, matching.
GMCQ is already empty.

@kirsty-hames kirsty-hames self-assigned this Nov 20, 2023
@kirsty-hames kirsty-hames moved this from New to Assigned in adapt_framework: The TODO Board Nov 20, 2023
This was referenced Nov 20, 2023
Merged
Closed
@kirsty-hames kirsty-hames moved this from Assigned to Needs Reviewing in adapt_framework: The TODO Board Nov 20, 2023
This was referenced Nov 20, 2023
Closed
Closed
Closed
@github-project-automation github-project-automation bot moved this from Needs Reviewing to Recently Released in adapt_framework: The TODO Board Nov 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kirsty-hames kirsty-hames

Labels
accessibility bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update: Remove ariaQuestion placeholder text (fixes #211) adaptlearning/adapt-contrib-mcq
3 participants
@kirsty-hames @tomallpress @joe-allen-89