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

Voicing turns on after entering a menu or dialog #541

Closed
Tracked by #874 ...
Nancy-Salpepi opened this issue Dec 13, 2022 · 10 comments
Closed
Tracked by #874 ...

Voicing turns on after entering a menu or dialog #541

Nancy-Salpepi opened this issue Dec 13, 2022 · 10 comments
Labels

Comments

@Nancy-Salpepi
Copy link

Nancy-Salpepi commented Dec 13, 2022

Test device
MacBook Air (m1 chip) and Chromebook

Operating System
macOS 13.0.1

Browser
safari and chrome

Problem description
For phetsims/qa#863, after turning Voicing off in the Toolbar, entering/exiting the PhET Menu, Preferences dialog, or Keyboard Shortcuts dialog turns Voicing on (even though switch is still in off position).

Steps to reproduce

  1. In the preferences menu turn on Voicing and check all Sim Voicing Options
  2. In the toolbar, turn off Sim Voicing
  3. Click on the Keyboard Shortcuts dialog (or preferences or Phet menu)
  4. Press Esc or click outside box
  5. Move a hand with the mouse, or tab through the sim with keyboard nav--Voicing is on

I was able to reproduce this in published GFLB.
EDIT: I can't reproduce in published GFLB. It must have been a caching issue. Sorry.

Visuals

VoicingTurnsOn.mov
Troubleshooting information: !!!!! DO NOT EDIT !!!!! Name: ‪Ratio and Proportion‬ URL: https://phet-dev.colorado.edu/html/ratio-and-proportion/1.2.0-rc.2/phet/ratio-and-proportion_all_phet.html Version: 1.2.0-rc.2 2022-12-05 22:59:52 UTC Features missing: applicationcache, applicationcache, touch Flags: pixelRatioScaling User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/108.0.0.0 Safari/537.36 Language: en-US Window: 1581x816 Pixel Ratio: 1.7999999523162842/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: 31 uniform: 1024 Texture: size: 16384 imageUnits: 16 (vertex: 16, combined: 32) Max viewport: 16384x16384 OES_texture_float: true Dependencies JSON: {}
@Nancy-Salpepi Nancy-Salpepi added the type:bug Something isn't working label Dec 13, 2022
@zepumph
Copy link
Member

zepumph commented Dec 14, 2022

Looks like a bug! Thanks

@jessegreenberg jessegreenberg self-assigned this Dec 16, 2022
jessegreenberg added a commit to phetsims/joist that referenced this issue Dec 16, 2022
@jessegreenberg
Copy link
Contributor

The above commit is a way to fix it. @zepumph can you please review?

@jessegreenberg jessegreenberg removed their assignment Dec 16, 2022
@zepumph
Copy link
Member

zepumph commented Dec 16, 2022

Amazing! Thanks.

@zepumph
Copy link
Member

zepumph commented Jan 3, 2023

Cherry picked. QA, feel free to close if all is well.

@Nancy-Salpepi
Copy link
Author

This is still broken in rc.3.
Steps:

  1. Turn on Voicing in the Preferences dialog
  2. Turn off Voicing in the Toolbar
  3. Turn Voicing off and then on in the Preferences dialog
  4. Exit Preferences dialog
  5. Click on or tab to anything--Voicing is on even though the toggle is still in the "off" position in the toolbar.

@jessegreenberg jessegreenberg self-assigned this Jan 4, 2023
@Nancy-Salpepi
Copy link
Author

Noting that I don't see the issue in #541 (comment) in Friction 1.6.0-dev.28

@jessegreenberg
Copy link
Contributor

jessegreenberg commented Jan 4, 2023

Thanks @Nancy-Salpepi, that helped identify it. It looks like the fix was cherry picked into a release branch but just didn't make it into dependencies.json. I confirmed I am not seeing this in Friction 1.6.0-dev.28 or master.

@zepumph
Copy link
Member

zepumph commented Jan 4, 2023

Oops! I'll make sure that this is fixed in the next rc.

@zepumph
Copy link
Member

zepumph commented Jan 4, 2023

It is in the next rc. Feel free to close in the spot check.

@Nancy-Salpepi
Copy link
Author

This looks good in rc.4

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

No branches or pull requests

3 participants