Closed Bug 1582533 Opened 5 years ago Closed 5 years ago

remove <observes> usage from mailnews/

Categories

(MailNews Core :: Feed Reader, task)

Product:
MailNews Core
Component:
Feed Reader
Type:
task
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
Thunderbird 71.0

People

(Reporter: mkmelin, Assigned: mkmelin)

References

Details

Attachments

(1 file)

bug1582533_observes.mailnews.patch
6.38 KB, patch
alta88
: review+
Details | Diff | Splinter Review

+++ This bug was initially created as a clone of Bug #1581305 +++

Remove <observes> usage from mailnews/

https://searchfox.org/comm-central/search?q=%3Cobserves&case=false&regexp=false&path=mailnews

All the indirection had me confused there for a while.

Attachment #9094008 - Flags: review?(alta88)
Status: NEW → ASSIGNED
Comment on attachment 9094008 [details] [diff] [review]
bug1582533_observes.mailnews.patch

rs+
too bad <fieldset> doesn't do labels on disable.
Attachment #9094008 - Flags: review?(alta88) → review+

Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/b19ccd7ebef3
remove <observes> usage from mailnews/. r=alta88

Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 71.0
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: