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

incorrect scaling of AboutDialog #639

Closed
pixelzoom opened this issue Jul 13, 2020 · 5 comments
Closed

incorrect scaling of AboutDialog #639

pixelzoom opened this issue Jul 13, 2020 · 5 comments

Comments

@pixelzoom
Copy link
Contributor

Reported in phetsims/ph-scale#168. Possibly related to #586.

AboutDialog scaling is incorrect. Unclear whether this issue is specific to AboutDialog, or is a more general problem with Dialog scaling. There are 2 main problems:

(1) On non-Home screens, long credits can result in the About dialog overlapping the navigation bar, and/or falling off the top of the screen. See screenshot below. Dialogs should always be scaled to fit the ScreenView layoutBounds.

screenshot_427

(2) On the Home screen, the dialog size is different than the other screens. See screenshot below. This may be because HomeScreenView.js is using non-standard layoutBounds:

const LAYOUT_BOUNDS = new Bounds2( 0, 0, 768, 504 );

screenshot_428

@pixelzoom pixelzoom self-assigned this Jul 13, 2020
pixelzoom added a commit that referenced this issue Jul 13, 2020
@pixelzoom
Copy link
Contributor Author

@ariel-phet asked me to apply the scaling workaround from #586 to AboutDialog. I noted that this would result in a smaller About dialog in all sims, as well as the Home screen. And a sim with long credits may still encounter the problem.

A real solution is still needed, probably in connection with #586. Assigning to @ariel-phet to prioritize and assign.

@ariel-phet
Copy link

I am currently marking this work deferred. As long as this workaround is working for the moment the work is not a major priority. It is important the about dialog fit within the bounds and everything in it can be read, but the aethetics of this dialog are not of major concern (no effect on pedagogy etc).

A long term solution should be found at some point.

@KatieWoe
Copy link

Noting here that the size difference is fairly noticeable and may be difficult to read in cases where there is a lot of text. Should be fine for now, but I will keep an eye out and make an issue if a future sim seems to have too small of text. Seen on master 7/14/2020

@KatieWoe
Copy link

Potential example of this in Acid Base Solutions. Has a fairly large amount of text and a home screen where things would be smaller. May not be problematic in this case, but shows how it might get there.
Published:
published
Master:
new

@pixelzoom
Copy link
Contributor Author

Looks like @jonathanolson removed the workaround in 695644f for #586.

Remaining AboutDialog tweaks will be done in #586.

Closing.

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

4 participants