Bug 241270: fix regression caused by <label> wrapping other form widgets. …

Bug 241270: fix regression caused by <label> wrapping other form widgets. Related widgets are now excluded from the <label> blocks. r=myk, a=justdave

Bug 241270: fix regression caused by <label> wrapping other form widgets. …
b9b79e59 · justdave%bugzilla.org · a83e4764 · b9b79e59
Commit b9b79e59 authored May 12, 2004 by justdave%bugzilla.org
Show whitespace changes
Inline Side-by-side

Showing with 8 additions and 5 deletions

knob.html.tmpl template/en/default/bug/knob.html.tmpl +8 -5

No files found.
--- 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
+      </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>
      <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 %] #
+      </label>
      <input name="dup_id" size="6"
             onchange="if (this.value != '')
                       {document.changeform.knob[[% knum %]].checked=true}">
-      </label>
      <br>
      [% knum = knum + 1 %]

@@ -95,6 +95,7 @@
      <label for="knob-reassign">
        <a href="page.cgi?id=fields.html#assigned_to">Reassign</a> 
        [% terms.bug %] to
+      </label>
      <input name="assigned_to" size="32"
             onchange="if ((this.value != '[% bug.assigned_to.email FILTER js %]') &&
                            (this.value != '')) {
@@ -103,11 +104,11 @@
             value="[% bug.assigned_to.email FILTER html %]">
      <br>
      [% IF bug.isunconfirmed && bug.user.canconfirm %]
-          &nbsp;&nbsp;&nbsp;&nbsp;<input type="checkbox" name="andconfirm">
+        &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 %]
-      </label>
      [% 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">
+          &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>