1547496 - Changing minimum font size no longer affects pref UI (but still shows a warning)

Status: RESOLVED FIXED

(Firefox :: Settings UI, defect, P1)

Description

[Mike Kaply [:mkaply]]

This is a regression from https://bugzilla.mozilla.org/show_bug.cgi?id=1528712 based on mozregression.

Before bug 1528712, when you changed the Minimum font size in preference, it affects the fonts in preferences as well. We have a warning because of that:

"You have selected a very large minimum font size (more than 24 pixels). This may make it difficult or impossible to use some important configuration pages like this one."

After bug 1528712, fonts in preferences are not changed by the minimum font size pref.

I'm concerned this affects other default font sizes.

Either way, I don't believe this was an expected side effect and should be investigated.

Comment 1

[Emilio Cobos Álvarez (:emilio)]

That's actually a sort-of expected difference from that change, see:

https://phabricator.services.mozilla.com/D20141#inline-110236

The checks were inconsistent before that, but I'm happy to change them so all the minimum font-size checks apply to documents in chrome docshells (which won't include about:preferences), not chrome documents (which does).

Are we 100% sure that we want to make min-font-size apply to chrome docs that are loaded in type="content" docshells? If so I can send a patch.

Comment 2

[:Gijs (he/him)]

(In reply to Emilio Cobos Álvarez (:emilio) from comment #1)

Are we 100% sure that we want to make min-font-size apply to chrome docs that are loaded in type="content" docshells?

I defer to Dão on this but I don't think there's a great necessity for this. On Windows and Linux (less sure about macOS), I believe OS font size will influence the UI in this case (worth doublechecking). If Dão agrees with me, we should remove the warning.

Comment 3

[Dão Gottwald [:dao]]

(In reply to :Gijs (he/him) from comment #2)

(In reply to Emilio Cobos Álvarez (:emilio) from comment #1)

Are we 100% sure that we want to make min-font-size apply to chrome docs that are loaded in type="content" docshells?

I defer to Dão on this but I don't think there's a great necessity for this. On Windows and Linux (less sure about macOS), I believe OS font size will influence the UI in this case (worth doublechecking). If Dão agrees with me, we should remove the warning.

Yeah, I think that's right.

Comment 4

[:Gijs (he/him)]

I've confirmed that on win10, changing the system font size affects preferences in nightly.

I don't think this is crucial to fix for 67 so I'll just put up a fix for 68.

Comment 5

[:Gijs (he/him)]

Attachment: Bug 1547496 - remove warning for minimum font sizes being big as internal pages are no longer affected, r?mconley

This warning was added in https://bugzilla.mozilla.org/show_bug.cgi?id=1047586 .
As a result of https://bugzilla.mozilla.org/show_bug.cgi?id=1528712 , internal pages
are no longer affected by this change. Instead, they obey the OS font size
configuration, in which we cannot warn. So let's remove the now-inaccurate warning dialog.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7abd077c8294
remove warning for minimum font sizes being big as internal pages are no longer affected, r=mconley,flod

Comment 7

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/7abd077c8294