Bug 524251: Move the bug visibility controls to the right of the timetracking…

skins/standard/show_bug.css

@@ -59,6 +59,26 @@ table#flags {
     
 }
 
+#bz_big_form_parts td {
+    vertical-align: top;
+}
+
+.bz_group_visibility_section {
+    margin-left: 1em;
+}
+
+.bz_group_visibility_section .instructions {
+    font-style: italic;
+}
+
+#bz_restrict_group_visibility_help .instructions {
+    margin-top: 0;
+}
+
+#bz_enable_role_visibility_help {
+    margin-top: 1em;
+}
+
 .bz_time_tracking_table {
     border-collapse: collapse;
 }

template/en/default/bug/edit.html.tmpl

@@ -211,15 +211,13 @@
     </tr>
   </table>
 
-  
-  [% PROCESS section_restrict_visibility %]
+  <table id="bz_big_form_parts" cellspacing="0" cellpadding="0"><tr>
+  <td>
     [% IF user.is_timetracker %]
-    <br>
       [% PROCESS section_timetracking %]
     [% END %]
 
-
-[%# *** Attachments *** %]
+    [%# *** Attachments *** %]
 
     [% PROCESS attachment/list.html.tmpl
                attachments = bug.attachments
@@ -228,8 +226,12 @@
                show_attachment_flags = bug.show_attachment_flags
     %]
 
-
     [% PROCESS comment_box %]
+  </td>
+  <td>
+    [% PROCESS section_restrict_visibility %]
+  </td>
+  </tr></table>
 
   [%# *** Additional Comments *** %]
   <div id="comments">
@@ -647,6 +649,7 @@
 [% BLOCK section_restrict_visibility %]
   [% RETURN UNLESS bug.groups.size %]
 
+  <div class="bz_group_visibility_section">
     [% inallgroups = 1 %]
     [% inagroup = 0 %]
     [% emitted_description = 0 %]
@@ -659,66 +662,45 @@
 
       [% IF NOT emitted_description %]
         [% emitted_description = 1 %]
-      <table class="bz_group_visibility_section">
-        <tr>
-          <td class="field_label">
-            <label id="bz_restrict_group_visibility_label"><b>Restrict Group Visibility</b>:</label>
-          </td>
-          <td>
           <div id="bz_restrict_group_visibility_help">
-              <b>Only users in all of the selected groups can view this [% terms.bug %]:</b>
-              <br>
-              <small>
-                (Unchecking all boxes makes this a more public [% terms.bug %].)
-              </small>
+            <b>Only users in all of the selected groups can view this 
+              [%+ terms.bug %]:</b>
+             <p class="instructions">
+               Unchecking all boxes makes this a more public [% terms.bug %].
+             </p>
           </div>
       [% END %]
 
       [% IF group.ingroup %]
         <input type="hidden" name="defined_bit-[% group.bit %]" value="1">
       [% END %]
-    <input type="checkbox" value="1" name="bit-[% group.bit %]" id="bit-[% group.bit %]"
+
+      <input type="checkbox" value="1" name="bit-[% group.bit %]" 
+             id="bit-[% group.bit %]"
              [% ' checked="checked"' IF group.ison %]
              [% ' disabled="disabled"' IF NOT group.ingroup %]>
-    <label for="bit-[% group.bit %]">[% group.description FILTER html_light %]</label>
+      <label for="bit-[% group.bit %]">
+      [%- group.description FILTER html_light %]</label>
       <br>
     [% END %]
 
     [% IF emitted_description %]
       [% IF NOT inallgroups %]
-      <b>Only members of a group can change the visibility of [% terms.abug %] for that group.</b>
-      <br>
+        <p class="instructions">Only members of a group can change the 
+          visibility of [% terms.abug %] for that group.</p>
       [% END %]
-      </td>
-    </tr>
-    [% "</table>" IF NOT inagroup %]
     [% END %]
 
     [% IF inagroup %]
-    [% IF NOT emitted_description %]
-      [% emitted_description = 1 %]
-      <table class="bz_group_visibility_section">
-    [% END %]
-    <tr>
-      <td class="field_label">
-        <label id="bz_enable_role_visibility_label"><b>Enable Role Visibility</b>:</label>
-      </td>
-      <td>
       <div id="bz_enable_role_visibility_help">
-          <b>Users in the roles selected below can always view this [% terms.bug %]:</b>
-          <br>
-          <small>
-            (The assignee
-            [% IF (Param('useqacontact')) %]
-               and QA contact
-            [% END %]
-            can always see [% terms.abug %], and this section does not take effect unless
-            the [% terms.bug %] is restricted to at least one group.)
-          </small>
+        <b>Users in the roles selected below can always view 
+          this [% terms.bug %]:</b>
       </div>
+      <div id="bz_enable_role_visibility">
         <div>
-          <div>
-            [% user_can_edit_accessible = bug.check_can_change_field("reporter_accessible", 0, 1) %]
+          [% user_can_edit_accessible = 
+            bug.check_can_change_field("reporter_accessible", 0, 1) 
+          %]
           [% IF user_can_edit_accessible %]
             <input type="hidden" name="defined_reporter_accessible" value="1">
           [% END %]
@@ -729,7 +711,9 @@
           <label for="reporter_accessible">Reporter</label>
         </div>
         <div>
-            [% user_can_edit_accessible = bug.check_can_change_field("cclist_accessible", 0, 1) %]
+          [% user_can_edit_accessible = 
+            bug.check_can_change_field("cclist_accessible", 0, 1) 
+          %]
           [% IF user_can_edit_accessible %]
             <input type="hidden" name="defined_cclist_accessible" value="1">
           [% END %]
@@ -739,11 +723,18 @@
                  [% " disabled=\"disabled\"" UNLESS user_can_edit_accessible %]>
           <label for="cclist_accessible">CC List</label>
         </div>
+        <p class="instructions">
+          The assignee
+          [% IF (Param('useqacontact')) %]
+             and QA contact
+          [% END %]
+          can always see [% terms.abug %], and this section does not
+          take effect unless the [% terms.bug %] is restricted to at
+          least one group.
+        </p>
       </div>
-      </td>
-    </tr>
-  </table>
     [% END %]
+  </div> [%# bz_group_visibility_section %]
 [% END %]
 
 [%############################################################################%]