Closed Bug 23521 Opened 25 years ago Closed 25 years ago

Scrolling thread pane in mailnews causes whole window to reflow

Categories

(Core :: XUL, defect, P1)

x86
Other
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: scottputterman, Assigned: eric)

References

Details

(Keywords: perf, Whiteboard: [PDT+] 2/4/00)

Quoting from an email from hyatt:

"The mailnews window had slower tree scrolling because it was reflowing the
whole window whenever you scrolled the thread pane.  I have managed to make it
so that - if you don't touch the splitter - this doesn't happen in the default
case.  Once you touch the splitter, however, things get wacky.  This is
definitely a bug in boxes that Eric will need to look at."

cc'ing hyatt in case more info is needed.
David, is this the same bug we talked about on Friday where the headers changing
size wouldn't change the size of the message pane?  Or should I file a separate
bug on that?
Severity: normal → critical
Priority: P3 → P1
If this is the cause of the problem, then this is pretty big.  I tried moving
the splitter and from then on reading mail becomes really slow. Every time I try
to do something the cpu goes to 100% for a while.  It makes it unuseable.
It looks like the calculated size gets off once the splitter is moved....
Summary: Scrolling thread pane in mailnews causes whole window to reflow → [DOGFOOD]Scrolling thread pane in mailnews causes whole window to reflow
Blocks: 9161
Keywords: perf
Whiteboard: [PDT+]
GFX scrollbars have to improve as a top priority.
Hey Eric, have you had a chance to look at this yet?
Target Milestone: M13
targetting M13
Status: NEW → ASSIGNED
Pulling a new build and taking a look
Whiteboard: [PDT+] → [PDT+] Trying to get a build that doesn't crash in mail
Target Milestone: M13 → M14
moving to m14 since there is a workaround - don't touch the splitter. This
should not be critical severity, since there is no crash, data loss or severe
leak.
Whiteboard: [PDT+] Trying to get a build that doesn't crash in mail → [PDT+] 1/28/00
Blocks: 25824
Whiteboard: [PDT+] 1/28/00 → [PDT+] 2/4/00 currently working on this
No longer blocks: 25824
Putting dogfood in the keyword field.
Keywords: dogfood
Whiteboard: [PDT+] 2/4/00 currently working on this → [PDT+] 2/4/00 fixed awaiting checking
Summary: [DOGFOOD]Scrolling thread pane in mailnews causes whole window to reflow → Scrolling thread pane in mailnews causes whole window to reflow
Blocks: 25824
Fixed!
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Whiteboard: [PDT+] 2/4/00 fixed awaiting checking → [PDT+] 2/4/00
Whiteboard: [PDT+] 2/4/00 → [PDT+] 2/4/00 Asked lchiang for help verifying
QA Contact: paulmac → lchiang
Whiteboard: [PDT+] 2/4/00 Asked lchiang for help verifying → [PDT+] 2/4/00
This seems to be ok now.  I moved the splitter around a lot. Selected lots of 
messages.  Move the splitter some more.  And I didn't slow down.  thanks.  
Marking Verified.
Status: RESOLVED → VERIFIED
No longer blocks: 25824
You need to log in before you can comment on or make changes to this bug.