Closed
Bug 45597
Opened 24 years ago
Closed 19 years ago
Scrollbars are appearing when not needed when clipped content is present
Categories
(Core :: Layout: Positioned, defect, P2)
Core
Layout: Positioned
Tracking
()
RESOLVED
INVALID
Future
People
(Reporter: magnus, Unassigned)
References
()
Details
(Keywords: perf, testcase, top100, Whiteboard: [nsbeta3-])
Attachments
(2 files)
From Bugzilla Helper:
User-Agent: Mozilla/4.0 (compatible; MSIE 5.5; Windows NT 5.0)
BuildID: 2000071520
Mozilla is slow when moving nested DIV's - only when the browsers scrollbars
are visible.
Reproducible: Always
Steps to Reproduce:
1. Create an nested DIV with a lot of content clipping it so only a part of it
is visible.
2. Use setTimeout to move the nested divs top-position.
3. Change content to something smaller.
Actual Results: Less content - faster performance.
Comment 2•24 years ago
|
||
Reassigning to layout, there's unneccessary scrollbars on the page and it looks
like the reason for those showing up is that the content outside the clipped
rect affects the size of the scroll view, thus as the content is moved up, we
recalculate and reflow too much...
As long as the scrollbars are'n shown we're doing pretty good...
Assignee: jst → clayton
Status: UNCONFIRMED → NEW
Component: DOM Level 2 → Layout
Ever confirmed: true
QA Contact: vidur → petersen
Comment 3•24 years ago
|
||
I'm changing this to reflect the problem of a scrollbar being shown when
absolutely positioned content is actually clipped and does not require a
scrollbar.
My (naive) guess is that nsGfxScrollFrameInner::GetScrolledSize needs to account
for the clipped size of the frame's view, not the bounds, but these absolute
positioning problems are always more complicated than I originally assume.
Over to evaughan for further analysis.
Assignee: clayton → evaughan
Summary: Poor performance when moving nested DIV → [ABS POS] Scrollbars are appearing when not needed when clipped content is present
Comment 4•24 years ago
|
||
Hmm. In current builds, that testcase crashes, and it's quite slow whether
scrollbars are showing or not. Has something changed?
Marc/Johnny, could one of you have a look. Eric is away on vacation for another
couple of weeks.
(Nominating nsbeta3; we probably need to fix this html crash at minimum).
Comment 5•24 years ago
|
||
It didn't crash for me (my build is from Monday, WinNT) but I am pulling fresh
source now...
Comment 7•24 years ago
|
||
I was getting the crash when I mouseover the anchor in the top left corner
(the one that initiates scrolling) -- usually the second (sometimes third)
time I did the mouseover.
Comment 8•24 years ago
|
||
I'm not getting this crash on either NT 4 or Linux on today's build. Perhaps it
went away or is this a Windows 2000 only crash?
Also, on the performance issue, I noticed that once we are scrolling, a whopping
100% of CPU time (as reported by the task manager) is being used. When I tried
this in IE 5, barley 1% of the CPU was being used and the scrolling was MUCH
faster (and no scroll bars). I think I've seen this bug before though. It
reminds me of the PR2 default home page slowness.
There is also quite a bit of lag from the test site. Attaching a copy of this
testcase to bugzilla.
Comment 9•24 years ago
|
||
Comment 10•24 years ago
|
||
Please file separate bugs for other defects, let's not morph this bug report to
cover performance issues.
marking as assigned, but if nobody is seeing it, please mark WFM.
Status: NEW → ASSIGNED
Comment 11•24 years ago
|
||
reproduced crash in d&d timer code after jumping through many hoops. Seems very
obscure, wouldn't hold N6 for this, or for an errant scrollbar.
nsbeta3-/future
Target Milestone: M18 → Future
Comment 13•24 years ago
|
||
Upon managerial request, adding the "testcase" keyword to 84 open layout bugs that
do not have the "testcase" keyword and yet have an attachement with the word
"test" in the description field. Apologies for any mistakes.
Keywords: testcase
Updated•24 years ago
|
Target Milestone: Future → mozilla0.9
Comment 15•23 years ago
|
||
build 2001112803 win32
performance in the testcase is beautiful.
not sure about the scrollbar, is it really not suppost to be there???
Comment 16•23 years ago
|
||
Bugs targeted at mozilla1.0 without the mozilla1.0 keyword moved to mozilla1.0.1
(you can query for this string to delete spam or retrieve the list of bugs I've
moved)
Target Milestone: mozilla1.0 → mozilla1.0.1
Comment 17•23 years ago
|
||
Peter Belesis (author of the famous HierMenus from webreference.com ) is
describing some problems in the latest release regarding this.
just see: http://www.webreference.com/dhtml/column62/6.html
Comment 18•23 years ago
|
||
*** Bug 140892 has been marked as a duplicate of this bug. ***
Comment 19•23 years ago
|
||
*** Bug 141264 has been marked as a duplicate of this bug. ***
Comment 20•22 years ago
|
||
*** Bug 166696 has been marked as a duplicate of this bug. ***
Comment 21•22 years ago
|
||
*** Bug 167647 has been marked as a duplicate of this bug. ***
Updated•22 years ago
|
Keywords: mozilla1.3
Comment 22•22 years ago
|
||
By the definitions on <http://bugzilla.mozilla.org/bug_status.html#severity> and
<http://bugzilla.mozilla.org/enter_bug.cgi?format=guided>, crashing and dataloss
bugs are of critical or possibly higher severity. Only changing open bugs to
minimize unnecessary spam. Keywords to trigger this would be crash, topcrash,
topcrash+, zt4newcrash, dataloss.
Severity: normal → critical
Updated•22 years ago
|
Component: Layout → Layout: R & A Pos
Keywords: mozilla1.3
Summary: [ABS POS] Scrollbars are appearing when not needed when clipped content is present → Scrollbars are appearing when not needed when clipped content is present
Updated•22 years ago
|
OS: Windows 2000 → All
Hardware: PC → All
Comment 23•22 years ago
|
||
*** Bug 155178 has been marked as a duplicate of this bug. ***
Comment 24•22 years ago
|
||
*** Bug 135503 has been marked as a duplicate of this bug. ***
Comment 25•22 years ago
|
||
.
Assignee: eric → position
Severity: critical → major
Status: ASSIGNED → NEW
Priority: P3 → --
QA Contact: cpetersen0953 → ian
Target Milestone: mozilla1.0.1 → ---
Comment 26•22 years ago
|
||
*** Bug 203598 has been marked as a duplicate of this bug. ***
Comment 27•22 years ago
|
||
*** Bug 203598 has been marked as a duplicate of this bug. ***
Comment 28•22 years ago
|
||
Guys,
Whole point of my findings in BUG #203598 is this is a problem only with
absolutely positioned elements. EVERYTHING works fine if child is relatively
positioned!
Pls. look at my tests for proof.
Updated•21 years ago
|
Priority: -- → P2
Updated•21 years ago
|
Target Milestone: --- → Future
Comment 30•21 years ago
|
||
Many DHTM librarys (scrollers & the very famous HierMenus) are encountering
this problem. We also do extra painting hence facing performance problems when
the scrollbars are appearing.
So, having this attacked in the 1.5 timeline would be really a fine thing.
Comment 31•21 years ago
|
||
Removing crash keyword, since the testcase does not crash, and if it would then
a separate bug would be better.
Keywords: crash
Comment 32•19 years ago
|
||
Will this ever get fixed?
needs a minimized testcase
Comment 34•19 years ago
|
||
This is reduced from the previous testcase.
The only reason why IE is not showing a vertical scrollbar here is because it
supports scroll="no" on the <body>. Otherwise, it would do the same as Mozilla.
And afaics, this is correct behavior per spec:
http://www.w3.org/TR/CSS21/visufx.html#clipping
I agree. Clipping only affects rendering, it doesn't affect layout. If you want
to affect layout, use overflow:hidden.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•