W3C home > Mailing lists > Public > www-style@w3.org > March 2015

Re: [css-fonts] What does font-size-adjust for '0' look like?

From: ChangSeok Oh <changseok@gnome.org>
Date: Thu, 19 Mar 2015 16:01:56 +0900
Cc: W3C www-style mailing list <www-style@w3.org>
Message-Id: <2A5ADD14-A7EA-43C6-B84B-E5D1A5149EF4@gnome.org>
To: Gérard Talbot <www-style@gtalbot.org>
Thanks for your great test, Gérard!

ChangSeok

> On Mar 19, 2015, at 6:26 AM, Gérard Talbot <www-style@gtalbot.org> wrote:
> 
> Le 2015-03-18 04:22, ChangSeok Oh a écrit :
>> Hi Everyone.
>> I have a simple question on font-size-adjust. [1]
>> According to the spec. font-size-adjust can have ‘none’ or <number>
>> for its value.
>> BTW what is expected to look like where font-size-adjust is ‘0’? I
>> checked Gecko(which is the only vendor supporting for the property
>> now) treated it as ‘none’.
>> Does it make sense? As my reading, font-size-adjust represents an
>> aspect value (x-height / font-size) of a font.
>> So ‘0’ might mean x-height is 0 or equivalent very small value here.
>> Thus It should be same effect with 'font-size = 0px’
> 
> ChangSeok Oh,
> 
> Yes, it should be same effect with 'font-size = 0px’. I agree with your reasoning. The spec editor, John Daggett, should probably be notified of this.
> 0 would have to mean a 0px font-size.
> Negative values are illegal and ignored.
> Percentage values are ignored.
> 'auto' value must be ignored.
> We could create 4 additional tests in
> 
> http://test.csswg.org/suites/css3-fonts/nightly-unstable/html/chapter-3.htm#s3.6
> 
> testing those 4 case scenarios.
> 
>> I’d like to clarify this before landing a relevant patch for blink [2]
>> Best regards.
>> [1] http://dev.w3.org/csswg/css-fonts/#propdef-font-size-adjust
>> <http://dev.w3.org/csswg/css-fonts/#propdef-font-size-adjust>
>> [2] https://codereview.chromium.org/983073002/#msg23
>> <https://codereview.chromium.org/983073002/#msg23>
>> ChangSeok
> 
> ChangSeok,
> 
> I have added a comment in
> 
> http://www.gtalbot.org/BrowserBugsSection/CSS3Fonts/font-size-adjust-001.html
> 
> so that you can use my test, so that you can include it in your patch.
> 
> Gérard
> 
Received on Thursday, 19 March 2015 07:02:29 UTC

This archive was generated by hypermail 2.3.1 : Monday, 2 May 2016 14:39:30 UTC