-
Notifications
You must be signed in to change notification settings - Fork 820
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
Fix spacing for 100M+ XP #7688
Fix spacing for 100M+ XP #7688
Conversation
How do we determine spacing in the Info Panel? Maybe it would be more maintainable to use the same method. I mean this for all of the Character panel values as well. The current method feels quite "hardcodey". |
I guess we could render the text to some internal buffer using smaller and smaller spacing until it fits. The alternative is to stress-test stuff by hand using zeroes and see what the breakpoints are.It's just XP and DMG in the char panel; the others fit just fine. |
We just pass |
Do we still need this PR with the new one being open? |
There is a new issue, but no new PR. |
Sorry, I mistook that for a PR 😂 |
@vlolteanu feel free to open a new PR that implements this suggestion |
I think @kphoenix137 has a fix ready, just needs to finish his PR that fixes kerning fit when centered |
Depending on how "fat" the digits happen to be, regular spacing in the char panel might be too wide at 100M XP. Notice how the "1" is really awkwardly placed. Something similar happens at 1B.
Before:
After: