Fix the scaling units to make better sense.

Review Request #13827 — Created May 6, 2024 and submitted

Information

Ink
master

Reviewers

The scaling units were set somewhat early in development to a base of
0.8em, which was a bit on the small size. The m (medium) size was
almost always too small, requiring use of the in-between ml
(medium-large) size. Similarly, l (large) was never large enough,
requiring xl or xl sizes. And those were still on the small size,
given their upper range on the scale.

The scale has now been redone to use 1em as the base, and to change
the l, xl, and xxl scales to better represent their names. This
gives us a slightly larger UI by default when using these units, which
works better on both touchscreens and modern desktop displays.

Mostly, this doesn't impact things too much until you get to the ml
size and higher. Any UI components making use of these will need updates
to choose more correct sizes.

Ink itself isn't really impacted much by these changes. Alerts required
small changes to the layout to reduce the distance to actions.

Tested this with all the Ink components, and with Review Board (which
does require updates).

Summary ID
Fix the scaling units to make better sense.
The scaling units were set somewhat early in development to a base of 0.8em, which was a bit on the small size. The `m` (medium) size was almost always too small, requiring use of the in-between `ml` (medium-large) size. Similarly, `l` (large) was never large enough, requiring `xl` or `xl` sizes. And those were still on the small size, given their upper range on the scale. The scale has now been redone to use `1em` as the base, and to change the `l`, `xl`, and `xxl` scales to better represent their names. This gives us a slightly larger UI by default when using these units, which works better on both touchscreens and modern desktop displays. Mostly, this doesn't impact things too much until you get to the `ml` size and higher. Any UI components making use of these will need updates to choose more correct sizes. Ink itself isn't really impacted by these changes, though there is a change to the button. The button now uses a larger size for the horizontal spacing for a button, giving it a wider hitbox and making the button feel less crowded.
4a290700635bfdf4411ea9b7bb6488dd8bfe731e

chipx86
chipx86
david
  1. Ship It!
  2. 
      
maubin
  1. Ship It!
  2. 
      
chipx86
Review request changed
Status:
Completed
Change Summary:
Pushed to master (05539e1)