80844 - Twisties for nested containers aren't indented

Status: RESOLVED FIXED

(Core :: XUL, defect)

Description

[Blake Ross]

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.

Comment 1

[Blake Ross]

Attachment: screenshot

Comment 2

[Jan Varga [:janv]]

Attachment: patch

Comment 3

[Jan Varga [:janv]]

Blake, could you try attached patch ?

Comment 4

[Blake Ross]

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.

Comment 5

[Blake Ross]

hyatt, can you take a look at Jan's patch?

Comment 6

[Blake Ross]

*** Bug 80840 has been marked as a duplicate of this bug. ***

Comment 7

[Jan Varga [:janv]]

Attachment: better patch

Comment 8

[Jan Varga [:janv]]

Attachment: even better patch, ready for review

Comment 9

[Blake Ross]

Jan, can you give me a quick runthrough of this fix?

Comment 10

[Jan Varga [:janv]]

I changed code back to update currX and remainingWidth according to indentation
before drawing twisty and the connecting lines.

Comment 11

[John Morrison]

*** Bug 92489 has been marked as a duplicate of this bug. ***

Comment 12

[Jan Varga [:janv]]

fixed along with folder pane landing