Closed Bug 324866 Opened 19 years ago Closed 19 years ago

Tree Widgets Are Not Painting Correctly (Fx and Tb)

Categories

(Core :: Layout, defect)

x86
Windows XP
defect
Not set
critical

Tracking

()

RESOLVED FIXED

People

(Reporter: mscott, Assigned: roc)

References

Details

(Keywords: regression)

Attachments

(2 files)

Regression that happened sometime yesterday. In the 01/26 builds, the thunderbird tree widgets are no longer painting there contents correctly. Could it be related to Bug 317375 which did touch the tree frame code yesterday? Screen shot coming up
Blocks: 317375
duplicate of bug 324836 (which has just been fixed on trunk)?
hmm, no, probably not. What's the simplest place this appears? anywhere in Firefox? Guess I'll spin up a Thunderbird build
I just updated layout\generic\nsFrame.cpp and rebuilt layout, but the problem appears to still be there. Shucks.
This might just be noise, but if I drag the columns somewhere else (like over to the left near the subject, they start painting fine. But anything after the sender column fails to paint.
Hey Roc, I can reproduce this in todays Firefox build too. Just got to the bookmark manager. You might need to add some extra tree columns to the bookmarks tree. I see lots of blank columns on the right most tree cols. And I just noticed that if I move the vertical splitter to the left and to the right in this window, I see some of the items get painted. I tried this in Thunderbird and saw the same thing. Resizing the splitter, triggered some level of painting to happen on these tree columns. I hope that helps.
Blocks: 317375
Its even easier now. Just take the bookmarks manager as is, and move the splitter to the right, you should see the bookmarks tree stop getting painted.
adjusting the summary since this applies to Firefox and Thunderbird. And moving the component.
Component: Mail Window Front End → Layout
Product: Thunderbird → Core
Summary: Thunderbird's Tree Widgets Are Not Painting Correctly → Tree Widgets Are Not Painting Correctly (Fx and Tb)
(In reply to comment #5) > This might just be noise, but if I drag the columns somewhere else (like over > to the left near the subject, they start painting fine. But anything after the > sender column fails to paint. I experienced this behaviour two years ago in AddressBook - bug 251079. It looks like you summoned it back ;)
No longer blocks: 317375
Component: Layout → Mail Window Front End
Product: Core → Thunderbird
Summary: Tree Widgets Are Not Painting Correctly (Fx and Tb) → Thunderbird's Tree Widgets Are Not Painting Correctly
Attached patch fix (deleted) — Splinter Review
This fixes it for me. OffsetForHorzScroll expects cellrect to be in frame coordinates. We need to adjust the x coordinate to rendering coordinates after OffsetForHorzScroll but before PaintCell. This is now consistent with the code around the PaintCell call earlier in the code.
Assignee: mscott → roc
Status: NEW → ASSIGNED
Attachment #209766 - Flags: superreview?(dbaron)
Attachment #209766 - Flags: review?(dbaron)
restoring fields lost due to bug stompage.
Blocks: 317375
Component: Mail Window Front End → Layout
Flags: review?(dbaron)
Keywords: regression
Product: Thunderbird → Core
Summary: Thunderbird's Tree Widgets Are Not Painting Correctly → Tree Widgets Are Not Painting Correctly (Fx and Tb)
This patch does indeed fix the problem for me in thunderbird. I didn't test it in Firefox. Thanks roc!
Attachment #209766 - Flags: superreview?(dbaron) → superreview+
Attachment #209766 - Flags: review+
*** Bug 324873 has been marked as a duplicate of this bug. ***
checked in.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
*** Bug 324881 has been marked as a duplicate of this bug. ***
*** Bug 324905 has been marked as a duplicate of this bug. ***
*** Bug 324930 has been marked as a duplicate of this bug. ***
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: