Consolidate font definitions and reduce font size overrides.

Review Request #13688 — Created April 1, 2024 and submitted

Information

Review Board
release-7.x

Reviewers

We now use Ink's font CSS variables and cease overriding or using
absolute font sizes wherever possible. This helps ensure consistency
throughout the UI.

Many of the font size overrides are there for legacy reasons, but aren't
really necessary anymore. In many of these cases we were just overriding
to what we expected the parent to use in order to override some style
that the selector picked up due to a precedence issue. We now just use
inherit where it's necessary, and remove the font-size where it's
not.

This doesn't mean that all font sizes are the same as what we had
before. The 9pt font sizes are a bit bigger than our default. However,
what we have now is more consistent, and further refinements can be made
with relative font sizes or variables.

There are some font families and styles still set in places, but they're
part of larger cleanup changes, and are being ignored for now.

Went through the affected parts of the UI and felt fine with the font
sizes. Made sure the font families were correct.

Summary ID
Consolidate font definitions and reduce font size overrides.
We now use Ink's font CSS variables and cease overriding or using absolute font sizes wherever possible. This helps ensure consistency throughout the UI. Many of the font size overrides are there for legacy reasons, but aren't really necessary anymore. In many of these cases we were just overriding to what we expected the parent to use in order to override some style that the selector picked up due to a precedence issue. We now just use `inherit` where it's necessary, and remove the `font-size` where it's not. This doesn't mean that all font sizes are teh same as what we had before. The `9pt` font sizes are a bit bigger than our default. However, what we have now is more consistent, and further refinements can be made with relative font sizes or variables. There are some font families and styles still set in places, but they're part of larger cleanup changes, and are being ignored for now.
d12cf863ab8e7063e57bca59fca819896076c70e
Description From Last Updated

Typo in description: "teh same"

daviddavid
david
  1. 
      
  2. Show all issues

    Typo in description: "teh same"

  3. 
      
chipx86
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-7.x (211be7b)
Loading...