Closed Bug 765466 Opened 12 years ago Closed 12 years ago

Implement click-to-play plugins management for feed messages

Categories

(SeaMonkey :: MailNews: Message Display, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: InvisibleSmiley, Assigned: neil)

References

Details

Attachments

(1 file)

Bug 743312 and bug 747519 implemented click-to-play plugins management for the browser (plugin area click handler plus UI in the form of doorhanger or notification bar). MailNews needs similar support for feed messages, too so you can choose to allow plugins temporarily or permanently for a certain site.

Since we have no doorhangers in MailNews, providing notification bars should be enough (in addition to the plugin area click handler).

If full management UI is considered too much for MailNews (one can always use the browser or Data Manager for these tasks), we should at least implement the plugin area click handler to temporarily enable individual plugin instances.
This should just be a case of adding a <notificationbox> to the appropriate windows, which should then do the rest automagically.
(In reply to neil@parkwaycc.co.uk from comment #1)
> This should just be a case of adding a <notificationbox> to the appropriate
> windows, which should then do the rest automagically.

Even if that worked: As I said, MailNews has no doorhangers, to it would only work if you had doorhangers disabled (or did you talk about the plugin area click handler?).
Well I'd hang on for a while. Once ewong's notificationbox patch for Thunderbird lands and he ports that to SeaMonkey adding click-to-play notifictions should be easier.
TB Bug 562048
SM Bug 633937
Depends on: 633937
(In reply to Jens Hatlak from comment #2)
> (In reply to comment #1)
> > This should just be a case of adding a <notificationbox> to the appropriate
> > windows, which should then do the rest automagically.
> 
> Even if that worked: As I said, MailNews has no doorhangers, to it would
> only work if you had doorhangers disabled (or did you talk about the plugin
> area click handler?).

Don't worry, <notificationbox> is way cleverer than that. While navigator.js turns on doorhanger support for the tabbrowser, other notificationboxes don't even know that doorhangers are enabled.

(In reply to Philip Chee from comment #3)
> Well I'd hang on for a while. Once ewong's notificationbox patch for
> Thunderbird lands and he ports that to SeaMonkey adding click-to-play
> notifications should be easier.
> TB Bug 562048
> SM Bug 633937

Sure, but we may need to backport the click-to-play plugins management to aurora.
Attached patch Proposed patchSplinter Review
Obviously the msgNotificationBar deck will go away at some point.

Anyone got any good ideas as to what to do about those long lines?
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #636688 - Flags: review?(mnyromyr)
Attachment #636688 - Flags: feedback?(jh)
Comment on attachment 636688 [details] [diff] [review]
Proposed patch

Tested MailNews main and standalone windows, single plugin activation through single clicking a plugin instance, Activate Plugins and Remember Choice (or whatever it's called) from the notification bar (in MailNews, got a doorhanger in the browser as expected without changing a pref). All working fine, f=me. Thanks!

(In reply to neil@parkwaycc.co.uk from comment #5)
> Anyone got any good ideas as to what to do about those long lines?

Don't know what you mean, but I guess I'm less picky.
Attachment #636688 - Flags: feedback?(jh) → feedback+
Comment on attachment 636688 [details] [diff] [review]
Proposed patch

(In reply to neil@parkwaycc.co.uk from comment #5)
> Anyone got any good ideas as to what to do about those long lines?

Leave them as is. The 80 columns limit is a bit overrated these days. ;-)
Attachment #636688 - Flags: review?(mnyromyr) → review+
Pushed comm-central changeset 0eb2a5d6ba93.
Status: ASSIGNED → RESOLVED
Closed: 12 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: