Closed
Bug 216030
Opened 22 years ago
Closed 21 years ago
InvalidateScrollbar() should be internal to nsTreeBodyFrame
Categories
(Core :: XUL, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: neil, Assigned: janv)
Details
Attachments
(1 file)
|
2.93 KB,
patch
|
janv
:
review+
rbs
:
superreview+
|
Details | Diff | Splinter Review |
The scrollbar only needs to be invalidated when:
* The tree body size changes
* The tree view changes
* The tree view row count changes
* The tree batch update ends
So it should be internal to nsTreeBodyFrame.
|
Reporter | |
Comment 1•22 years ago
|
||
|
Assignee | |
Comment 2•22 years ago
|
||
Comment on attachment 129731 [details] [diff] [review]
Proposed patch
this does make sense
Attachment #129731 -
Flags: review+
|
|
Reporter | |
Updated•21 years ago
|
Attachment #129731 -
Flags: superreview?(rbs)
Comment on attachment 129731 [details] [diff] [review]
Proposed patch
not a big deal either way.
sr=rbs
Attachment #129731 -
Flags: superreview?(rbs) → superreview+
|
|
Reporter | |
Comment 4•21 years ago
|
||
Fix checked in.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Component: XP Toolkit/Widgets: Trees → XUL
QA Contact: shrir → xptoolkit.widgets
You need to log in
before you can comment on or make changes to this bug.
Description
•