From: Kohei Yoshino <kohei.yoshino@gmail.com>
Date: Thu, 25 Apr 2019 21:38:30 +0000 (-0400)
Subject: Bug 1543438 - Enter Bug page: Hide Severity and Mentors as advanced fields
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=a8d378ad01a078a94f6311de1329828bdc92c307;p=thirdparty%2Fbugzilla.git

Bug 1543438 - Enter Bug page: Hide Severity and Mentors as advanced fields
---

diff --git a/extensions/Review/template/en/default/hook/bug/create/create-after_custom_fields.html.tmpl b/extensions/Review/template/en/default/hook/bug/create/create-after_custom_fields.html.tmpl
index 4a8f05755..2ef267a9c 100644
--- a/extensions/Review/template/en/default/hook/bug/create/create-after_custom_fields.html.tmpl
+++ b/extensions/Review/template/en/default/hook/bug/create/create-after_custom_fields.html.tmpl
@@ -6,7 +6,7 @@
   # defined by the Mozilla Public License, v. 2.0.
   #%]
 
-<tr>
+<tr class="expert_fields">
   <th class="field_label">Mentors:</th>
   <td colspan="3" class="field_value">
      [% INCLUDE global/userselect.html.tmpl
diff --git a/skins/standard/enter_bug.css b/skins/standard/enter_bug.css
index 936b51f11..0f5884449 100644
--- a/skins/standard/enter_bug.css
+++ b/skins/standard/enter_bug.css
@@ -39,7 +39,7 @@
  * pretty well on all browsers except IE 8.
  */
 #Create #field_container_component { width: 1px; }
-#Create #field_container_reporter  { width: 100%; }
+#Create #required_marker           { width: 100%; }
 
 #Create .comment {
     vertical-align: top;
diff --git a/template/en/default/bug/create/create.html.tmpl b/template/en/default/bug/create/create.html.tmpl
index a06351be8..47e946827 100644
--- a/template/en/default/bug/create/create.html.tmpl
+++ b/template/en/default/bug/create/create.html.tmpl
@@ -234,19 +234,17 @@ TUI_hide_default('expert_fields');
                                    'bz_default_hidden');
       </script>
     </td>
-    <td colspan="2">
-      (<span class="required_star">*</span> =
-      <span class="required_explanation">Required Field</span>)
-   </td>
+    <td colspan="2">&nbsp;</td>
   </tr>
 
   <tr>
     [% INCLUDE bug/field.html.tmpl
       bug = default, field = bug_fields.product, editable = 0,
       value = product.name %]
-    [% INCLUDE bug/field.html.tmpl
-      bug = default, field = bug_fields.reporter, editable = 0,
-      value = user.login %]
+    <td colspan="2" id="required_marker">
+      (<span class="required_star">*</span> =
+      <span class="required_explanation">Required Field</span>)
+    </td>
   </tr>
 
   [%# We can't use the select block in these two cases for various reasons. %]
@@ -314,9 +312,9 @@ TUI_hide_default('expert_fields');
 
   <tr>
     [% INCLUDE "bug/field-label.html.tmpl"
-      field = bug_fields.version editable = 1 rowspan = 4
+      field = bug_fields.version editable = 1 rowspan = 3
     %]
-    <td rowspan="4">
+    <td rowspan="3">
       <select name="version" id="version" size="5">
         [%- FOREACH v = version %]
           [% NEXT IF NOT v.is_active %]
@@ -332,12 +330,6 @@ TUI_hide_default('expert_fields');
       value = default.bug_type %]
   </tr>
 
-  <tr>
-    [% INCLUDE bug/field.html.tmpl
-      bug = default, field = bug_fields.bug_severity, editable = 1,
-      value = default.bug_severity %]
-  </tr>
-
   [% needs_extra_tr = 1 %]
   <tr>
     <th>
@@ -396,18 +388,15 @@ TUI_hide_default('expert_fields');
       <td colspan="2">&nbsp;</td>
     [% END %]
   </tr>
-</tbody>
-
-<tbody class="expert_fields">
-  <tr>
-    <td colspan="4">&nbsp;</td>
-  </tr>
 
   <tr>
     [% INCLUDE bug/field.html.tmpl
       bug = default, field = bug_fields.bug_status,
       editable = (bug_status.size > 1), value = default.bug_status
       override_legal_values = bug_status %]
+    [% INCLUDE bug/field.html.tmpl
+      bug = default, field = bug_fields.bug_severity, editable = 1,
+      value = default.bug_severity %]
   </tr>
 
   <tr>