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

Dialogs may be a different size if opened while zoomed in #611

Closed
KatieWoe opened this issue Jul 31, 2020 · 6 comments
Closed

Dialogs may be a different size if opened while zoomed in #611

KatieWoe opened this issue Jul 31, 2020 · 6 comments
Assignees
Labels

Comments

@KatieWoe
Copy link

For phetsims/qa#520
When you open the about dialog while zoomed in to the sim, it seems to be larger (in comparison to the rest of the sim, not just the obvious) than if it is opened when not zoomed in. It does not go off screen from what I can see, is very minor, and I believe the dialog code has already changed since this SHAs so I don't think anything really needs to be done here, but feel free to disagree.

Steps:

  1. Put iPad in landscape mode
  2. Zoom in to sim.
  3. Open about dialog
  4. Zoom out and observe size
  5. Close out of about dialog
  6. Open about dialog again while fully zoomed out. Observe size of dialog

ezgif com-video-to-gif

@jessegreenberg
Copy link
Contributor

I am having a difficult time reproducing this. I haven't been able to see this yet on my Windows or iPad devices. Over slack @KatieWoe mentioned that it only seemed to happen in landscape mode, but I haven't seen it in landscape or portrait mode yet.

@jessegreenberg
Copy link
Contributor

I can produce the bug with stringTest=double. So the dialog needs to be larger for this to happen.

@jessegreenberg
Copy link
Contributor

OK, I verified that this is not an issue in master because of recent changes to Dialog that position pop ups based on layoutBounds.

@jessegreenberg
Copy link
Contributor

I spent some time trying to figure out why it was happening on the old shas. I was able to verify that bounds, screenBounds, and scale used by dialog layout are not changing with zoom which is as I would expect. I also made sure that even longer strings like stringTest=long didn't cause worse problems with layout in the MAL RC.

Since this is fixed on master I am going to go ahead and close, I don't think it is worth getting to the bottom of this since we wouldn't fix in branch anyway.

@jessegreenberg
Copy link
Contributor

Actually, reopening for @KatieWoe to verify that this behavior is indeed fixed in master.

@KatieWoe
Copy link
Author

This does look fixed on master

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

2 participants