1787523 - [Colorway Closet] Further tweak initial width to deal with font size changes

Status: RESOLVED FIXED

(Firefox :: Theme, defect, P1)

Description

[Dão Gottwald [:dao]]

Bug 1786858 made the modal's font size system-based, which is good but also means it's opened at a smaller size than before, and the degree differs across OSes. This is sort of fine in theory, as a smaller font size means we should need less space overall. But we're now way smaller than what we aimed for it seems. I'm going to propose 67em which should amount to just over 800px on Windows where the default font size is 12px. It's going to be a bit less than 800px on macOS and slightly above on Linux. If we still run into problems with that, we may also have to adjust breakpoint values (which are px based) or consider using 800px instead of an em value for the initial width.

Comment 1

[Dão Gottwald [:dao]]

Attachment: Bug 1787523 - [Colorway Closet] Further tweak initial width to deal with font size changes. r=amy

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9a345de38916
[Colorway Closet] Further tweak initial width to deal with font size changes. r=amy

Comment 3

[Marian-Vasile Laza]

https://hg.mozilla.org/mozilla-central/rev/9a345de38916

Comment 4

[Marian-Vasile Laza]

https://hg.mozilla.org/mozilla-central/rev/9a345de38916