From 7bf1c7d23ffb82247cfaf73069908f29bf891e35 Mon Sep 17 00:00:00 2001
From: "mkanat%bugzilla.org" <>
Date: Mon, 20 Jul 2009 04:14:15 +0000
Subject: 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

---
 template/en/default/attachment/create.html.tmpl | 7 +++++--
 template/en/default/attachment/edit.html.tmpl   | 7 +++++--
 2 files changed, 10 insertions(+), 4 deletions(-)

(limited to 'template/en/default/attachment')

diff --git a/template/en/default/attachment/create.html.tmpl b/template/en/default/attachment/create.html.tmpl
index 687cd7cbf..f725f1076 100644
--- a/template/en/default/attachment/create.html.tmpl
+++ b/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/template/en/default/attachment/edit.html.tmpl b/template/en/default/attachment/edit.html.tmpl
index db92d18d3..1c6d22c87 100644
--- a/template/en/default/attachment/edit.html.tmpl
+++ b/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>
-- 
cgit v1.2.3-24-g4f1b