120584 - Refresh thread lines in outliner after inserting, moving or deleting

Status: RESOLVED WONTFIX

(Core :: XUL, defect, P3)

Description

[Jan Varga [:janv]]

While I was playing with dnd in bookmarks I noticed that thread lines are
sometimes not correct.

Comment 1

[Jan Varga [:janv]]

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

Comment 2

[Pierre Chanial]

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

Comment 3

[Jan Varga [:janv]]

Nominating for buffy.
This is higly visible when moving bookmarks in bookmark manager and it looks ugly.

Comment 4

[Samir Gehani]

Could you indicate steps to reproduce and possibly screen shots?  Thanks.

Comment 5

[Jan Varga [:janv]]

Attachment: manage bookmarks shot

Folder 1
 - bookmark 1
 - bookmark 2
 - bookmark 3 (last bookmark in this folder)

Delete bookmark 3 and then you will see that thread line near bookmark 2 is
invalid.

Comment 6

[Jan Varga [:janv]]

Attachment: folder pane shot

1. Create a new profile
2. Start mailnews
3. Create an IMAP account which already has some folders
4. Login to this account (you should see how it's discovering folders)

Some thread lines are not continuous.

Comment 7

[Samir Gehani]

Nav triage team: nsbeta1+/adt3

Comment 8

[[e-mail]]

Ok, so when removing the last child or appending a child you need to invalidate
the previous sibling and its children (i.e. the range of rows from the previous
to the added/deleted row). Is it up to the view to do this?

Comment 9

[Jan Varga [:janv]]

I think it would be better to fix it in tree widget itself.
So, we should make RowCountChanged smarter and fix ToggleOpenState too while we
are there.

Comment 10

[Samir Gehani]

Nav triage team: nsbeta1-

Comment 11

[Wayne Mery (:wsmwk)]

Jan, do you still see this issue?

Comment 12

[Jan Varga [:janv]]

I don't think it was fixed in the tree widget, maybe there are some hacks at higher level to fix the problem.

Comment 13

[Jan Varga [:janv]]

I don't think anyone is going to work on this given the plan to remove the XUL "tree" widget (bug 1446335).