962812 - Use the "required" attribute where appropriate

Status: RESOLVED FIXED

(Bugzilla :: Bugzilla-General, enhancement)

Description

[Frédéric Buclin]

There are many fields which are required, e.g. when creating a new product, component, version, target milestone, flag, etc... The "name" field immediately comes to mind, but also the "description" field and some other fields such as the default assignee when creating a new component. For these fields, we should add the "required" HTML5 attribute so that the browser can notify the user before submission.

Comment 1

[Sunil Joshi [:sjoshi]]

Attachment: Patch-v1

Hope I have covered all of them. :).

Comment 2

[:glob ✱]

Comment on attachment 8365222 [details] [diff] [review]
Patch-v1

LpSolit said he's happy to review this.

Comment 3

[Frédéric Buclin]

Comment on attachment 8365222 [details] [diff] [review]
Patch-v1

>=== modified file 'template/en/default/account/prefs/account.html.tmpl'

>-        <input type="password" name="old_password">
>+        <input type="password" name="old_password" required>

The password is currently not required to change your real name. This may change in the future, but for now, it's optional.



>=== modified file 'template/en/default/admin/flag-type/edit.html.tmpl'

The sortkey is required too (that's silly, because it should fall back to 0, but it's required).



>=== modified file 'template/en/default/admin/keywords/create.html.tmpl'
>=== modified file 'template/en/default/admin/keywords/edit.html.tmpl'

The description is also required.



>=== modified file 'template/en/default/admin/users/userdata.html.tmpl'

>+             value="" autocomplete="off" [% 'required' IF !editform %] >

Remove the extra whitespace before >.



>=== modified file 'template/en/default/global/userselect.html.tmpl'

>+  # mandatory: optional, if its a mandatory

Write this: optional; if true, the field cannot be empty.


I didn't check if we fixed all the places, but this looks good. :)

Comment 4

[Sunil Joshi [:sjoshi]]

Attachment: Patch-v2

Review comments incorporated.

Comment 5

[Frédéric Buclin]

Comment on attachment 8388074 [details] [diff] [review]
Patch-v2

>=== modified file 'template/en/default/account/email/confirm-new.html.tmpl'

>-        <input type="password" id="passwd1" name="passwd1" value="">
>+        <input type="password" id="passwd1" name="passwd1" value="" required>

Nit: there are several places where the useless value="" is set. It could be safely removed, but leaving it alone is fine.



>=== modified file 'template/en/default/admin/keywords/create.html.tmpl'

>           wrap    = 'virtual'
>+          mandatory      = 1

The indentation of = 1 is a bit weird.


r=LpSolit

Comment 6

[Frédéric Buclin]

To ssh://[e-mail]/bugzilla/bugzilla.git
   d9cd470..165689a  master -> master