Uploaded image for project: 'Fluid Infusion'
  1. Fluid Infusion
  2. FLUID-4703

UI Enhancer shouldn't use 'em' for line-height value


    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.4
    • Fix Version/s: None
    • Component/s: UIEnhancer
    • Labels:


      The UI Enhancer currently uses a unit of 'em' on the line-height setting it applies o the body element. I believe it shouldn't use a unit.

      When an absolute value is used for line-height, the computed line-height (based on the the font-size of the element where the line-height was specified) is inherited by all elements that don't have their own line-height set. In the case of UIO, this means that everything inherits the computed - and therefore fixed - line height based on the body font size. Because UI Enhancer is setting the line height this way, elements with larger fonts end up with too little line-spacing, requiring integrators to hard-code a line-height for those elements, which means that increasing the line-height with UIO doesn't affect those elements (see FLUID-4491).

      In contrast, when the line-height is a unitless number, it is treated as a scaling factor, and that factor is inherited. The factor will be used by each element to calculate the line height based on the element's own font size.

      If we fix this bug, integrators shouldn't have to set line-heights at all, so the incidence of FLUID-4491 would greatly decrease.




            • Assignee:
              anastasia Anastasia Cheetham
              anastasia Anastasia Cheetham
            • Votes:
              0 Vote for this issue
              0 Start watching this issue


              • Created: