ICEfaces
  1. ICEfaces
  2. ICE-4833

Incorrect tab order with ice:selectInputDate popup when used with time entry

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 1.8.1
    • Fix Version/s: 1.8.2-RC1, 1.8.2
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      ICEfaces 1.8.1, ice:selectInputText comp used with time entry as well as date.

      Description

      When using the ice:selectInputText component to enter time values (as well as date) the tab order within the calendar popup is incorrect. Specifically, when tabbing into the popup from the calendar button, it first focusses the time-entry fields on the bottom of the popup, then goes to the calendar at the top. It should always go to the calendar at the top first and then the days of the month, and finally the time entry fields at the bottom. Here's a video illustrating this issue:

      http://screencast.com/t/diB5wsNPOeq

      Note that fixing the tab order will be a little tricky as the last focusable element before the time entry fields will differ depending on which month is displayed.

        Activity

        Hide
        Adnan Durrani added a comment -

        Fixed:

        Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\selectinputdate\SelectInputDateRenderer.java
        Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\selectinputdate\SelectInputDateRenderer.java
        Completed: At revision: 19168

        Show
        Adnan Durrani added a comment - Fixed: Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\selectinputdate\SelectInputDateRenderer.java Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\selectinputdate\SelectInputDateRenderer.java Completed: At revision: 19168
        Hide
        Joanne Bai added a comment -

        Fix verified with success on trunk revision 19173 using component showcase popup calendar

        Tested on FF3.5, IE7

        Show
        Joanne Bai added a comment - Fix verified with success on trunk revision 19173 using component showcase popup calendar Tested on FF3.5, IE7

          People

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

            Dates

            • Created:
              Updated:
              Resolved: