Last Comment Bug 372498 - [reflow branch] Right padding is missing in the scroll view
: [reflow branch] Right padding is missing in the scroll view
Status: NEW
: dev-doc-needed, regression, testcase
Product: Core
Classification: Components
Component: Layout: Block and Inline (show other bugs)
: Trunk
: x86 Linux
-- normal with 1 vote (vote)
: ---
Assigned To: Nobody; OK to take it and work on it
: Jet Villegas (:jet)
: 446601 (view as bug list)
Depends on: 665597
Blocks: reflow-refactor
  Show dependency treegraph
Reported: 2007-03-03 09:46 PST by Mats Palmgren (:mats)
Modified: 2013-07-18 14:10 PDT (History)
12 users (show)
roc: wanted1.9.1+
mbeltzner: blocking1.9.0.1-
samuel.sidler+old: wanted1.9.0.x+
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Testcase #1 (792 bytes, text/html)
2007-03-03 09:46 PST, Mats Palmgren (:mats)
no flags Details
Testcase #2 (1.12 KB, text/html)
2011-12-14 09:56 PST, Mats Palmgren (:mats)
no flags Details

Description User image Mats Palmgren (:mats) 2007-03-03 09:46:49 PST
Created attachment 257150 [details]
Testcase #1

Right padding is missing in the scroll view.

1. load the attached testcase

In the scroll view there is a yellow box surrounded by 30px padding (white)
on all sides except on the right side where it is missing.

In the scroll view there is a yellow box surrounded by 30px padding (white)
on all sides.

Bug occurs in a SeaMonkey 20070303 nightly build on Linux
Bug does not occur in SeaMonkey 2006120701 (pre reflow branch) on Linux.
Bug occurs in SeaMonkey 2006120801 (post reflow branch) on Linux.
Comment 1 User image Eli Friedman 2007-06-18 16:53:39 PDT
Our behavior in terms of the right padding actually agrees with Safari.  The justification is that the width of the child block is greater than the width of the parent, so it overflows, covering the right padding.

Our behavior is arguably wrong in terms of the bottom padding, though.  The child block should also overlap the bottom padding.  We aren't calulating the computed height of the scrolled frame correctly, at least according to the Safari model.  And I think the Safari model is the only one that can really make sense in the long run.
Comment 3 User image Samuel Sidler (old account; do not CC) 2008-08-17 20:56:57 PDT
Since this is a regression from the reflow branch, let's try and get this into 1.9.0.x/1.9.1.
Comment 4 User image Daniel.S 2008-09-28 06:43:18 PDT
*** Bug 446601 has been marked as a duplicate of this bug. ***
Comment 5 User image Daniel.S 2008-09-28 06:46:18 PDT
The testcase renders the same in Gecko 1.9.x, IE8b2 and Safari 3.1.2. Only Opera 9.5.2 differs and shows a white bottom padding and a twice as wide right padding.
Comment 6 User image Zack Weinberg (:zwol) 2010-08-18 09:42:00 PDT
I no longer work for Mozilla, I am deassigning myself from bugs I have no intention of working on as a volunteer.
Comment 7 User image Mats Palmgren (:mats) 2011-12-14 09:56:08 PST
Created attachment 581691 [details]
Testcase #2
Comment 8 User image Mats Palmgren (:mats) 2011-12-14 09:59:57 PST
With the patches in bug 665597, the scroll box accounts for margin/padding on the
right and bottom sides (so that yellow/blue boxes have an equal gap on all sides)
Comment 9 User image Mats Palmgren (:mats) 2012-01-03 13:24:45 PST
The original issue is invalid, and as Eli says in comment 1 we should also
overflow the bottom padding.  Bug 665597 will fix that, and also the margin case.

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