-
Notifications
You must be signed in to change notification settings - Fork 4
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
Voicing reads combo box in a previously used locale #528
Comments
Voicing is an "english only" feature, so this bug is actually that voicing doesn't turn off when switching locale. I'll take a look. |
@zepumph voicing seems to be working correctly with mac + safari/chrome. With Voicing enabled, when I switch locale it becomes disabled and I hear "Voicing off. Only available in English." |
Also not seeing this issue with the Chromebook. |
when the locale was danish, voicing was actually correctly disabled, but when I switched back to english and voicing was re-enabled everything except the combo box was being read correctly |
Thanks for the clarification @stemilymill! Steps:
|
Wonderful. You are seriously talented folks. Thanks! I also found the same issue with voicing name response for the combo box list items. To cherry pick: phetsims/sun@ddb0240 |
Done and ready for confirmation in next version. |
This looks good in rc.2 |
This seems to still be broken in rc.2 with the keyboard dialog this time. Steps:
Italian2.mov |
Excellent! I can reproduce in rc.2 but not on master. I believe it to be because of phetsims/scenery-phet@005be16 which I thought wouldn't matter for RAP, oops. I can cherry pick that into the rc, but first will you confirm that it is not reproducible on master please? |
I can't reproduce on master either. yay. |
Wonderful! |
I'll cherry pick phetsims/scenery-phet@005be16 but I will also make sure that fixes it and doesn't need any other changes from that monster issue. |
I also had to cherry pick phetsims/axon@2e738ee also. But now it is all cherry picked. Cherry picked. QA, feel free to close if all is well. |
Looks good in 1.2.0-rc.3! Closing |
For phetsims/qa#852
Test device
Dell laptop
Operating System
Win10 21H2
Browser
Chrome 107.0.5304.107
Problem description
With voicing enabled, I was switching between different locales and using keynav. When I had switched from Danish to English, and shift+tabbed back to play with the hands, voicing read over "Challenge 3" (which was currently selected in the combo box) as "Opgave 3" with 3 as english three and Opgave in an English pronunciation.
When I entered the combo box, voicing read everything as expected, including "Challenge 3."
Steps to reproduce
I am still working on this. After recording the video below I changed the locale to German and then back to English and the self-voicing behaved as normally expected.
Visuals
voicing.reading.danish.translated.string.mp4
Troubleshooting information:
!!!!! DO NOT EDIT !!!!!
Name: Ratio and Proportion
URL: https://phet-dev.colorado.edu/html/ratio-and-proportion/1.2.0-rc.1/phet/ratio-and-proportion_all_phet.html
Version: 1.2.0-rc.1 2022-11-11 22:40:36 UTC
Features missing: applicationcache, applicationcache, touch
Flags: pixelRatioScaling
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/107.0.0.0 Safari/537.36
Language: en-US
Window: 1493x775
Pixel Ratio: 1.125/1
WebGL: WebGL 1.0 (OpenGL ES 2.0 Chromium)
GLSL: WebGL GLSL ES 1.0 (OpenGL ES GLSL ES 1.0 Chromium)
Vendor: WebKit (WebKit WebGL)
Vertex: attribs: 16 varying: 30 uniform: 4096
Texture: size: 16384 imageUnits: 16 (vertex: 16, combined: 32)
Max viewport: 32767x32767
OES_texture_float: true
Dependencies JSON: {}
The text was updated successfully, but these errors were encountered: