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)
Bugzilla
User Interface
Tracking
()
RESOLVED
FIXED
Bugzilla 3.6
People
(Reporter: myk, Assigned: mkanat)
References
Details
Attachments
(2 files, 1 obsolete file)
2.92 KB,
patch
|
dkl
:
review+
|
Details | Diff | Splinter Review |
72.60 KB,
image/png
|
Details |
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.
Comment 1•22 years ago
|
||
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.
Reporter | ||
Updated•21 years ago
|
Severity: normal → minor
Target Milestone: --- → Bugzilla 2.22
Comment 2•20 years ago
|
||
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!
Comment 4•19 years ago
|
||
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: "(*)" }
![]() |
||
Updated•18 years ago
|
Assignee: myk → ui
Comment 5•18 years ago
|
||
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.
Comment 6•18 years ago
|
||
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.
Assignee | ||
Comment 7•16 years ago
|
||
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
Assignee | ||
Updated•16 years ago
|
Blocks: bz-hci2008
Comment 8•16 years ago
|
||
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)
Comment 9•16 years ago
|
||
(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 )
Assignee | ||
Comment 10•16 years ago
|
||
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-
Comment 11•16 years ago
|
||
(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.
Assignee | ||
Comment 12•16 years ago
|
||
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)
Assignee | ||
Comment 13•16 years ago
|
||
Comment 14•16 years ago
|
||
Great to see you're making use of aria-required here!
Comment 15•16 years ago
|
||
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+
Updated•16 years ago
|
Flags: approval?
Assignee | ||
Updated•16 years ago
|
Flags: approval? → approval+
Assignee | ||
Comment 16•16 years ago
|
||
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
You need to log in
before you can comment on or make changes to this bug.
Description
•