-
Notifications
You must be signed in to change notification settings - Fork 263
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
Update Map and Token Labels to allow users to define how they look #4662
Conversation
It looks like there is extra padding on the top of the text boxes. |
Fonts are going to font :( |
Yeah, I see that the smaller the font it starts to drift up and increase the padding below. Seems like a difference between the padding and font sizes calcs. It looks like the fonts are using the point system... would using EM or true pixel size normalize things? Does running 4k exacerbate the issue? |
Cant use EM its not web browser based, It's just a case of asking the renderer how big the font is and then using that info to draw the box/centre. It is what it is, trying to do it another way will lead to problems on someones display. |
I like that a lot better, too. |
Identify the Bug or Feature request
fixes ##4660
Description of the Change
Token Labels
These are not the default colours, which are closer to the current ones, but I have changed them for these screenshots to make them more obvious. These are all set on the local client's side. (These are not the default colours I just needed something that stood out as being very different when testing)
<img width="257"src="https://github.com/RPTools/maptool/assets/73173/f1d72ad7-1dde-4e20-b2be-ce12cf152d0c">
Map Labels
Labels that are from pre 1.15 campaigns default to have the background colour (well one of them) from the fake 3D "bubble". These can not be changed client side due to the problems
I have also added comments to the majority of the files I touched that didn't have comments (adding comments to all the methods and fields the renderer and app preferences probably need an issue to address that so I don't completely annoy whom ever will review this change :)), hoping to set a trend with this but doubt it will :)
Possible Drawbacks
Some will miss the fake 3D bubble effect; I am sure someone out there prefers fake 3D bubbles over being able to read and configure the look.
Documentation Notes
See above
Release Notes
This change is