ICEfaces
  1. ICEfaces
  2. ICE-4069

Focus is lost on selectOneRadio and selectManyCheckbox with spread=true on IE

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Invalid
    • Affects Version/s: 1.8RC1
    • Fix Version/s: 1.8RC2, 1.8
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      IE, selectOneRadio or selectManyCheckbox with spread=true.

      Description


      On IE only, when selecting a radio button or checkbox that is in spread layout, the focus is lost on the component.

      To reproduce:

         1. Run the regression test app. for ICE-3916 on IE7.
         2. Use the tab key to move focus to the radio button or checkbox controls at the top of the page.
         3. Use the space bar key to select the focussed component. Notice how the focus indicator is no longer around the component.
         4. Press tab again, and focus appears to be returned to the original component again. Strange...

        Issue Links

          Activity

          Ken Fyten created issue -
          Ken Fyten made changes -
          Field Original Value New Value
          Link This issue depends on ICE-3720 [ ICE-3720 ]
          Ken Fyten made changes -
          Salesforce Case []
          Fix Version/s 1.8 [ 10161 ]
          Security Private [ 10001 ]
          Priority Major [ 3 ] Minor [ 4 ]
          Ken Fyten made changes -
          Salesforce Case []
          Assignee Priority P2
          Assignee Yip Ng [ yip.ng ]
          Ken Fyten made changes -
          Salesforce Case []
          Security Private [ 10001 ]
          yip.ng made changes -
          Attachment ScreenHunter_02 Feb. 19 17.42.jpg [ 11534 ]
          Ken Fyten made changes -
          Comment [ Got a runtime error when tabbing in the test app. See attached screenshot.
          ]
          Hide
          yip.ng added a comment -

          This has nothing to do with spread = true or not. The component showcase example has the same behavior without using spread. The focus hasn't been lost. It has been passed from the label to the radio button or checkbox itself.

          This is actually the intended behavior specified by W3C. We use the <label for="..."> to specify the label so that users can click on the label to change the state. The click is actually passed through to the radio button or checkbox. The same thing happens with focus. According to the HTML spec:

          "When a LABEL element receives focus, it passes the focus on to its associated control."

          I have tried adding the onfocus event handling to the label like other controls. Didn't seem to work. The onfocus event didn't seem to be recognized at all by the label.

          Show
          yip.ng added a comment - This has nothing to do with spread = true or not. The component showcase example has the same behavior without using spread. The focus hasn't been lost. It has been passed from the label to the radio button or checkbox itself. This is actually the intended behavior specified by W3C. We use the <label for="..."> to specify the label so that users can click on the label to change the state. The click is actually passed through to the radio button or checkbox. The same thing happens with focus. According to the HTML spec: "When a LABEL element receives focus, it passes the focus on to its associated control." I have tried adding the onfocus event handling to the label like other controls. Didn't seem to work. The onfocus event didn't seem to be recognized at all by the label.
          Hide
          Ken Fyten added a comment -

          This looks like a none-issue.

          Show
          Ken Fyten added a comment - This looks like a none-issue.
          Ken Fyten made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Assignee Priority P2
          Resolution Invalid [ 6 ]
          Ken Fyten made changes -
          Fix Version/s 1.8RC2 [ 10163 ]
          Fix Version/s 1.8 [ 10161 ]
          Ken Fyten made changes -
          Fix Version/s 1.8 [ 10161 ]
          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Assignee Yip Ng [ yip.ng ]

            People

            • Assignee:
              Unassigned
              Reporter:
              Ken Fyten
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: