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

Screen layout is scaled way too high #62

Closed
phet-steele opened this issue Jul 12, 2017 · 8 comments
Closed

Screen layout is scaled way too high #62

phet-steele opened this issue Jul 12, 2017 · 8 comments

Comments

@phet-steele
Copy link

Somewhere down the road (between the release of 1.2.0 and 1.3.0), the layout of many things on the screen changed.

1.2.0 (the good one)
screen shot 2017-07-12 at 11 37 06 am

Every version 1.3.0+
screen shot 2017-07-12 at 11 37 19 am

Generally, it looks like the entire screen was "zoomed in" causing everything to be larger and:

  • The equation and circuit (all parts of it) are shifted down to the left
  • The panel and the buttons below are shifted right and up

@zepumph, myself and @ariel-phet recommend changing the screen layout back to its original size (like that of 1.2.0).

For phetsims/qa/issues/18.

@zepumph
Copy link
Member

zepumph commented Jul 14, 2017

I took a first pass at working on the layout. Most of the sim's layout is hardcoded, and after talking with @jonathanolson and @jessegreenberg, I ended up getting some great help converting it to a more dynamic layout. It is in a branch, just in case we decide to go a different way.

It's not perfectly the same at all, but I want to get feedback on it before continuing. I also found it helpful to use the ?dev parameter to look at screen bounds.

http://www.colorado.edu/physics/phet/dev/html/ohms-law/1.4.0-dev.1/ohms-law_en.html

https://phet.colorado.edu/sims/html/ohms-law/1.2.0/ohms-law_en.html?dev

assigning to @ariel-phet and @phet-steele for comments.

@phet-steele
Copy link
Author

@zepumph, most of this is looking great! There are little things that are different, like the initial scaling of some of the letters. I'll defer to @ariel-phet on how important the differences are.

My question, why are the ?dev bounds different between the two versions?

@phet-steele phet-steele removed their assignment Jul 17, 2017
@zepumph
Copy link
Member

zepumph commented Jul 17, 2017

My question, why are the ?dev bounds different between the two versions?

I deliberated on this with @jessegreenberg and he seemed to think that it was because 1.2 was published before we had a default screen layout bounds for all sims. I bet that at one point the OL layout bounds were changed to use this new default. I would even say that I think that this is where this issue's root problem came from. No matter it has been good to go back and improve OL's layout. It seems like it has been a long time coming.

@zepumph
Copy link
Member

zepumph commented Jul 17, 2017

I'm working on it a bit more this afternoon. I will make another dev version in a few hours. I already changed a few things to make the versions even closer.

@zepumph
Copy link
Member

zepumph commented Jul 17, 2017

@ariel-phet at some point I would love to chat about this, because there may be a line that we don't wish to cross in terms of getting it to be exactly like 1.2 vs cost to do it. Currently I'm just improving layout approach, so I don't think we are there yet.

@ariel-phet
Copy link

@zepumph no need for it to be exactly the same. Close enough will be good enough for this sim.

@ariel-phet ariel-phet removed their assignment Jul 17, 2017
zepumph added a commit that referenced this issue Jul 17, 2017
zepumph added a commit that referenced this issue Jul 17, 2017
zepumph added a commit that referenced this issue Jul 17, 2017
zepumph added a commit that referenced this issue Jul 17, 2017
@zepumph
Copy link
Member

zepumph commented Jul 17, 2017

http://www.colorado.edu/physics/phet/dev/html/ohms-law/1.4.0-dev.2/ohms-law_en.html

Could you confirm that this is "close enough" and you can close. I will merge into master now.

zepumph added a commit that referenced this issue Jul 17, 2017
@ariel-phet
Copy link

Looking good 🥇

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