From 57e88fa6fc43df22798032708749025485a790fc Mon Sep 17 00:00:00 2001
From: "mkanat%bugzilla.org"
 <mkanat%bugzilla.org@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Mon, 20 Jul 2009 04:14:18 +0000
Subject: [PATCH] Bug 488943: Clarify the text of the "Private" checkbox for
 comments and attachments Patch by Max Kanat-Alexander <mkanat@bugzilla.org>
 r=pyrzak, r=LpSolit, a=LpSolit

git-svn-id: svn://10.0.0.236/trunk@257728 18797224-902f-48f8-a5cc-f745e15eee43
---
 .../template/en/default/attachment/create.html.tmpl       | 7 +++++--
 .../template/en/default/attachment/edit.html.tmpl         | 7 +++++--
 .../template/en/default/bug/create/create.html.tmpl       | 5 +++--
 .../bugzilla/template/en/default/bug/edit.html.tmpl       | 8 ++++++--
 .../template/en/default/list/edit-multiple.html.tmpl      | 5 ++++-
 5 files changed, 23 insertions(+), 9 deletions(-)

diff --git a/mozilla/webtools/bugzilla/template/en/default/attachment/create.html.tmpl b/mozilla/webtools/bugzilla/template/en/default/attachment/create.html.tmpl
index 687cd7cbf24..f725f107648 100644
--- a/mozilla/webtools/bugzilla/template/en/default/attachment/create.html.tmpl
+++ b/mozilla/webtools/bugzilla/template/en/default/attachment/create.html.tmpl
@@ -110,10 +110,13 @@
       <tr>
         <th>Privacy:</th>
         <td>
-          <em>If the attachment is private, check the box below.</em><br>
           <input type="checkbox" name="isprivate" id="isprivate"
           value="1" onClick="updateCommentPrivacy(this)">
-        <label for="isprivate">Private</label>
+          <label for="isprivate">
+            Make attachment and comment private (visible only to members of
+            the <strong>[% Param('insidergroup') FILTER html %]</strong>
+            group)
+          </label>
         </td>
       </tr>
     [% END %]
diff --git a/mozilla/webtools/bugzilla/template/en/default/attachment/edit.html.tmpl b/mozilla/webtools/bugzilla/template/en/default/attachment/edit.html.tmpl
index db92d18d396..1c6d22c8707 100644
--- a/mozilla/webtools/bugzilla/template/en/default/attachment/edit.html.tmpl
+++ b/mozilla/webtools/bugzilla/template/en/default/attachment/edit.html.tmpl
@@ -94,10 +94,13 @@
           <input type="checkbox" id="isobsolete" name="isobsolete" value="1"
                  [%+ 'checked="checked"' IF attachment.isobsolete %]>
           <label for="isobsolete">obsolete</label>
-          [% IF (Param("insidergroup") && user.in_group(Param("insidergroup"))) %]
+          [% IF user.is_insider %]
+            <br>
             <input type="checkbox" id="isprivate" name="isprivate" value="1"
                    [% " checked" IF attachment.isprivate %]>
-            <label for="isprivate">private</label><br>
+            <label for="isprivate">private (only visible to 
+              <strong>[% Param('insidergroup') FILTER html %]</strong>)
+            </label>
           [% END %]
           <br>
         </small>
diff --git a/mozilla/webtools/bugzilla/template/en/default/bug/create/create.html.tmpl b/mozilla/webtools/bugzilla/template/en/default/bug/create/create.html.tmpl
index 2cee5c91d72..e26e0a29ae1 100644
--- a/mozilla/webtools/bugzilla/template/en/default/bug/create/create.html.tmpl
+++ b/mozilla/webtools/bugzilla/template/en/default/bug/create/create.html.tmpl
@@ -517,7 +517,7 @@ TUI_hide_default('expert_fields');
     </td>
   </tr>
 
-  [% IF Param("insidergroup") && user.in_group(Param("insidergroup")) %]
+  [% IF user.is_insider %]
     <tr class="expert_fields">
       <th>&nbsp;</th>
       <td colspan="3">
@@ -525,7 +525,8 @@ TUI_hide_default('expert_fields');
         <input type="checkbox" id="commentprivacy" name="commentprivacy"
           [% " checked=\"checked\"" IF commentprivacy %]>
         <label for="commentprivacy">
-          Initial Description is Private
+          Make description private (visible only to members of the 
+          <strong>[% Param('insidergroup') FILTER html %]</strong> group)
         </label>
       </td>
     </tr>
diff --git a/mozilla/webtools/bugzilla/template/en/default/bug/edit.html.tmpl b/mozilla/webtools/bugzilla/template/en/default/bug/edit.html.tmpl
index de89f1b8e7b..50cf575b246 100644
--- a/mozilla/webtools/bugzilla/template/en/default/bug/edit.html.tmpl
+++ b/mozilla/webtools/bugzilla/template/en/default/bug/edit.html.tmpl
@@ -240,11 +240,15 @@
         [% IF user.id %]
         <table><tr><td>
           <label for="comment" accesskey="c"><b>Additional <u>C</u>omments</b></label>:
-          [% IF Param("insidergroup") && user.in_group(Param("insidergroup")) %]
+          [% IF user.is_insider %]
             <input type="checkbox" name="commentprivacy" value="1"
                    id="newcommentprivacy"
                    onClick="updateCommentTagControl(this, form)">
-            <label for="newcommentprivacy">Private</label>
+            <label for="newcommentprivacy"> 
+              Make comment private (visible only to members of the
+              <strong>[% Param('insidergroup') FILTER html %]</strong>
+              group)
+            </label>
           [% END %]
           <br>
           [% INCLUDE global/textarea.html.tmpl
diff --git a/mozilla/webtools/bugzilla/template/en/default/list/edit-multiple.html.tmpl b/mozilla/webtools/bugzilla/template/en/default/list/edit-multiple.html.tmpl
index 46130ef6bf1..4cbc8b6e2b3 100644
--- a/mozilla/webtools/bugzilla/template/en/default/list/edit-multiple.html.tmpl
+++ b/mozilla/webtools/bugzilla/template/en/default/list/edit-multiple.html.tmpl
@@ -290,7 +290,10 @@
   <input type="checkbox" name="commentprivacy" value="1"
          id="newcommentprivacy"
          onClick="updateCommentTagControl(this, form)"/>
-  <label for="newcommentprivacy">Private</label>
+   <label for="newcommentprivacy">
+     Make comment private (visible only to members of the
+     <strong>[% Param('insidergroup') FILTER html %]</strong> group)
+   </label>
 [% END %]
 <br>
 [% INCLUDE global/textarea.html.tmpl