122642 - Ignore/Watch thread: icon doesn't appear until next visit to newsgroup or switch views.

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Message Display, defect)

Description

[laurel]

Using jan30 commercial trunk build

When you ignore a thread, the ignored icon won't appear until exit the newsgroup
and revisit it.

Steps:
1.  Open a newsgroup, view threaded, turn view ignored threads toggle ON.
2.  Select a multi-message thread (avoiding bug 122640 about single message
threads) and ignored it -- Message|Ignore Thread.
    Any unread messages in thread are marked read, thread collapses, you're
moved to next unread message in group.  No icon appears, though.
3.  Select another newsgroup then return... now the icon appears on the thread
you ignored.

Result:  ignored thread icon doesn't appear until subsequent visit to newsgroup.
Expected: ignored thread icon should appear immediately on ignore command.

Comment 1

[laurel]

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

Comment 2

[laurel]

Same for watch thread -- if you switch views while still in the newsgroup, the
icon will appear or select another newgroup and return.

Comment 3

[David :Bienvenu]

I'll take this - we're not sending the notification in this case.

Comment 4

[Håkan Waara]

A simple NoteChange(), right?  I was just about to start debugging this...

Comment 5

[David :Bienvenu]

no. I've already fixed it - patch upcoming, and you'll see what the fix is.

Comment 6

[David :Bienvenu]

Attachment: proposed fix

need to adjust the extra flags as well.

Comment 7

[David :Bienvenu]

Hwaara, no, that's wrong. People want to be able to ignore a thread whenever
they're reading any message in the thread. General useage is read a few messages
in a thread, get sick of it, and ignore the whole thread by pressing ignore.

Comment 8

[Håkan Waara]

Comment on attachment 67136 [details] [diff] [review]
proposed fix

r=hwaara

Comment 9

[Håkan Waara]

bienvenu, can you reproduce bug 122640 with this fix in your tree?

Comment 10

[David :Bienvenu]

fix checked in

Comment 11

[laurel]

OK using jan31 commercial trunk build: win98, mac OS 10.1, linux rh6.2