W3C home > Mailing lists > Public > public-css-testsuite@w3.org > February 2012

Re: [RC6] line-height-002 to line-height-104 have various issues

From: Gérard Talbot <css21testsuite@gtalbot.org>
Date: Wed, 22 Feb 2012 18:41:43 -0500
Message-ID: <99aa9f9599c64565d73108f1c1351683.squirrel@ed-sh-cp3.entirelydigital.com>
To: "Arron Eicholz" <arron.eicholz@microsoft.com>
Cc: "Public CSS testsuite mailing list" <public-css-testsuite@w3.org>

Le Mer 22 février 2012 17:23, "Gérard Talbot" a écrit :
> Hello,
>
> I see several issues with the following tests:
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-002.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-003.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-004.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-005.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-013.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-014.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-015.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-016.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-024.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-025.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-026.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-027.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-035.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-036.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-037.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-038.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-046.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-047.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-048.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-049.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-057.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-059.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-060.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-068.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-070.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-071.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-079.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-080.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-081.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-082.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-090.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-091.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-092.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-093.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-101.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-103.htm
>
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-104.htm
>
>
>
>
> If a set line-height is smaller than font-size, then glyphs bleed out of
> the line box and then content of contiguous line boxes overlap.
>
> Eg.
> http://www.gtalbot.org/BrowserBugsSection/css21testsuite/font-size-vs-smaller-line-height.html
>
> Section 10.8 is not clear on this, definitely not sufficiently clear on
> this.
>
> The issues I see with the listed tests are
>
> 1-
> The overlapping box (abs. pos. #div3) increases line box; eg.
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-080.htm
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-102.htm
>
> The added black area we can see around the first "X" of #div2 is given
> by #div2, #div3
>             {
>                 background: black;
>             }
>
> which will affect content area.

I got that wrong; it's the 2nd "X" that creates this.

>
> When line-height is taller than font-size, then background set to black
> on #div2 makes sense as this will paint content area. eg
> http://test.csswg.org/suites/css2.1/20110323/html4/line-height-006.htm
>
> 2-
> The 2nd "X" in #div2 is always useless, irrelevant in all listed tests
> and can be safely removed. It also has been removed too in a few tests.

I got that wrong; the 2nd "X" appears at the bottom of the first black 
box. Since #div2 is only 1em wide, there is a linewrap occuring. I
missed totally that.

Gérard
-- 
Contributions to the CSS 2.1 test suite:
http://www.gtalbot.org/BrowserBugsSection/css21testsuite/

CSS 2.1 Test suite RC6, March 23rd 2011:
http://test.csswg.org/suites/css2.1/20110323/html4/toc.html

CSS 2.1 test suite harness:
http://test.csswg.org/harness/

Contributing to to CSS 2.1 test suite:
http://www.gtalbot.org/BrowserBugsSection/css21testsuite/web-authors-contributions-css21-testsuite.html
Received on Wednesday, 22 February 2012 23:42:15 GMT

This archive was generated by hypermail 2.2.0+W3C-0.50 : Wednesday, 22 February 2012 23:42:21 GMT