Opened 4 years ago

Closed 2 years ago

#16365 closed bug (no change required)

Create B_DEFAULT_FONT_SIZE or some other such constant

Reported by: waddlesplash Owned by: nobody
Priority: normal Milestone: Unscheduled
Component: Kits/Interface Kit Version: R1/beta2
Keywords: hidpi Cc:
Blocked By: Blocking:
Platform: All

Description

A good amount of the scaling code is independent of font size entirely, and computes margins purely based on what it is, e.g. (font size / 2) for 6px margins. However, some code requires an actual "scaling factor", and this code presently hardcodes 12.0f for this purpose. Instead a constant should be created, e.g. B_DEFAULT_FONT_SIZE, as suggested by X512 in #13170.

Change History (1)

comment:1 by waddlesplash, 2 years ago

Resolution: no change required
Status: newclosed

I don't think we need to do this after all. More and more places in the code are being changed to use BControlLook spacing calculations instead of "font size / 12", and if we really did need it, we should instead add the global scale factor elsewhere.

Note: See TracTickets for help on using tickets.