-
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
Sound blurs when dragging starts off of thumb #204
Comments
Hard to record, so I showed @jbphet directly. |
I've implemented a fix for this, but I'm not sure it will be fully acceptable on master because it involves some changes to common code that I think other devs may want to address in a more general manner. I'll leave it open and assigned to me to follow up on how to propagate to master. |
This issue seems fixed on Win 10 chrome 1.6.0-rc.3 |
Reopening because I'd like to discuss the fix with other devs. What I did was to make flags available for checking whether the slider thumb or track was being dragged, but this seems a bit hacky and I suspect that a similar situation may come up for other controls, i.e. where we want to be able to tell if the change to the control was done via mouse/touch versus keyboard interaction. |
This was discussed in the 1/10/2019 developer meeting, and it was pointed out that the scenery event could be used to distinguish between the different usage modes, but that event isn't being passed through to the |
This is done. I handled it a little differently on the 1.6 release branch of RIAW than on master, since I had implemented a fix on that branch before I discussed the issue with the other devs, who provided an idea for a more general solution. The more general solution was done on master. |
…hetsims/resistance-in-a-wire#204 (cherry picked from commit 274a341)
Test device:
Dell and Mac
Operating System:
Win 10 and 10.11
Browser:
Chrome
Problem description:
For phetsims/qa#239
It is possible to click on a part of the slider with the mouse and make the thumb jump to that position. If you start dragging the thumb the resulting sound is grainy and seems to blur together.
Steps to reproduce:
Troubleshooting information (do not edit):
Name: Resistance in a Wire
URL: https://phet-dev.colorado.edu/html/resistance-in-a-wire/1.6.0-rc.2/phet/resistance-in-a-wire_en_phet.html
Version: 1.6.0-rc.2 2018-12-13 16:36:32 UTC
Features missing: touch
Flags: pixelRatioScaling
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/71.0.3578.98 Safari/537.36
Language: en-US
Window: 1536x732
Pixel Ratio: 2.5/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: 16384x16384
OES_texture_float: true
Dependencies JSON: {"assert":{"sha":"928741cf","branch":"HEAD"},"axon":{"sha":"de77d4b5","branch":"HEAD"},"brand":{"sha":"1fd6682e","branch":"HEAD"},"chipper":{"sha":"fa2fbadf","branch":"HEAD"},"dot":{"sha":"bbbd8526","branch":"HEAD"},"joist":{"sha":"82521d0c","branch":"HEAD"},"kite":{"sha":"380cef53","branch":"HEAD"},"phet-core":{"sha":"1b90ac2f","branch":"HEAD"},"phet-io":{"sha":"38d7b161","branch":"HEAD"},"phet-io-wrapper-classroom-activity":{"sha":"246085c1","branch":"HEAD"},"phet-io-wrapper-hookes-law-energy":{"sha":"7479b0ec","branch":"HEAD"},"phet-io-wrapper-lab-book":{"sha":"c46f7839","branch":"HEAD"},"phet-io-wrappers":{"sha":"a6bc62ca","branch":"HEAD"},"phetcommon":{"sha":"cd63d89a","branch":"HEAD"},"query-string-machine":{"sha":"06ed6276","branch":"HEAD"},"resistance-in-a-wire":{"sha":"c996a2e3","branch":"HEAD"},"scenery":{"sha":"9953c5f7","branch":"HEAD"},"scenery-phet":{"sha":"15ed6d54","branch":"HEAD"},"sherpa":{"sha":"2cd50500","branch":"HEAD"},"sun":{"sha":"9ee72759","branch":"HEAD"},"tambo":{"sha":"65315b32","branch":"HEAD"},"tandem":{"sha":"ed8f8f1d","branch":"HEAD"}}
The text was updated successfully, but these errors were encountered: