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

stringTest=long makes lines not aligned with associated strings #305

Closed
ghost opened this issue Jul 25, 2018 · 11 comments
Closed

stringTest=long makes lines not aligned with associated strings #305

ghost opened this issue Jul 25, 2018 · 11 comments

Comments

@ghost
Copy link

ghost commented Jul 25, 2018

Noticed this while looking at #303...

The natural length, mass equilibrium, and movable lines become slightly misaligned when stringTest=long is used:

screen shot 2018-07-25 at 1 53 43 pm

Default for contrast:

screen shot 2018-07-25 at 1 53 23 pm

This does not, however, occur with stringTest=double, so I don't know if we care about this... Are there languages where two words can be about 50 characters long?

For phetsims/qa/issues/153.

@ghost ghost added the type:i18n label Jul 25, 2018
@ghost ghost assigned Denz1994 Jul 25, 2018
@arouinfar
Copy link

Good catch @lmulhall-phet. I think the purpose of ?stringTest=long is to reveal layout issues when strings are pushed to/beyond their limit, and this certainly looks like a layout issue to me.

@ghost
Copy link
Author

ghost commented Jul 25, 2018

Ah, I see. That makes much more sense. Thanks, @arouinfar.

@ghost
Copy link
Author

ghost commented Jul 25, 2018

Also, on the Energy screen, a slider thumb can overlap with a string:

image

@Denz1994
Copy link
Contributor

Denz1994 commented Aug 1, 2018

@arouinfar Can you review master? I added some alignBBox/alignGroup refactors.

@arouinfar
Copy link

@Denz1994 the strings and icons now remained aligned under i18n, which is great.

However, it looks like the Displacement and Natural Length icons are missing from the Lab screen, but they're correctly appearing on the Energy screen.

image

image

It also looks like #305 (comment) hasn't been addressed.

@Denz1994
Copy link
Contributor

Denz1994 commented Aug 1, 2018

You are right @arouinfar, what about master now?

@arouinfar
Copy link

Looks good, thanks @Denz1994!

@KatieWoe
Copy link
Contributor

KatieWoe commented Aug 8, 2018

This issue seems to be appearing in RC 3 phetsims/qa/issues/165
rc3

@KatieWoe KatieWoe reopened this Aug 8, 2018
@KatieWoe
Copy link
Contributor

KatieWoe commented Aug 8, 2018

Taking a second look at this, I'm no longer sure. It looks better than it used to, but still seems a bit misaligned to me.

@arouinfar
Copy link

arouinfar commented Aug 8, 2018

@KatieWoe I think the Displacement and Natural Length icons are grouped as a single icon, which is then centered with respect to the checkbox.

image

It looks a bit misaligned for ?stringTest=long but less so for ?stringTest=double (which is more realistic for a sim translation).
image

@Denz1994 if the icons are indeed grouped, I think we can close this issue. Otherwise, please investigate.

@arouinfar arouinfar removed their assignment Aug 8, 2018
@Denz1994
Copy link
Contributor

Denz1994 commented Aug 8, 2018

The icons are grouped with their labels respectively. The slight difference here is one, we have a bracket included in the label group so it isn't as uniform as the other three labels. Two, the arrow isn't the same size as the line symbols in height so there is a bit more variability in the symbol size as well.

I am going to agree with closing this issue on the basis that the symbols for the bracketed options are centered with respect to the symbols' sizes and the bracket group.

@Denz1994 Denz1994 closed this as completed Aug 8, 2018
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