jlennox / HeartRate

Bluetooth heart rate monitor
MIT License
109 stars 35 forks source link

Font Size #19

Closed TheLoneUs closed 3 years ago

TheLoneUs commented 3 years ago

When setting the font size directly to something like "128" it is reduced to "96" - it appears to be a percentage reduction and it's really hard to get the font size I'm trying to set.

Appears to happen for all font sizes selected via the GUI.

jlennox commented 3 years ago

@TheLoneUs

Ah, I see what's happening. It's converting my pixel sizes to point's for the dialog. fix-font-size-test1.zip

Let me know if this fixes your issue.

I'm also curious why you're using manual font sizes instead of letting it scale with the window?

Good eye, btw. 1 pixel = .75 points, so 128 becomes 96. Thanks for the report!

TheLoneUs commented 3 years ago

@jlennox I am using the specific font size for consistencies sake while streaming. I figured if I set it explicitly it will be harder to accidentally mess it up, and I can set it to the size I want without resizing in OBS. My hope there is that with no resizing in OBS it will be better quality on stream.

I tested your font size update and it appears to work perfectly!

jlennox commented 3 years ago

Thanks for testing and for the information! I'm glad to know the use cases. If anything else comes up, open a new issue and I'll take a look.