summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorjustdave%bugzilla.org <>2004-05-12 08:27:23 +0200
committerjustdave%bugzilla.org <>2004-05-12 08:27:23 +0200
commitb9b79e598793821895d26412fe2eaa6e08f3c336 (patch)
tree1b8a091749b13b4d351ca0cb6644be6ced36c0bb
parenta83e476438d44ebe967a115d8d7685b832e7a9c8 (diff)
downloadbugzilla-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.tmpl45
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 %]
- &nbsp;&nbsp;&nbsp;&nbsp;<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 %]
+ &nbsp;&nbsp;&nbsp;&nbsp;<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 %]
- &nbsp;&nbsp;&nbsp;&nbsp;<input type="checkbox" name="compconfirm">
- and confirm [% terms.bug %] (change status to <b>NEW</b>)
+ &nbsp;&nbsp;&nbsp;&nbsp;<input type="checkbox" id="compconfirm" name="compconfirm">
+ <label for="compconfirm">
+ and confirm [% terms.bug %] (change status to <b>NEW</b>)
+ </label>
<br>
[% END %]
</label>