Closed Bug 204106 Opened 22 years ago Closed 16 years ago

enter bug form should indicate which fields are required/mandatory

Categories

(Bugzilla :: User Interface, defect, P1)

defect

Tracking

()

RESOLVED FIXED
Bugzilla 3.6

People

(Reporter: myk, Assigned: mkanat)

References

Details

Attachments

(2 files, 1 obsolete file)

The "enter bug" form doesn't say which fields are required/mandatory and which are not. It should, so users don't have to submit the form to find out this information.
It should be split into two halves, each with a header. The optional fields would be in the second section, which could be ignored, so users don't need to hunt and peck to figure out what needs to be changed.
Severity: normal → minor
Target Milestone: --- → Bugzilla 2.22
A standard UI on many webapps I've seen is to put a (*) next to the required fields... Perhaps this could be implemented as a CSS style for "required_field" which could be applied in many other places?
QA Contact: mattyt-bugzilla → default-qa
Target Milestone: Bugzilla 2.22 → ---
This might get fixed with bug# 36843 or bug# 317051 when the required fields will have defaults!
This should be done with a css based solution of putting all required elements in a required_field class. Then it would be up to CSS to decide what to do with required elements. I suggest making the background #F7F7BD (the color yellow as the location box in firefox when a secure site is being visited). That way the elements stand out more than they would if there was a * on the side of the field (I always seem to miss them on other sites). A * could still be added on fields in browsers that support css like the following: .required_field:after { content: "(*)" }
Assignee: myk → ui
It would be even better if there were a script preventing me from submitting the form with required fields missing. I keep forgetting to select a component, and when I attach a testcase, I often forget to give it a description.
Mandatory fields comes not only at the time of creation of the bug. But also at the time of updating the bugs. for example while closing the bug we may need to have some fields to be filled like build fixed.
Pyrzak's students' research pointed this out as a significant issue for new users, and I agree that we should note that certain fields are required, and remove the (optional) after Keywords (because almost all the fields are optional...).
Severity: minor → normal
Priority: -- → P1
Target Milestone: --- → Bugzilla 3.6
Blocks: bz-hci2008
Attached patch v1 (obsolete) — Splinter Review
Made the change for the "Summary:" label on the Create New Bug page. It shows up in a yellow background with a red star next to it. Added the required_field class to global.css Please let me know if this is fine, and I can make changes at other places.
Attachment #377701 - Flags: review?(mkanat)
(In reply to comment #8) > Made the change for the "Summary:" label on the Create New Bug page. It shows > up in a yellow background with a red star next to it. > Added the required_field class to global.css > > Please let me know if this is fine, and I can make changes at other places. For accessibility's sake, perhaps it may also be worth making use of the "aria-required" attribute on required form field(s)? For instance, such as this? <input name="short_desc" size="70" value="[% short_desc FILTER html %]" maxlength="255" spellcheck="true" aria-required="true"> (This page has more on "aria-required", in case that may be of use: http://wiki.codetalks.org/wiki/index.php/Technique:_marking_form_fields_as_required_using_aria-required )
Comment on attachment 377701 [details] [diff] [review] v1 I don't want a background color on those required fields. I do want the field labels themselves colored, or if they're links, the star itself is enough. However, you must indicate what the star means, on the page itself.
Attachment #377701 - Flags: review?(mkanat) → review-
(In reply to comment #9) > (In reply to comment #8) > For accessibility's sake, perhaps it may also be worth making use of the > "aria-required" attribute on required form field(s)? Yes. Absolutely.
Attached patch v2Splinter Review
Okay, here we go, this adds a red star before the field name (which was a bit cleaner than adding it after), sets the background of required fields to a specific color, adds aria-required, and also adds a note that explains what the stars and background colors mean.
Assignee: ui → mkanat
Attachment #377701 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #412726 - Flags: review?(dkl)
Attached image Screenshot of V2
Great to see you're making use of aria-required here!
Comment on attachment 412726 [details] [diff] [review] v2 >Index: template/en/default/bug/create/create.html.tmpl > <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 Nit: indentation on a few lines got messed up here. Please correct on checkin. Otherwise everything looks good to me and works as expected. r=dkl
Attachment #412726 - Flags: review?(dkl) → review+
Flags: approval?
Flags: approval? → approval+
Thanks! Indentation fixed on checkin. Checking in skins/standard/global.css; /cvsroot/mozilla/webtools/bugzilla/skins/standard/global.css,v <-- global.css new revision: 1.70; previous revision: 1.69 done Checking in template/en/default/bug/create/create.html.tmpl; /cvsroot/mozilla/webtools/bugzilla/template/en/default/bug/create/create.html.tmpl,v <-- create.html.tmpl new revision: 1.101; previous revision: 1.100 done
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Keywords: relnote
Resolution: --- → FIXED
Added to the release notes in bug 547466.
Keywords: relnote
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: