diff options
| author | justdave%bugzilla.org <> | 2004-05-12 08:27:23 +0200 |
|---|---|---|
| committer | justdave%bugzilla.org <> | 2004-05-12 08:27:23 +0200 |
| commit | b9b79e598793821895d26412fe2eaa6e08f3c336 (patch) | |
| tree | 1b8a091749b13b4d351ca0cb6644be6ced36c0bb | |
| parent | a83e476438d44ebe967a115d8d7685b832e7a9c8 (diff) | |
| download | bugzilla-b9b79e598793821895d26412fe2eaa6e08f3c336.tar.gz bugzilla-b9b79e598793821895d26412fe2eaa6e08f3c336.tar.xz | |
Bug 241270: fix regression caused by <label> wrapping other form widgets. Related widgets are now excluded from the <label> blocks.
r=myk, a=justdave
| -rw-r--r-- | template/en/default/bug/knob.html.tmpl | 45 |
1 files changed, 24 insertions, 21 deletions
diff --git a/template/en/default/bug/knob.html.tmpl b/template/en/default/bug/knob.html.tmpl index 4156391d0..f0194155d 100644 --- a/template/en/default/bug/knob.html.tmpl +++ b/template/en/default/bug/knob.html.tmpl @@ -71,23 +71,23 @@ <label for="knob-resolve"> Resolve [% terms.bug %], changing <a href="page.cgi?id=fields.html#resolution">resolution</a> to - <select name="resolution" - onchange="document.changeform.knob[[% knum %]].checked=true"> + </label> + <select name="resolution" + onchange="document.changeform.knob[[% knum %]].checked=true"> [% FOREACH r = bug.choices.resolution %] <option value="[% r FILTER html %]">[% r FILTER html %]</option> [% END %] - </select> - </label> + </select> <br> [% knum = knum + 1 %] <input type="radio" id="knob-duplicate" name="knob" value="duplicate"> <label for="knob-duplicate"> Resolve [% terms.bug %], mark it as duplicate of [% terms.bug %] # - <input name="dup_id" size="6" - onchange="if (this.value != '') - {document.changeform.knob[[% knum %]].checked=true}"> </label> + <input name="dup_id" size="6" + onchange="if (this.value != '') + {document.changeform.knob[[% knum %]].checked=true}"> <br> [% knum = knum + 1 %] @@ -95,19 +95,20 @@ <label for="knob-reassign"> <a href="page.cgi?id=fields.html#assigned_to">Reassign</a> [% terms.bug %] to - <input name="assigned_to" size="32" - onchange="if ((this.value != '[% bug.assigned_to.email FILTER js %]') && - (this.value != '')) { - document.changeform.knob[[% knum %]].checked=true; - }" - value="[% bug.assigned_to.email FILTER html %]"> - <br> - [% IF bug.isunconfirmed && bug.user.canconfirm %] - <input type="checkbox" name="andconfirm"> - and confirm [% terms.bug %] (change status to <b>NEW</b>) - <br> - [% END %] </label> + <input name="assigned_to" size="32" + onchange="if ((this.value != '[% bug.assigned_to.email FILTER js %]') && + (this.value != '')) { + document.changeform.knob[[% knum %]].checked=true; + }" + value="[% bug.assigned_to.email FILTER html %]"> + <br> + [% IF bug.isunconfirmed && bug.user.canconfirm %] + <input type="checkbox" id="andconfirm" name="andconfirm"> + <label for="andconfirm"> + and confirm [% terms.bug %] (change status to <b>NEW</b>) + <br> + [% END %] [% knum = knum + 1 %] <input type="radio" id="knob-reassign-cmp" name="knob" value="reassignbycomponent"> @@ -117,8 +118,10 @@ of selected component <br> [% IF bug.isunconfirmed && bug.user.canconfirm %] - <input type="checkbox" name="compconfirm"> - and confirm [% terms.bug %] (change status to <b>NEW</b>) + <input type="checkbox" id="compconfirm" name="compconfirm"> + <label for="compconfirm"> + and confirm [% terms.bug %] (change status to <b>NEW</b>) + </label> <br> [% END %] </label> |