-
Notifications
You must be signed in to change notification settings - Fork 8
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
Feature test: Pan and Zoom #785
Comments
QA is done |
Awesome, thanks very much QA. This is a super helpful list of issues. Will be going through each. |
These are the issues from this test that would be good to review with someone from the design team:
The rest are about the implementation or almost ready to close. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
It would be really helpful to do some testing of the Pan and Zoom feature. There were many changes to that code over the last two weeks. But mostly this will help make sure that this feature is behaving well for the sims that are going to be published next quarter and so we can triage things before taking RC SHAs.
Here is the list of sims for upcoming publications, can you please test Pan and Zoom in each of these?
Any issues that you find are likely to be a problem for all sims, so could you please put new issues in https://github.com/phetsims/scenery/issues ?
I think testing off of master should be sufficient for this, it doesn't seem worth building 14 dev versions. But let me know if that would be helpful.
I wrote up some new notes about the Pan and Zoom feature for things that may not be intuitive in the QA book. https://github.com/phetsims/qa/blob/master/documentation/qa-book.md#pan-and-zoom-testing
The text was updated successfully, but these errors were encountered: