Uploaded image for project: 'Click'
  1. Click
  2. CLK-577

The <label for=".."> for a RadioGroup in a Form references an non-existing id.

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.0 RC1
    • Fix Version/s: 2.2.0
    • Component/s: None
    • Labels:
      None

      Description

      Using the example in the API docs for RadioGroup,

      radioGroup.add(new Radio("STD", "Standard "));
      radioGroup.add(new Radio("PRO", "Protective "));
      radioGroup.add(new Radio("GFT", "Gift Wrap "));
      radioGroup.setValue("STD");
      radioGroup.setVerticalLayout(true);
      form.add(radioGroup);

      a <label for="form_packaging">Packaging</label> is generated, but the radiogroup only creates individual radio buttons with id like form_packaging_STD, form_packaging_PRO and form_packaging_GFT.

      <td class="fields" align="left"><label for="form_packaging">Packaging</label> </td>
      <td align="left">
      <input type="radio" name="packaging" value="STD" id="form_packaging_STD" checked="checked"/><label for="form_packaging_STD">Standard </label><br/>
      <input type="radio" name="packaging" value="PRO" id="form_packaging_PRO"/><label for="form_packaging_PRO">Protective </label><br/>
      <input type="radio" name="packaging" value="GFT" id="form_packaging_GFT"/><label for="form_packaging_GFT">Gift Wrap </label>
      </td>

      I would guess that wrapping the list of Radio controls in a <div id="form_packaging">..</div> would solve the issue.

        Attachments

          Activity

            People

            • Assignee:
              sabob Bob Schellink
              Reporter:
              bckfnn Finn Bock
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: