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

Need name response on Range Combobox #445

Closed
terracoda opened this issue Mar 17, 2022 · 8 comments
Closed

Need name response on Range Combobox #445

terracoda opened this issue Mar 17, 2022 · 8 comments

Comments

@terracoda
Copy link
Contributor

I think implementation is missing a name response, but I am double checking. Will report back asap.

@terracoda terracoda self-assigned this Mar 17, 2022
@terracoda
Copy link
Contributor Author

Yes, we have an object response instead the combined name response when the combobox is opened with pointer input.

And the text for the hint needs to be a customized one, different from PDOM.

Upon opening the range combobox with a click or tap, with hints checked, I should hear:

  • Range: 0 to 10, Choose range for tick marks.

And the same hint for the other inputs.

Everything else is good.

@zepumph
Copy link
Member

zepumph commented Mar 17, 2022

We will want to talk about this. It is quite hard to do this, but I'd like to explain this more. I believe we discussed this a bit in phetsims/sun#726.

@terracoda
Copy link
Contributor Author

Ok, we can discuss at today's meeting. The weirdest part for me, is that there is no Name Response at all when no checkboxes are checked in Preferences, and "Range: 0 to 10" is text on the screen.

I am just copying in current design to document it:
image
image
image

@zepumph
Copy link
Member

zepumph commented Mar 18, 2022

@terracoda and I talked through this problem today. The main issue is that when you select the button, you are actually getting the voicing because the list item Node is focused, thus, to add this, we would also need to say the name response when cycling through options. This is not ideal.

@terracoda thinks this may be acceptable, because you still get the name response upon selection, but will want to discuss with @emily-phet.

The weirdest part for me, is that there is no Name Response at all when no checkboxes are checked in Preferences, and "Range: 0 to 10" is text on the screen.

You will still hear "Range: 0 to 10" upon selection (even if the selection didn't change), you just don't hear it when you open the combo box with a mouse/touch.

Over to @terracoda to see if there is anything else here.

@zepumph
Copy link
Member

zepumph commented Mar 18, 2022

@terracoda! I think I can make this happen easily afterall!!

@jessegreenberg and I discussed, I can adapt supplyHintResponseOnNextFocus to handle the case where we supply the nameResponse instead of the hint response. Assigning back to me.

@terracoda
Copy link
Contributor Author

That would be ideal, especially for Sims that have really meaningful comboboxes like Molarity.

@zepumph
Copy link
Member

zepumph commented Mar 18, 2022

Ok. @terracoda please review master now. I think it is you desire with the name response on open. Commits in phetsims/sun#726

@terracoda
Copy link
Contributor Author

The combobox boxes sound great with pointer and keyboard input. Nice job.

I was thinking of shortening the hint response from:

  • When tick marks shown, choose range for tick marks.

To:

  • Choose range for tick marks.

But since we cannot voice the disabled state of the combobox, I think it best to keep the longer hint. We can reconsider the hint response string in the future once we are able to voice the disabled state with "Range: Tick Marks Hidden."

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants