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.

Sign up for GitHub

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

Jump to bottom

polish layout #41

Closed
pixelzoom opened this issue Feb 10, 2020 · 3 comments
Closed

polish layout #41

pixelzoom opened this issue Feb 10, 2020 · 3 comments
Assignees
@arouinfar
Labels
design:general dev:code-review

Comments

@pixelzoom
Copy link
Contributor

Noticed during #2 (code review).

The layout of both screens could use some polishing, to:

  • make more effective use of negative space
  • alleviate some crowding

See screenshots below with ?dev to show layoutBounds.

screenshot_109

screenshot_110
@ariel-phet ariel-phet self-assigned this Feb 11, 2020
@ariel-phet
Copy link

Self-assigning

@ariel-phet ariel-phet assigned arouinfar and unassigned ariel-phet Feb 13, 2020
@ariel-phet
Copy link

@arouinfar and I had an initial discussion today. She will put some mockups in this issue.

@ariel-phet ariel-phet mentioned this issue Feb 13, 2020
Closed
@DianaTavares
Copy link

We have been working on the design of this simulation considering the information in this issue. An updated version of the mock-up can be found in the design doc, and issues related to the changes that are needed in the dev version are going to be opened.

Closing this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@arouinfar arouinfar

Labels
design:general dev:code-review
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@pixelzoom @ariel-phet @arouinfar @DianaTavares