ICEfaces
  1. ICEfaces
  2. ICE-1466

Clicking on label does not select SelectManyCheckbox in IE

          Details

          • Type: Improvement Improvement
          • Status: Closed
          • Priority: Minor Minor
          • Resolution: Fixed
          • Affects Version/s: 1.6DR#3
          • Fix Version/s: 1.7.1
          • Component/s: None
          • Labels:
            None
          • Environment:
            any

            Description

            If you click on the rendered label of a SelectManyCheckbox selectitem, the item should be selected. This works in Firefox, but not IE. This is because we render the label outside of the input element, instead of the input containing the label:

            <label class="iceCheckbox">
                <input type="checkbox" value="value1" onclick="setFocus('');iceSubmitPartial(form, this, event);" name="gropupName" id="groupName:_1"/>Segment</label>

            instead of

            <input type="checkbox" value="value1" onclick="setFocus('');iceSubmitPartial(form, this, event);" name="groupId" id="groupId:_1"/>
             <label class="iceCheckbox" for="groupId:_1">
             Segment</label>

              Activity

              Hide
              Permalink
              Philip Breau added a comment -

              rev 16700 (trunk) and 16701( 1.7)

              • render label with for attribute
              • remove nesting of input in label (html & xhtml compliance)
              • make selectManyCheckbox label selection work in IE6
              Show
              Philip Breau added a comment - rev 16700 (trunk) and 16701( 1.7) render label with for attribute remove nesting of input in label (html & xhtml compliance) make selectManyCheckbox label selection work in IE6

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  Philip Breau
                • Votes:
                  1 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: