Closed Bug 358398 Opened 18 years ago Closed 18 years ago

Horizontally scrolled trees incorrectly clip text

Categories

(Core :: XUL, defect)

x86
Linux
defect
Not set
major

Tracking

()

RESOLVED FIXED

People

(Reporter: bzbarsky, Assigned: neil)

References

()

Details

Attachments

(1 file, 1 obsolete file)

STEPS TO REPRODUCE: 1) Load https://bugzilla.mozilla.org/attachment.cgi?id=243782&action=view 2) Scroll the tree to the right EXPECTED RESULTS: Text appears in all columns ACTUAL RESULTS: Text does not appear in columns that were not originally visible.
Regression range is from 2006-02-13-06 to 2006-02-14-08. Checkins: http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=MozillaTinderboxAll&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2006-02-13+06&maxdate=2006-02-14+08&cvsroot=%2Fcvsroot This regressed due to the patch in bug 306990 (tested via putting back the early return if (aCol->GetNext()) -- that fixed all the columns except the last one.
Blocks: 306990
Flags: blocking1.9?
Attached patch First draft (obsolete) (deleted) — Splinter Review
This is just a proof of concept review. CalcColumnRect looks as if it should be inlined. I also need to figure out how clipping works.
Assignee: Jan.Varga → neil
Status: NEW → ASSIGNED
Attachment #243832 - Flags: superreview?(roc)
Attachment #243832 - Flags: review?(enndeakin)
Attached patch Proposed patch (deleted) — Splinter Review
Attachment #243832 - Attachment is obsolete: true
Attachment #243853 - Flags: superreview?(roc)
Attachment #243853 - Flags: review?(enndeakin)
Attachment #243832 - Flags: superreview?(roc)
Attachment #243832 - Flags: review?(enndeakin)
Flags: in-testsuite?
Attachment #243853 - Flags: review?(enndeakin) → review+
Attachment #243853 - Flags: superreview?(roc) → superreview+
Fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Depends on: 359649
Flags: blocking1.9?
Component: XP Toolkit/Widgets: Trees → XUL
QA Contact: xptoolkit.trees → xptoolkit.widgets
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: