Unable to unset minimum font size
Reported by
chrisage...@gmail.com,
Apr 29 2016
|
||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_4) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/50.0.2661.94 Safari/537.36 Example URL: https://jsfiddle.net/chrisg/Lsr19w2q/ Steps to reproduce the problem: 1. Make a new Chrome Profile 2. Go to URL: https://jsfiddle.net/chrisg/Lsr19w2q/ 3. Notice everything is rendered properly and with the correct calculations. 4. Go to chrome://settings and adjust the minimum font size - any number will do. 5. Revisit or refresh the page: https://jsfiddle.net/chrisg/Lsr19w2q/ 6. Notice that the font-size for the text is calculated properly in em, but every other attribute is using the minimum font-size. What is the expected behavior? The set font-size is respected for all styled elements despite the minimum font-size set. What went wrong? Calculations for styling are inconsistent. Does it occur on multiple sites: N/A Is it a problem with a plugin? N/A Did this work before? N/A Does this work in other browsers? N/A Chrome version: 50.0.2661.94 Channel: stable OS Version: OS X 10.11.4 Flash Version: Shockwave Flash 21.0 r0 There's also no way to unset the minimum font-size - currently this is affecting customers on our platform, where the only resolution is to make a new platform.
,
May 2 2016
I'm afraid I don't follow, what do you consider to be the problem here? The test case doesn't illustrate it very well and from your description it sounds like it is working as expected. I.e. text is not allowed to be smaller than the minimum font size.
,
May 2 2016
Sorry if I was unclear. The issue is two fold: 1. There is no way to "unset" a minimum-font size, meaning that if I set it once, the lowest I can ever go to again is "6px" - not completely removing the option. 2. The minimum font-size is ignored for the text, but not for the padding, margin, etc. Hopefully this video will illustrated it a little better: https://db.tt/KnuWqHsU Notice how at the beginning, on a new profile, I have no minimum font-size set. Once I set it to "7px" the text-size uses the website's set font-size (preferred), but the padding and other elements using "em" use the set minimum font-size in Chrome's preferences. Afterwards, there is no way to return to the original state where there is no minimum font-size set.
,
May 2 2016
Thank you for providing more feedback. Adding requester "eae@chromium.org" for another review and adding "Needs-Review" label for tracking. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
May 2 2016
None of the font sizes in the test ever go below the minimum font size. No matter what minimum font size I set I cannot get your test to ever resolve any to a value smaller to it. This is as expected. As for there being no way to unset the minimum font size that is a chrome settings ui bug.
,
May 2 2016
Sorry, what I'm really trying to point out is: the width, outline, etc all in EM are sized if someone sets the minimum font size (as you mentioned as expected) but the inconsistency is that the font-size for the text "TEST" using the font-size of that on the page, 0.55px. 0.55px * 40em == 22px, which is what the font-size is before AND after changing the minimum font-size. If I set my minimum font-size to 10px then 40em should be 400px for the font-size. Unless I misunderstand how it's supposed to work, I expect that "minimum font-size" would also apply to the text.
,
May 2 2016
The minimum font size is applied after style resolution and is indeed applied to the text. It only comes into play when the minium is _higher_ than the resolved size. If you set the minimum to 24px you'll see that the text "TEST" is bumped up to 24.
,
May 13 2016
,
May 15 2016
can we add a way to unset "webkit.webprefs.minimum_font_size" in md-settings?
,
May 16 2016
Forwarding. This is out of my purview
,
May 24 2016
,
Jul 7 2016
,
Oct 21 2016
On some sites, the minimum font size doesn't override site styling. e.g. with minimum font size set at 18, the text is https://vuejs.org/guide/ is still too small Should I make a separate report for this?
,
Oct 21 2016
Re #13: Yes please.
,
Mar 13 2017
Cleaning up "Needs-Review" label as we are not using this label for triage anymore. Ref bug for this cleanup 684919 |
||||||||||
►
Sign in to add a comment |
||||||||||
Comment 1 by patricia...@chromium.org
, May 2 2016Labels: OS-Chrome OS-Linux OS-Windows