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

Resizing window during input event playback should not change scene size #37

Closed
jonathanolson opened this issue Aug 9, 2013 · 12 comments

Comments

@jonathanolson
Copy link
Contributor

No description provided.

@ghost ghost assigned jonathanolson Aug 9, 2013
@samreid
Copy link
Member

samreid commented Oct 17, 2013

@jonathanolson is this solved?

@pixelzoom
Copy link
Contributor

Same question that @samreid had on 10/17/2013. @jonathanolson is this solved?

@jonathanolson
Copy link
Contributor Author

Input event recording and playback hasn't yet been fully updated for Scenery 0.2. Is getting that working a priority for the summer deployment @pixelzoom?

@pixelzoom
Copy link
Contributor

Now that I understand what this is... No, not a priority for the "Summer 2015 redeployment", so removing that label.

@pixelzoom
Copy link
Contributor

@jonathanolson Is this issue still relevant? If no, please close. If yes, please briefly describe what needs to be done.

@jonathanolson
Copy link
Contributor Author

@samreid, is this an issue for current playback?

@samreid
Copy link
Member

samreid commented Nov 30, 2016

This is an issue. Our current workaround is to instruct researchers to not resize the window during playbacks. What we should do instead is make it so that window resize events do not end up changing the scenery display size during playback.

@jonathanolson
Copy link
Contributor Author

@samreid, is there documentation on how to record and play back? I'd like to reproduce and understand so I can fix it.

I was able to record a session with the record wrapper, but the playback wrapper (linked in phetmarks) doesn't seem to exist.

@jonathanolson
Copy link
Contributor Author

Via @phet-steele, saw the metacog playback code (playback looks nice!)

It looks like this problem should be present if it wasn't wrapped in an iframe.

@samreid or @zepumph, what flags would Sim be able to check to see if it is in "playback" mode? handlePlaybackEvent looks like it's reaching in very directly, so I'm really not sure if there is a status that lets Sim know it's being played back.

Can I use activeProperty, that sets display.interactive?

@samreid
Copy link
Member

samreid commented Feb 2, 2017

Our current workaround is to instruct researchers to not resize the window during playbacks.

By embedding the playback frame inside an iframe, this is no longer a problem. The size from the recording is set to the playback iframe and the playback iframe doesn't resize otherwise.

@samreid or @zepumph, what flags would Sim be able to check to see if it is in "playback" mode?

Looking in mirror.html, I'm seeing disableRequestAnimationFrame as a flag that is currently set for the playback frame. And in playSession.js, I see setPlaybackMode which looks like it sets phet.joist.playbackMode. That seems like the best flag to use. I'll create an issue to see if we should adopt that flag for mirror.html too.

@samreid samreid assigned jonathanolson and unassigned samreid Feb 2, 2017
jonathanolson added a commit that referenced this issue Feb 2, 2017
@jonathanolson
Copy link
Contributor Author

Done, can you review?

@samreid
Copy link
Member

samreid commented Feb 2, 2017

The code change looks good, thanks!

@samreid samreid closed this as completed Feb 2, 2017
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

3 participants