Closed
Bug 80844
Opened 23 years ago
Closed 23 years ago
Twisties for nested containers aren't indented
Categories
(Core :: XUL, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: bugzilla, Assigned: janv)
References
Details
Attachments
(4 files)
|
(deleted),
image/gif
|
Details | |
|
(deleted),
patch
|
Details | Diff | Splinter Review | |
|
(deleted),
patch
|
Details | Diff | Splinter Review | |
|
(deleted),
patch
|
Details | Diff | Splinter Review |
screenshot
All twisties are on the same level -- aligned to the left -- even if they're
for nested containers. They need to be indented like the cell text, so the
hierarchy is clearer. I'll attach a screenshot.
|
Reporter | |
Comment 1•23 years ago
|
||
|
Assignee | |
Comment 2•23 years ago
|
||
|
|
Assignee | |
Comment 3•23 years ago
|
||
Blake, could you try attached patch ?
|
|
Reporter | |
Comment 4•23 years ago
|
||
Jan, can you attach the complete patch? I added the |separator| atom to
nsXULAtomList.h but wasn't sure what the value of |mozoutlinerseparator| is
supposed to be. Also, you forgot the changes to nsOutlinerBodyFrame's header
file.
|
|
Reporter | |
Comment 5•23 years ago
|
||
hyatt, can you take a look at Jan's patch?
|
|
Assignee | |
Comment 7•23 years ago
|
||
|
|
Assignee | |
Updated•23 years ago
|
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 8•23 years ago
|
||
|
|
Reporter | |
Comment 9•23 years ago
|
||
Jan, can you give me a quick runthrough of this fix?
|
|
Assignee | |
Comment 10•23 years ago
|
||
I changed code back to update currX and remainingWidth according to indentation
before drawing twisty and the connecting lines.
|
||
Comment 11•23 years ago
|
||
*** Bug 92489 has been marked as a duplicate of this bug. ***
|
|
Assignee | |
Comment 12•23 years ago
|
||
fixed along with folder pane landing
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Component: XP Toolkit/Widgets: Trees → XUL
QA Contact: jrgmorrison → xptoolkit.widgets
You need to log in
before you can comment on or make changes to this bug.
Description
•