NNTP threaded collapsed message list shows unread messages (underscore)
Categories
(Thunderbird :: Folder and Message Lists, defect)
Tracking
(relnote-thunderbird 133+, thunderbird_esr115 wontfix, thunderbird_esr128? verified, thunderbird131 wontfix, thunderbird132 affected, thunderbird133 verified)
People
(Reporter: shopik, Assigned: welpy-cw)
References
Details
(Keywords: regression)
Attachments
(3 files)
62.41 KB,
image/png
|
Details | |
55.07 KB,
image/jpeg
|
Details | |
48 bytes,
text/x-phabricator-request
|
corey
:
approval-comm-esr128+
|
Details | Review |
Comment 1•15 years ago
|
||
Reporter | ||
Comment 2•15 years ago
|
||
Reporter | ||
Comment 3•15 years ago
|
||
Reporter | ||
Updated•15 years ago
|
Reporter | ||
Comment 4•15 years ago
|
||
Reporter | ||
Comment 6•15 years ago
|
||
Reporter | ||
Comment 7•15 years ago
|
||
Comment 8•15 years ago
|
||
Reporter | ||
Comment 9•15 years ago
|
||
Comment 10•15 years ago
|
||
Reporter | ||
Comment 11•15 years ago
|
||
Reporter | ||
Updated•15 years ago
|
Reporter | ||
Comment 12•14 years ago
|
||
Updated•14 years ago
|
Comment 14•13 years ago
|
||
Comment 15•13 years ago
|
||
Reporter | ||
Comment 16•13 years ago
|
||
Comment 17•13 years ago
|
||
Comment 18•13 years ago
|
||
Comment 19•13 years ago
|
||
Comment 20•12 years ago
|
||
Comment 21•12 years ago
|
||
Comment 22•11 years ago
|
||
Comment 23•11 years ago
|
||
Comment 24•11 years ago
|
||
Comment 25•9 years ago
|
||
Updated•2 years ago
|
Comment 28•5 months ago
|
||
(In reply to Wayne Mery (:wsmwk) from comment #27)
Alfred does this reproduce for you?
Yes, I can still see that here in TB132.0a1.
I have also seen the double header download (Comment #8) in the past. But not at the moment. So that is probably not the cause.
Updated•5 months ago
|
Assignee | ||
Comment 29•4 months ago
|
||
(In reply to Alfred Peters [:infofrommozilla] from comment #28)
(In reply to Wayne Mery (:wsmwk) from comment #27)
Alfred does this reproduce for you?
Yes, I can still see that here in TB132.0a1.
If repairing the folder helps, this could be a remnant of the behavior fixed in bug 448876. If you encounter this with newly received threads as well, then rather not.
Comment 30•4 months ago
•
|
||
(In reply to Hartmut Welpmann [:welpy-cw] from comment #29)
If repairing the folder helps,
In fact, “Repair Folder” (see comment #8 / comment #15) is causing the problem for me.
STR:
- Unsubscribe group.
- Close Thunderbird.
- Make sure that all group files in the profile have been deleted.
- Start Thunderbird.
- Subscribe to the group.
- "Group" -> Context menu -> Properties -> /Synchronization\
=> [x] Select this folder for offline use - "Group" -> Context menu -> Properties -> /Synchronization\
=> [Download Now] - Select "Group"
- Menu -> Message -> Mark -> All Read
- Restart Thunderbird.
Check: all articles are still read
- "Group" -> Context menu -> Properties -> /General Information\
=> [Repair Folder]
Check: closed threads have underscore now.
- Select an article and Menu -> Go -> Next -> Unread Message
Thunderbird opens all threads but doesn't find any unread articles.
Assignee | ||
Comment 31•4 months ago
|
||
(In reply to Alfred Peters [:infofrommozilla] from comment #30)
Check: all articles are still read
- "Group" -> Context menu -> Properties -> /General Information\
=> [Repair Folder]Check: closed threads have underscore now.
I can reproduce this. There certainly is something off …
- Select an article and Menu -> Go -> Next -> Unread Message
Thunderbird opens all threads but doesn't find any unread articles.
Does this persist afterwards (or after typing *
and \
)?
Comment 32•4 months ago
|
||
(In reply to Hartmut Welpmann [:welpy-cw] from comment #31)
(In reply to Alfred Peters [:infofrommozilla] from comment #30)
- "Group" -> Context menu -> Properties -> /General Information\
=> [Repair Folder]Check: closed threads have underscore now.
These also survive a restart of Thunderbird.
I noticed something else. Despite the underlines, the Menu->Message->Mark->All Read menu item is disabled.
Thunderbird opens all threads but doesn't find any unread articles.
Does this persist afterwards (or after typing
*
and\
)?
No, once a thread has been opened (no matter how) it loses this status and thus also the underline after it has been closed again.
Assignee | ||
Comment 33•4 months ago
|
||
This patch sets the unread status of downloaded headers according to the newsrc file as soon as they are added to the message database. Previously this was only done when the view was initialized, but nsMsgDBView::AdjustReadFlag
wasn't called on elided messages in collapsed threads. Especially after repairing the folder or getting the next n messages, this could result in an incorrect and fragile view.
Updated•4 months ago
|
Assignee | ||
Updated•4 months ago
|
Assignee | ||
Updated•4 months ago
|
Assignee | ||
Updated•4 months ago
|
Comment 34•4 months ago
|
||
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/f4ff3b339a71
Adjust unread status of NNTP headers when downloading. r=mkmelin
Comment 35•4 months ago
|
||
comm/mailnews/news/test/unit/test_filter.js is failing
Comment 36•4 months ago
|
||
Assignee | ||
Comment 37•4 months ago
•
|
||
The resetFilter()
function used in that test did not reset the newsrc
file as well between iterations. This works now, Try looking much greener.
Assignee | ||
Updated•4 months ago
|
Comment 38•4 months ago
|
||
Pushed by brendan@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/105af642383a
Adjust unread status of NNTP headers when downloading. r=mkmelin
Comment 40•3 months ago
|
||
Confirming this issue as verified fixed with 133.0b3(20241107210012) using Windows 11, macOS 14, and Ubuntu 22.
Assignee | ||
Comment 41•3 months ago
|
||
Comment on attachment 9430958 [details]
Bug 520582 - Adjust unread status of NNTP headers when downloading. r=infofrommozilla,mkmelin!
[Approval Request Comment]
User impact if declined: Threads in a newsgroup may appear unread even when containing only read messages. Repairing the folder could make the matter worse, also getting the next n message may not work as expected. Ultimately the thread pane may hang/break in these cases.
Testing completed (on c-c, etc.): c-c and beta
Risk to taking this patch (and alternatives if risky): low
Assignee | ||
Comment 42•3 months ago
|
||
Release Note Request (optional, but appreciated)
[Suggested wording]: Correctly maintain the unread/read status of newsgroup articles.
Comment 43•3 months ago
|
||
Comment on attachment 9430958 [details]
Bug 520582 - Adjust unread status of NNTP headers when downloading. r=infofrommozilla,mkmelin!
[Triage Comment]
Approved for esr128
Comment 44•3 months ago
|
||
bugherder uplift |
Thunderbird 128.4.3esr:
https://hg.mozilla.org/releases/comm-esr128/rev/f849d90feb72
Comment 45•3 months ago
|
||
Confirming this issue as verified fixed with 128.4.3(20241108210256) on macOS 14, Windows 11 and Ubuntu 24.
Updated•3 months ago
|
Description
•