ICEfaces
  1. ICEfaces
  2. ICE-10740

ace:dataTable Paginator buttons not keyboard navigable/focussable

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 4.0
    • Fix Version/s: 4.1
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      All
    • Assignee Priority:
      P2

      Description

      The ace:dataTable pagination buttons / select list cannot currently be navigated to via the keyboard (tab key). They do support using the PgUp/PgDn/Home/End keys within the datatable to change the currently viewed page (correctly). However, these controls should also be in the browser tab order such that focus could be moved to them via the Tab-key. Also, focus should persist on the selected control when activated via the keyboard (space when focussed).

      Also note that the space key event should be consumed by the button when activated to avoid having the browser scroll the page when space is pressed, etc.

        Activity

        Hide
        Arturo Zambrano added a comment -

        r45093: allowed paginator buttons to receive focus and to persist this focus; also added event to activate buttons when pressing the space key.

        Show
        Arturo Zambrano added a comment - r45093: allowed paginator buttons to receive focus and to persist this focus; also added event to activate buttons when pressing the space key.
        Hide
        Liana Munroe added a comment -

        Tested with ICEfaces 4 trunk r45093.
        Verified Chrome 43. After tabbing to a paginator control either space bar or Enter key can be used to activate it.
        IE 11, 10, 9, 8 - After tabbing to a paginator control only the space bar can be used to activate it, not the Enter Key as with Chrome.
        FF 38, 34, 24.5. - Using the Tab key does not place focus on any of the paginator controls except for the "first page" control. The focussed area also includes the part of the header or footer facet to the left of that control.

        Show
        Liana Munroe added a comment - Tested with ICEfaces 4 trunk r45093. Verified Chrome 43. After tabbing to a paginator control either space bar or Enter key can be used to activate it. IE 11, 10, 9, 8 - After tabbing to a paginator control only the space bar can be used to activate it, not the Enter Key as with Chrome. FF 38, 34, 24.5. - Using the Tab key does not place focus on any of the paginator controls except for the "first page" control. The focussed area also includes the part of the header or footer facet to the left of that control.
        Hide
        Ken Fyten added a comment -

        Re-opened for FF oddities.

        Show
        Ken Fyten added a comment - Re-opened for FF oddities.
        Hide
        Arturo Zambrano added a comment -

        r45258: fix to activate controls with the Enter key on IE and with the space key on FF; fix to be able to give focus to the controls on FF and to only outline the control itself.

        Show
        Arturo Zambrano added a comment - r45258: fix to activate controls with the Enter key on IE and with the space key on FF; fix to be able to give focus to the controls on FF and to only outline the control itself.
        Hide
        Liana Munroe added a comment -

        Tested with ICEfaces 4 trunk r45261, IE 11, 10, 8, FF 34, Chrome 43.
        In all browsers the behaviour of the paginator when used with Tab, Enter and Space bar is verified but there is now a noticeable change in the alignment of the paginator page buttons and the FFWD/reverse buttons. See attached screen shot 1.PNG

        Show
        Liana Munroe added a comment - Tested with ICEfaces 4 trunk r45261, IE 11, 10, 8, FF 34, Chrome 43. In all browsers the behaviour of the paginator when used with Tab, Enter and Space bar is verified but there is now a noticeable change in the alignment of the paginator page buttons and the FFWD/reverse buttons. See attached screen shot 1.PNG
        Hide
        Arturo Zambrano added a comment -

        r45577: committed fix to re-align buttons.

        Removing the text from the icon spans caused this misalignment, which has now been fixed via CSS.

        Show
        Arturo Zambrano added a comment - r45577: committed fix to re-align buttons. Removing the text from the icon spans caused this misalignment, which has now been fixed via CSS.
        Hide
        Liana Munroe added a comment -

        Verified ICEfaces 4 trunk r45580, Tomcat 7, IE 11, 10, 9, 8, FF 34, Chrome 43.

        Show
        Liana Munroe added a comment - Verified ICEfaces 4 trunk r45580, Tomcat 7, IE 11, 10, 9, 8, FF 34, Chrome 43.

          People

          • Assignee:
            Arturo Zambrano
            Reporter:
            Ken Fyten
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: