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 compares hand location to same hand #526

Closed
Tracked by #863
Nancy-Salpepi opened this issue Nov 16, 2022 · 7 comments
Closed
Tracked by #863

Voicing compares hand location to same hand #526

Nancy-Salpepi opened this issue Nov 16, 2022 · 7 comments

Comments

@Nancy-Salpepi
Copy link

Nancy-Salpepi commented Nov 16, 2022

Test device
MacBook Air (m1 chip)

Operating System
macOS 13.0.1

Browser
safari 16.1 and chrome

Problem description
For phetsims/qa#852, when moving the left hand, voicing compares its location to 'left hand' and the same thing happens when moving the right hand.

Steps to reproduce

  1. Turn on Voicing and check Sim Voicing Options checkboxes
  2. On either screen, move left hand --new position compared to left hand
  3. Move right hand--new position compared to right hand

Visuals

wronghandcomparison.mov
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 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/107.0.0.0 Safari/537.36 Language: en-US Window: 1544x712 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
Copy link
Author

Nancy-Salpepi commented Nov 18, 2022

I also see this issue with VoiceOver and in a11y view

@zepumph
Copy link
Member

zepumph commented Nov 23, 2022

94f074f has the bug, 5ee9dc4 does not.

@zepumph
Copy link
Member

zepumph commented Nov 23, 2022

Classic, looks like the bug was introduced by a simple mistake in d4f186a#diff-b1e62322eee34f09e99fbdae2de1b90ed22ebae29b875e92c7bf5567705483a2L284.

zepumph added a commit that referenced this issue Nov 23, 2022
@zepumph
Copy link
Member

zepumph commented Nov 23, 2022

Fixed by 91fa9af, to cherry pick that.

@zepumph zepumph removed their assignment Nov 23, 2022
@zepumph
Copy link
Member

zepumph commented Dec 5, 2022

Done and ready for confirmation in next version.

@Nancy-Salpepi
Copy link
Author

This looks good in 1.2.0-rc.2 with Voicing, VoiceOver and a11y view on mac + safari. Will reopen if anything comes up.

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