Refresh thread lines in outliner after inserting, moving or deleting

ASSIGNED
Assigned to

Status

()

Core
XUL
P3
normal
ASSIGNED
17 years ago
4 years ago

People

(Reporter: janv, Assigned: janv)

Tracking

(Blocks: 1 bug)

Trunk
mozilla1.4beta
x86
All
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [adt3])

Attachments

(2 attachments)

(Assignee)

Description

17 years ago
While I was playing with dnd in bookmarks I noticed that thread lines are
sometimes not correct.
(Assignee)

Updated

17 years ago
Priority: -- → P3
Target Milestone: --- → Future
(Assignee)

Updated

17 years ago
Status: NEW → ASSIGNED
Target Milestone: Future → mozilla1.0
(Assignee)

Comment 1

17 years ago
*** Bug 124605 has been marked as a duplicate of this bug. ***
(Assignee)

Updated

17 years ago
Target Milestone: mozilla1.0 → mozilla1.0.1

Comment 2

16 years ago
*** Bug 129698 has been marked as a duplicate of this bug. ***
(Assignee)

Comment 3

16 years ago
Nominating for buffy.
This is higly visible when moving bookmarks in bookmark manager and it looks ugly.
Keywords: nsbeta1

Comment 4

16 years ago
Could you indicate steps to reproduce and possibly screen shots?  Thanks.
Whiteboard: [need info]
(Assignee)

Comment 5

16 years ago
Created attachment 91875 [details]
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.
(Assignee)

Comment 6

16 years ago
Created attachment 91876 [details]
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

16 years ago
Nav triage team: nsbeta1+/adt3
Keywords: nsbeta1 → nsbeta1+
Whiteboard: [need info] → [adt3]

Comment 8

16 years ago
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?
(Assignee)

Comment 9

16 years ago
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.
(Assignee)

Updated

16 years ago
Target Milestone: mozilla1.0.1 → mozilla1.4beta

Comment 10

16 years ago
Nav triage team: nsbeta1-
Keywords: nsbeta1+ → nsbeta1-

Updated

15 years ago
Blocks: 236849

Updated

10 years ago
Component: XP Toolkit/Widgets: Trees → XUL
QA Contact: jrgmorrison → xptoolkit.widgets

Comment 11

4 years ago
Jan, do you still see this issue?
Flags: needinfo?(Jan.Varga)
(Assignee)

Comment 12

4 years ago
I don't think it was fixed in the tree widget, maybe there are some hacks at higher level to fix the problem.
Flags: needinfo?(Jan.Varga)
You need to log in before you can comment on or make changes to this bug.