Bug 204106: Make enter_bug indicate which fields are mandatory

Patch by Max Kanat-Alexander <mkanat@bugzilla.org> r=dkl, a=mkanat

diff --git a/skins/standard/global.css b/skins/standard/global.css
index 8eb7cc3b1cfc2443ef3b0c41e68da2f1b45787cd..457f909ee091e14ae298656849c77ba1167a89fd 100644 (file)
--- a/skins/standard/global.css
+++ b/skins/standard/global.css
@@ -455,6 +455,15 @@ div.user_match {
 .field_value, form#Create th, form#Create td {
     vertical-align: top;
 }
+th.required:before {
+    content: "* ";
+}
+th.required:before, span.required_star {
+    color: red;
+}
+input.required, select.required, span.required_explanation {
+    background-color: #fff7cd;
+}
 
 .calendar_button {
     background: transparent url("global/calendar.png") no-repeat;

diff --git a/template/en/default/bug/create/create.html.tmpl b/template/en/default/bug/create/create.html.tmpl
index 21f7959a2e0410b7a0532581efce2e5c2c0ff6b5..9b5944cc52b7105cef1bb8c0946083a7bba60cf3 100644 (file)
--- a/template/en/default/bug/create/create.html.tmpl
+++ b/template/en/default/bug/create/create.html.tmpl
@@ -193,16 +193,20 @@ TUI_hide_default('expert_fields');
   </tr>
 
   <tr>
-    <td colspan="4">
-        <a id="expert_fields_controller" class="controller bz_default_hidden"
-            href="javascript:TUI_toggle_class('expert_fields')">Hide
+    <td colspan="2">
+      <a id="expert_fields_controller" class="controller bz_default_hidden"
+         href="javascript:TUI_toggle_class('expert_fields')">Hide
         Advanced Fields</a>
       [%# Show the link if the browser supports JS %]
       <script type="text/javascript">
-          YAHOO.util.Dom.removeClass('expert_fields_controller', 
-                                     'bz_default_hidden');
+        YAHOO.util.Dom.removeClass('expert_fields_controller', 
+                                   'bz_default_hidden');
       </script>
     </td>
+    <td colspan="2">
+      (<span class="required_star">*</span> =
+      <span class="required_explanation">Required Field</span>)
+   </td>
   </tr>
 
   <tr>
@@ -215,12 +219,13 @@ TUI_hide_default('expert_fields');
 
   [%# We can't use the select block in these two cases for various reasons. %]
   <tr>
-    <th>
+    <th class="required">
       <a href="describecomponents.cgi?product=[% product.name FILTER url_quote %]">
       Component</a>:
     </th>
     <td>
-      <select name="component" onchange="set_assign_to();" size="7">
+      <select name="component" onchange="set_assign_to();" size="7"
+              aria-required="true" class="required">
         [%# Build the lists of assignees and QA contacts if "usemenuforusers" is enabled. %]
         [% IF Param("usemenuforusers") %]
           [% assignees_list = user.get_userlist.clone %]
@@ -482,10 +487,11 @@ TUI_hide_default('expert_fields');
 <tbody>
 
   <tr>
-    <th>Summary:</th>
+    <th class="required">Summary:</th>
     <td colspan="3">
       <input name="short_desc" size="70" value="[% short_desc FILTER html %]"
-             maxlength="255" spellcheck="true">
+             maxlength="255" spellcheck="true" aria-required="true"
+             class="required">
     </td>
   </tr>