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

Moving book with WASD in fullscreen causes Mac "funk" noise #167

Closed
ghost opened this issue Jan 2, 2019 · 19 comments
Closed

Moving book with WASD in fullscreen causes Mac "funk" noise #167

ghost opened this issue Jan 2, 2019 · 19 comments
Assignees
Labels

Comments

@ghost
Copy link

ghost commented Jan 2, 2019

Test Device

Dirac

Operating System

macOS 10.10.5
macOS 10.11.6

Browser

Safari

Problem Description

For phetsims/qa#244. In fullscreen, if the book is moved with WASD, the Mac "funk" noise is heard.

Funk Noise: https://www.youtube.com/watch?v=8rz44o_gGpE

@ghost ghost added the type:bug label Jan 2, 2019
@ghost ghost assigned zepumph Jan 2, 2019
@jessegreenberg
Copy link
Contributor

@lmulhall-phet @KatieWoe do you know if this has ever happened in BASE? For instance, does it happen with https://phet.colorado.edu/sims/html/balloons-and-static-electricity/latest/balloons-and-static-electricity_en.html?phetioValidateTandems in full screen mode?

@KatieWoe
Copy link
Contributor

KatieWoe commented Jan 8, 2019

Does happen on BASE in master @jessegreenberg

@jessegreenberg jessegreenberg assigned ghost and unassigned ghost Jan 8, 2019
@jessegreenberg
Copy link
Contributor

OK thanks @KatieWoe.

@KatieWoe
Copy link
Contributor

KatieWoe commented Jan 8, 2019

Also note, since I was a bit confused, it doesn't stop movement. It moves normally but also plays the sound.

@jessegreenberg
Copy link
Contributor

I see. Does this also happen in the deployed version of BASE?

@KatieWoe
Copy link
Contributor

KatieWoe commented Jan 8, 2019

Seems to yes

@jessegreenberg
Copy link
Contributor

OK, that is helpful, thanks.

@zepumph
Copy link
Member

zepumph commented Jan 14, 2019

We pushed a potential fix to master, @KatieWoe will you see if master is fixed please?

@zepumph zepumph assigned KatieWoe and unassigned zepumph Jan 14, 2019
@KatieWoe KatieWoe assigned zepumph and unassigned KatieWoe Jan 14, 2019
@KatieWoe
Copy link
Contributor

Seems fixed on master @zepumph
I checked on a 10.13.6 device, but the issue was present in the rc on this platform as well.

@zepumph
Copy link
Member

zepumph commented Jan 15, 2019

Thanks for checking! I have not yet cherry-picked this into master, but will do so to the branch soon.

@zepumph
Copy link
Member

zepumph commented Jan 26, 2019

Commit has been cherry-picked into branch (note to self)

@zepumph zepumph removed their assignment Jan 26, 2019
This was referenced Jan 26, 2019
@KatieWoe
Copy link
Contributor

1.5.0-rc.2 looks good

@JRomero0613
Copy link

I have been testing on 1.5.0-rc.2 on Safari on MacOS 10.12.6 and have seen this issue occur. This occurs when the book has been navigated to using tab, but has not been picked up using space. The outline around the book should be a solid pink line to get the "funk" noise to occur. If the book has been picked up the noise will not occur.

@KatieWoe
Copy link
Contributor

Since you can't move the book until it has been picked up, this may be intended. @zepumph?

@KatieWoe
Copy link
Contributor

phetsims/qa#276

@zepumph
Copy link
Member

zepumph commented Jan 31, 2019

Since you can't move the book until it has been picked up, this may be intended. @zepumph?

I agree. Arrow keys should not do anything in the sim when the book has not been grabbed. Since the "solid red lined" book is basically the same type of button as highlighting the reset all button, I would think this behavior is expected. I'm ready to close this if that seems alright to @JRomero0613.

@zepumph zepumph assigned JRomero0613 and unassigned zepumph Jan 31, 2019
@KatieWoe
Copy link
Contributor

One last note. The sound does only play when in full screen. Not sure if that says anything.

@zepumph
Copy link
Member

zepumph commented Jan 31, 2019

Sounds fine to @jessegreenberg and I, @JRomero0613 please close once you have reviewed this.

@zepumph
Copy link
Member

zepumph commented Feb 25, 2019

I'm going to close this. @JRomero0613 let me know if you have any other questions.

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

4 participants