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

Problem with "Other" #443

Open
danielruss opened this issue Jun 18, 2024 · 3 comments
Open

Problem with "Other" #443

danielruss opened this issue Jun 18, 2024 · 3 comments
Assignees

Comments

@danielruss
Copy link
Member

When the code was changed for in-survey accessibility enhancements (#436) The code to hand the "others" was changed.

originally:

 [...divElement.querySelectorAll("label input,label textarea")].forEach(inputElement => {
      let radioCB = document.getElementById(inputElement.closest('label').htmlFor) 

RadioCB is the input[type="radio"] or input[type="checkbox"] associated with the other input box.

This was changed to

let radioCB = document.getElementById(inputElement.id)
which is the type box.

This is why the results are screwing up when the "other" is selected.

quest/replace2.js

Lines 1036 to 1037 in ec22434

[...divElement.querySelectorAll("label input,label textarea")].forEach(inputElement => {
let radioCB = document.getElementById(inputElement.id)

@anthonypetersen
Copy link
Contributor

@danielruss - thanks for highlighting that change! switching the one line back appears to resolve everything, Joe has an additional test he is going to run for VoiceOver

@anthonypetersen
Copy link
Contributor

related to episphere/questionnaire#370

@JoeArmani
Copy link
Contributor

This bug was addressed in #444, and the revert/fix is included in release v1.1.7.

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

No branches or pull requests

3 participants