467174 - Add preferences options for popup and missing plugin infobars

Status: VERIFIED FIXED

(SeaMonkey :: Preferences, enhancement)

Description

[Jens Hatlak (:InvisibleSmiley)]

As per discussion in m.d.seamonkey:

Add checkboxes to Privacy & Security > Popup Windows and Advanced > Scripts & Plugins respectively that change the corresponding already existing prefs, namely privacy.popups.showBrowserMessage and plugins.hide_infobar_for_missing_plugin. For the latter the middle part of the pref pane ("Allow scripts to:") could be reduced in size.

Comment 1

[Jens Hatlak (:InvisibleSmiley)]

Attachment: Popups part

Comment 2

[Jens Hatlak (:InvisibleSmiley)]

Attachment: patch for both

Chose to use different labels, now all in one patch

Comment 3

[[e-mail]]

Comment on attachment 350653 [details] [diff] [review]
patch for both

>+    
Nit: whitespace

>+      <hbox>
pref-popups likes to wrap its checkboxes in hboxes but pref-scripts doesn't.

>+        <checkbox id="displayPluginsInfoBar"
>+                  label="&displayPluginsInfoBar.label;"
>+                  accesskey="&displayPluginsInfoBar.accesskey;"
>+                  preference="plugins.hide_infobar_for_missing_plugin"/>
I don't think that this checkbox really fits in this group.

>+<!ENTITY displayPopupsInfoBar.label     "Display an info bar at the top of the content area">
Aren't we supposed to call this a notification bar, rather than an info bar?
Also, this is pref-popups, no need to repeat yourself.

>+<!ENTITY displayPopupsInfoBar.accesskey "I">
I is a very poor choice of accesskey. (I've always wanted to say "I is" ;-)

>+<!ENTITY displayPluginsInfoBar.label     "Display an info bar at the top of the content area">
Except that this label doesn't explain why the notification might occur.

>\ No newline at end of file
Oops ;-)

Comment 4

[Jens Hatlak (:InvisibleSmiley)]

Attachment: patch v2

Nits addressed. Instead of changing the groupbox captions or introducing yet another groupbox I chose to make the Scripts & Plugins groupboxes look more similar (caption, checkboxes, separator, description, checkboxes).

Comment 5

[[e-mail]]

Comment on attachment 350873 [details] [diff] [review]
patch v2

>+      <description id="whenAddlPluginsRequired">&whenAddlPluginsRequired.description;</description>
>+      <checkbox id="displayPluginsNotificationBar"
Nit: blank line between these two please.
I don't really like the Addl there so just call that one whenPluginsRequired and you might as well use displayNotification without the Bar (in all files) so that you can align the text up in the .dtd files ;-)

Comment 6

[Jens Hatlak (:InvisibleSmiley)]

Attachment: final patch [Checkin: Comment 9]

nits addressed, carrying over r+

Comment 7

[Stefan [:stefanh]]

How about filing a bug for updating our help files? ;-)

Comment 8

[Jens Hatlak (:InvisibleSmiley)]

Thanks Neil.

To whoever checks this in: don't get confused, r+sr=Neil of course.

(In reply to comment #7)
> How about filing a bug for updating our help files? ;-)

Filed bug 467674.

Comment 9

[Serge Gautherie (:sgautherie)]

Comment on attachment 351008 [details] [diff] [review]
final patch
[Checkin: Comment 9]

http://hg.mozilla.org/comm-central/rev/98e20aa2f22e

Comment 10

[Tony Mechelynck [:tonymec]]

Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1b3pre) Gecko/20081206 SeaMonkey/2.0a3pre - Build ID: 20081206000455

Checkboxes are there on both pref panels (on Linux) => VERIFIED.

REOPEN if the fix doesn't work in the latest nightly for some other platform.