Last Comment Bug 625702 - Outline on (display:box) flexbox children is padded as if by a transparent border the same width
: Outline on (display:box) flexbox children is padded as if by a transparent bo...
Status: NEW
: regression, testcase
Product: Core
Classification: Components
Component: Layout (show other bugs)
: Trunk
: x86 All
: -- normal with 1 vote (vote)
: ---
Assigned To: Nobody; OK to take it and work on it
Depends on:
  Show dependency treegraph
Reported: 2011-01-14 02:38 PST by Oli Studholme
Modified: 2012-11-08 19:13 PST (History)
4 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Smaller saner testcase (308 bytes, text/html)
2011-01-14 06:41 PST, Boris Zbarsky [:bz]
no flags Details

Description Oli Studholme 2011-01-14 02:38:24 PST
User-Agent:       Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.0b8) Gecko/20100101 Firefox/4.0b8
Build Identifier: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.0b8) Gecko/20100101 Firefox/4.0b8

Refer to the third example on (“Outline on box element children is padded by the same width”) for more info.

Reproducible: Always

Actual Results:  
When adding a 1px outline to a box element’s child, the outline is separated from the child by the same amount, showing 1px of the background. The separation matches the width of the outline

Expected Results:  
The outline should be drawn immediately outside the element’s area, as it is in Firefox 3.6.10
Comment 1 Boris Zbarsky [:bz] 2011-01-14 06:37:41 PST
Oli, if this behavior changed since 3.6, would you be willing to find the regression range?

roc, it looks like we're including the block's outline somewhere in the box code when doing sizing....  Are we maybe looking at overflow areas somewhere?
Comment 2 Boris Zbarsky [:bz] 2011-01-14 06:41:33 PST
Created attachment 503830 [details]
Smaller saner testcase
Comment 3 Oli Studholme 2011-01-14 08:08:30 PST
Boris, I’m not actually sure how to find a regression range (or ahem what it is), and after filing 6 flexbox bugs today I’d like to pass if possible ;)

sorry for the non-minimal test case, I got lazy after the first couple and combined em.
Comment 4 Boris Zbarsky [:bz] 2011-01-14 09:44:31 PST
Oli, a regression range is basically "when the problem appeared".  Usually found by downloading nightly builds so as pin it down to a day.

The regression range here is in December 2009:

dbaron, you have some overflow changes in there....  This changeset looks suspicious:
Comment 5 Boris Zbarsky [:bz] 2011-01-14 09:47:53 PST
David, see end of comment 4.
Comment 6 Oli Studholme 2012-11-08 19:13:52 PST
Just FYI this is regarding OLD Flexbox (display: box;), not NEW Flexbox (display: flex;), and the chances of it being fixed are probably not great. For more background:

Note You need to log in before you can comment on or make changes to this bug.