- From: Gérard Talbot <css21testsuite@gtalbot.org>
- Date: Tue, 14 Aug 2012 21:26:14 -0400
- To: "Arron Eicholz" <Arron.Eicholz@microsoft.com>
- Cc: "Public CSS test suite mailing list" <public-css-testsuite@w3.org>
Arron, [RC6] http://test.csswg.org/suites/css2.1/20110323/html4/margin-bottom-004.htm [nightly-unstable] http://test.csswg.org/suites/css2.1/nightly-unstable/html4/margin-bottom-004.htm I have made several small changes to margin-bottom-* tests. 1- All these tests were declaring 'height: 0' but this was unneeded, unnecessary. An empty block-level element will not generate a line box [1]; so, 'height: auto' - which is the default - was good enough. So, 'height: 0' was simply removed. [1]: "Line boxes are created as needed to hold inline-level content within an inline formatting context. Line boxes that contain no text, no preserved white space, no inline elements with non-zero margins, padding, or borders, and no other in-flow content (such as images, inline blocks or inline tables), and do not end with a preserved newline must be treated as zero-height line boxes" 9.4.2 Inline formatting contexts http://www.w3.org/TR/CSS21/visuren.html#inline-formatting 2- 0.2in was changed to 0.25in in tests margin-bottom-067, margin-bottom-068 and margin-bottom-113 to avoid fraction of a pixel. 3- [RC6] http://test.csswg.org/suites/css2.1/20110323/html4/margin-bottom-103.htm [nightly-unstable] http://test.csswg.org/suites/css2.1/nightly-unstable/html4/margin-bottom-103.htm I have given this test (and margin-bottom-104) a lot of thinking and I think this test is correct. 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, 15 August 2012 01:26:41 UTC