-
Notifications
You must be signed in to change notification settings - Fork 6
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
PhetFont text bounds have changed on macOS #785
Comments
@jonathanolson - Is there a way to adjust this so its correct for this font family? Or is there something fundamental off here, and the team should look for a different font family? Please comment. |
It seems likely we may want to try another font family. Likely the font is slightly shorter than the bounds, and has non-vertically-centered math operators. |
Can you investigate and try some font families to find an improvement? I think its best to address this asap so we don't end up publishing sims that are using this font family for mac. Question - is this font family specific to mac os? |
@jonathanolson @arouinfar @chrisklus @amanda-phet - I've read through the issue that motivated the font family change, and through the issues that the font family change caused (off center vertically, and issues with missing characters in other languages - my understanding is the original family didn't have those issues with the international languages.. is that right?). In my opinion, the original design issue (slightly off center behavior on long strings) is less of a concern than these new concerns. What do you think of restoring the original font family as the lesser of the problematic issues - at least for now? |
@jonathanolson - Looks like we have a consensus here from the designers. Can you revert the commit that changed the font family for macOS? |
Font reverted in the commit above. |
Couldn't test in number-play (NumberPlayGameLevelNode isn't specifying or allowing a tandem into the InfiniteStatusBar). |
Thanks @jonathanolson! I tested in Number Compare and things are looking great. (InfiniteStatusBar should have a default tandem option soon.) Should we close here or is there longer term work to do still? |
This issue has been in limbo for 8+ months, assigned to @kathy-phet. It's high priority and labeled as "blocks-sim-publication", guessing Number Compare. From the last comment, it looks like @chrisklus wanted @jonathanolson to close/comment but did not assign to him. I'll start by assigning to @chrisklus and @jonathanolson. Let's wrap this up. |
First reported in phetsims/joist#645 - changes to the font-family of PhetFont have caused text to be positioned differently.
Here are two screenshots to compare (best to open in separate tabs):
before:
after:
In phetsims/joist#645 (comment), @kathy-phet asked:
It is math symbols most noticeably, but numbers and regular text are a bit misaligned, too. The numbers in the total accordion boxes are a little low and are no longer vertically centered with the buttons. Also notice the comparison statement and sim title are a little lower than the original.
For this screen of Number Play, I think just the math symbols and total numbers are blocking, but given that general text is low too, I have to think there are plenty of pixel-perfect text alignments in many other sims that are now a little off in a noticeable way. Like text that is vertically aligned with a checkbox, for example, is going to look a little different. So i think ideally, the bounds should match what they were before for all cases.
Marking high priority and assigning to @kathy-phet and @jonathanolson.
The text was updated successfully, but these errors were encountered: