Add 'Show an alert' checkbox to OSX Preferences

RESOLVED FIXED in Thunderbird 24.0

Status

Thunderbird
Preferences
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: Paenglab, Assigned: Paenglab)

Tracking

unspecified
Thunderbird 24.0
All
Mac OS X

Thunderbird Tracking Flags

(thunderbird24+ fixed)

Details

(URL)

Attachments

(2 attachments)

(Assignee)

Description

4 years ago
With OS X using now the built-in notifications the user should be able to deactivate them easily like on Linux and Windows in Preferences window.
(Assignee)

Comment 1

4 years ago
Created attachment 754548 [details] [diff] [review]
patch

This patch adds only the already existing lines for Linux/Windows to the OS X #ifdef part.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #754548 - Flags: ui-review?(bwinton)
Attachment #754548 - Flags: review?(bwinton)
(Assignee)

Comment 2

4 years ago
Created attachment 754549 [details]
screenshot for easier ui-r
tracking-thunderbird24: --- → +

Comment 3

4 years ago
Is bug 879559 a duplicate of this one? Looks like that. I don't have a Mac, thus cannot comment on the interaction with Growl, but have a look at the test for the notification service I mention in bug 879559 comment #2 if it's the right thing.
(Assignee)

Updated

4 years ago
Duplicate of this bug: 879559
Comment on attachment 754548 [details] [diff] [review]
patch

ui-r=me, although I'm starting to worry about the number of things we're adding here, what with the dock options, and all…

And the code is simple enough, so r=me, too.
Attachment #754548 - Flags: ui-review?(bwinton)
Attachment #754548 - Flags: ui-review+
Attachment #754548 - Flags: review?(bwinton)
Attachment #754548 - Flags: review+
(Assignee)

Updated

4 years ago
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/824f5c8fdae2
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
status-thunderbird24: --- → fixed
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 24.0
You need to log in before you can comment on or make changes to this bug.