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

review ComboBox in all sims to verify that they look "good" #452

Closed
21 tasks done
pixelzoom opened this issue Jan 17, 2019 · 25 comments
Closed
21 tasks done

review ComboBox in all sims to verify that they look "good" #452

pixelzoom opened this issue Jan 17, 2019 · 25 comments

Comments

@pixelzoom
Copy link
Contributor

pixelzoom commented Jan 17, 2019

In #430 (comment), @jessegreenberg noted:

The look in sims slightly changed because the option defaults and behavior for margins changed and the default for cornerRadius was reduced by half. I reviewed bending-light and molecule-shapes and the differences seem OK to me but want to make sure this is acceptable.

I replied:

FYI, trying to make things look exact is a rathole that I did go down. But there were so many things that made that difficult to accomplish. There were something about layout that were just wrong to begin with inside ComboBox and its components. The options for margins were overly complicated and were changed dramatically. And some of the defaults didn't really make senses as defaults; they were probably the values used by the sim that birthed ComboBox (BLL?) Anyway... I examined every ComboBox in every sim (which took forever) and verified that they all look reasonable. (Some btw did not look great before the changes.) If we want everything to look exactly the same as it did before the changes, that's going to take an undeterminable amount of time.

But I think this is a rathole that I should go down again, given how extensive the changes have been to ComboBox. So I'm going to review (and potentially adjust) all ComboBoxes in all sims.

Sims to review:

  • balancing-chemical-equations
  • beers-law-lab, concentration
  • bending-light
  • circuit-construction-kit-black-box-study
  • energy-skate-park
  • fluid-pressure-and-flow
  • gas-properties
  • interaction-dash-board (research)
  • least-squares-regression
  • masses-and-springs
  • molarity
  • molecule-polarity
  • molecule-shapes
  • pendulum-lab
  • ph-scale, phet-scale-basics
  • projectile-motion
  • states-of-matter
  • tambo (demo)
  • twixt (demo)
  • unit rates
  • wave-interference
@pixelzoom pixelzoom self-assigned this Jan 17, 2019
@pixelzoom
Copy link
Contributor Author

pixelzoom commented Jan 17, 2019

@ariel-phet gave me the go ahead, said this sounds worth doing. But there is no need to make ComboBoxes look exactly as before (good, because that's not even possible). Making it look "good" is important, and I can use my judgement on what is "good".

I've done this for my sims, but still need to visit other sims.

@pixelzoom pixelzoom changed the title review ComboBox uses in all sims review ComboBox in all sims to verify that they look "good" Jan 18, 2019
pixelzoom added a commit to phetsims/energy-skate-park that referenced this issue Jan 18, 2019
@pixelzoom
Copy link
Contributor Author

pixelzoom commented Jan 18, 2019

balancing-chemical-equations:

screenshot_981

pixelzoom added a commit to phetsims/bending-light that referenced this issue Jan 18, 2019
@pixelzoom
Copy link
Contributor Author

pixelzoom commented Jan 18, 2019

beers-law-lab, concentration:

screenshot_982

screenshot_983

@pixelzoom
Copy link
Contributor Author

pixelzoom commented Jan 18, 2019

bending-light:

screenshot_978

@pixelzoom
Copy link
Contributor Author

pixelzoom commented Jan 18, 2019

circuit-construction-kit-black-box-study:

screenshot_979

@pixelzoom
Copy link
Contributor Author

pixelzoom commented Jan 18, 2019

energy-skate-park:

screenshot_977

@pixelzoom
Copy link
Contributor Author

pixelzoom commented Jan 18, 2019

fluid-pressure-and-flow:

screenshot_980

@pixelzoom
Copy link
Contributor Author

gas-properties:

screenshot_984

@pixelzoom
Copy link
Contributor Author

interaction-dashboard:

screenshot_985

screenshot_986

@pixelzoom
Copy link
Contributor Author

least-squares-regression:

screenshot_987

@pixelzoom
Copy link
Contributor Author

masses-and-spring:

screenshot_988

@pixelzoom
Copy link
Contributor Author

molarity:

screenshot_989

@pixelzoom
Copy link
Contributor Author

molecule-polarity:

screenshot_990

@pixelzoom
Copy link
Contributor Author

molecules-shapes:

screenshot_991

@pixelzoom
Copy link
Contributor Author

pendulum-lab:

screenshot_992

@pixelzoom
Copy link
Contributor Author

ph-scale, ph-scale-basics:

screenshot_993

@pixelzoom
Copy link
Contributor Author

projectile-motion:

screenshot_994

@pixelzoom
Copy link
Contributor Author

states-of-matter:

screenshot_995

@pixelzoom
Copy link
Contributor Author

pixelzoom commented Jan 18, 2019

tambo (demo):

screenshot_996

@pixelzoom
Copy link
Contributor Author

twixt (demo):

screenshot_997

@pixelzoom
Copy link
Contributor Author

unit-rates:

screenshot_998

@pixelzoom
Copy link
Contributor Author

wave-interference:

screenshot_999

@pixelzoom
Copy link
Contributor Author

pixelzoom commented Jan 18, 2019

@ariel-phet I inspected all of the ComboBoxes in all sims, and only adjusted bending-light and energy-skate-park. Please look through the screenshots above. If you see any that you'd like to adjust, let me know. Otherwise please close this issue.

@pixelzoom pixelzoom assigned ariel-phet and unassigned pixelzoom Jan 18, 2019
@pixelzoom
Copy link
Contributor Author

@ariel-phet ping. I'd like to get the ComboBox work wrapped up.

@ariel-phet
Copy link

All looks good to me. 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

2 participants