Closed Bug 1571287 Opened 5 years ago Closed 5 years ago

Add isShowingJunkNotification function to SeaMonkey mail component.

Categories

(SeaMonkey :: MailNews: Message Display, enhancement)

enhancement
Not set
normal

Tracking

(seamonkey2.49esr fixed, seamonkey2.63 wontfix, seamonkey2.53 fixed, seamonkey2.57esr fixed)

RESOLVED FIXED
seamonkey2.67
Tracking Status
seamonkey2.49esr --- fixed
seamonkey2.63 --- wontfix
seamonkey2.53 --- fixed
seamonkey2.57esr --- fixed

People

(Reporter: frg, Assigned: frg)

Details

Attachments

(2 files)

The isShowingJunkNotification() function is missing from SeaMonkey. It was added to Thunderbird in bug 562048.

The latest version of the lookout add-on no longer works in SeaMonkey because of this.

https://addons.thunderbird.net/en-US/thunderbird/addon/lookout-fix-version/

Just add the missing function for 2.49.5. r+ a+ from IanN over irc.

ewong how far are you with the Windows 2.49.5 uploads. IanN needs to do a linux plus macOS build because of a missing l10n patch but Windows is basically done. I would only add this to 2.49.5 and do a new Windows build if it does not cause a problem for you.

Flags: needinfo?(ewong)
Attachment #9082897 - Flags: review+
Attachment #9082897 - Flags: approval-comm-esr52+

Add the missing function and aligns its usage with Thunderbird.

[Approval Request Comment]
Regression caused by (bug #): --
User impact if declined: lookout add-on not working
Testing completed (on m-c, etc.): 2.53
Risk to taking this patch (and alternatives if risky): 2.53 is still in development so not risky.
String changes made by this patch: none

Attachment #9082898 - Flags: review?(iann_bugzilla)
Attachment #9082898 - Flags: approval-comm-release?
Attachment #9082898 - Flags: approval-comm-esr60?
Comment on attachment 9082898 [details] [diff] [review]
1571287-add-isShowingJunkNotification.patch

[Triage Comment]
LGTM r/a=me
Attachment #9082898 - Flags: review?(iann_bugzilla)
Attachment #9082898 - Flags: review+
Attachment #9082898 - Flags: approval-comm-release?
Attachment #9082898 - Flags: approval-comm-release+
Attachment #9082898 - Flags: approval-comm-esr60?
Attachment #9082898 - Flags: approval-comm-esr60+

(In reply to Frank-Rainer Grahl (:frg) from comment #1)

Created attachment 9082897 [details] [diff] [review]
1571287-add-isShowingJunkNotification-52.patch

Just add the missing function for 2.49.5. r+ a+ from IanN over irc.

ewong how far are you with the Windows 2.49.5 uploads. IanN needs to do a linux plus macOS build because of a missing l10n patch but Windows is basically done. I would only add this to 2.49.5 and do a new Windows build if it does not cause a problem for you.

No worries. Uploaded build2 already today.

When build3 is available to be processed, I should be able to quickly upload it. My next plan is to have a local system that'd send the files to a staging system.

Flags: needinfo?(ewong)

Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/97f7da288241
Add isShowingJunkNotification for LookOut add-on. r=IanN

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

Attachment

General

Created:
Updated:
Size: