Closed Bug 1920747 Opened 4 months ago Closed 4 months ago

The "hasNew" attribute is not cleared after all messages have been read

Categories

(Thunderbird :: Folder and Message Lists, defect, P2)

Thunderbird 128
Desktop
All

Tracking

(thunderbird_esr128+ affected, thunderbird132 fixed)

RESOLVED FIXED
133 Branch
Tracking Status
thunderbird_esr128 + affected
thunderbird132 --- fixed

People

(Reporter: aleca, Assigned: aleca)

Details

(Keywords: regression, regressionwindow-wanted, ux-userfeedback)

Attachments

(2 files)

STR

  • Have a thread with multiple new and unread messages
  • Open the thread and read all the messages
  • Close the thread

Result:
The # replies button remains blue colored with new star icon.

Upon inspecting the code it looks like the hasNew property is not correctly cleared while the hasUnread is.
The property is cleared after interacting again with the message (expand/collapse or select again).

The "hasNew" style of the reply button should be styled consistently
in all instances, and clear out immediately after the last new/unread
message of a thread is touched.

Assignee: nobody → alessandro
Status: NEW → ASSIGNED
Target Milestone: --- → 133 Branch

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/70bc03aa3289
Ensure the hasNew CSS property is only used when also hasUnread is present. r=Paenglab,micahilbery,vineet

Status: ASSIGNED → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED

Is this ready for uplift to beta?

Comment on attachment 9428935 [details]
Bug 1920747 - Ensure the hasNew CSS property is only used when also hasUnread is present. r=#thunderbird-front-end-reviewers,Paenglab!

[Approval Request Comment]
Regression caused by (bug #): -
User impact if declined: The "replies" button in cards view remains temporarily styled as if there are still new messages in the collapsed thread of children
Testing completed (on c-c, etc.): on c-c
Risk to taking this patch (and alternatives if risky): low, just a simple CSS change

Attachment #9428935 - Flags: approval-comm-beta?

Comment on attachment 9428935 [details]
Bug 1920747 - Ensure the hasNew CSS property is only used when also hasUnread is present. r=#thunderbird-front-end-reviewers,Paenglab!

[Triage Comment]
Approved for beta

Attachment #9428935 - Flags: approval-comm-beta? → approval-comm-beta+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: