Closed Bug 74835 Opened 23 years ago Closed 21 years ago

marking thread/all read needs to invalidate collapsed thread(s) having unread children

Categories

(SeaMonkey :: MailNews: Message Display, defect)

Product:
SeaMonkey
Component:
MailNews: Message Display
Platform:
x86
All
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
mozilla1.4beta

People

(Reporter: sspitzer, Assigned: ch.ey)

References

Details

Attachments

(1 file)

Proposed patch
1.33 KB, patch
neil
: review+
sspitzer
: superreview+
Details | Diff | Splinter Review 
mark all read can leave the outliner needed an invalidate

more details on the way..
to reproduce:

have a thread, and the top level message is read, but there are unread children.
collapse the thread.

do "mark all read".

we need to invalidate the outliner row for the thread parent, since the icon
should change and so should the unread thread counts.
Status: NEW → ASSIGNED
QA Contact: esther → fenella
QA Contact: fenella → laurel
*** Bug 116898 has been marked as a duplicate of this bug. ***
I don't understand the summary.

Bug 113000 might be a dupe.

pi
Yes, Bug 113000 is probably a dupe of this one.
For sure my bug 116898 is a dupe of this one.

*** Bug 113000 has been marked as a duplicate of this bug. ***
*** Bug 91107 has been marked as a duplicate of this bug. ***
*** Bug 162503 has been marked as a duplicate of this bug. ***
Summary: mark all read can leave the outliner needed an invalidate → marking thread/all read needs to invalidate collapsed thread(s) having unread children
*** Bug 132622 has been marked as a duplicate of this bug. ***
Attached patch Proposed patch — — Splinter Review 
For an explanation of this patch see bug 132622, comment 6.
This patch has been changed to eliminate double notify in case of the first
message of the thread.
Attachment #119307 - Flags: superreview?(sspitzer)
Attachment #119307 - Flags: review+
looks good, re-assign for proper credit
Assignee: sspitzer → ch.ey
Status: ASSIGNED → NEW
Target Milestone: --- → mozilla1.4beta
Comment on attachment 119307 [details] [diff] [review]
Proposed patch

sr=sspitzer.  thanks for the patch (and to neil, for the review)

neil, can you land this?
Attachment #119307 - Flags: superreview?(sspitzer) → superreview+
Fix checked in.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: