ICEfaces
  1. ICEfaces
  2. ICE-10741

ace:expansionToggler keyboard activation issues

    Details

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

      Description

      When focus is set on the ace:expansionToggler component via the keyboard / Tab key, and space is pressed to toggle its state, the state toggles as expected, however, focus is then lost to the dataTable row instead of the originating toggle element. Ideally, focus would remain on the toggler such that space could be pressed again to toggle the state indefinitely.

      In addition, when toggled/activated via the Space bar, it doesn't seem to consume the space key event, resulting in the browser scrolling down a page in response to the space key being pressed.

        Activity

        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #45823 Wed Aug 05 17:21:47 MDT 2015 art.zambrano ICE-10741 replaced dots for dashes in the ace:expansionToggler client id, in order to prevent the request from being aborted
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/expansiontoggler/ExpansionTogglerRenderer.java
        Hide
        Liana Munroe added a comment -

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

        Show
        Liana Munroe added a comment - Verified ICEfaces 4 trunk r45251, Tomcat 7, IE 11, 10, 9, 8, FF 34 Chrome 43.
        Arturo Zambrano made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Arturo Zambrano added a comment -

        r45104: committed fix to persist focus in ace:expansionToggler button as well as consuming the event when activated via the space button.

        Show
        Arturo Zambrano added a comment - r45104: committed fix to persist focus in ace:expansionToggler button as well as consuming the event when activated via the space button.
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #45104 Thu Jun 25 11:03:48 MDT 2015 art.zambrano ICE-10741, ICE-10742 fixes to persist focus in ace:expansionToggler and ace:rowEditor buttons as well as consuming the event when activated via the space button
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/expansiontoggler/ExpansionTogglerRenderer.java
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/datatable/datatable.js
        Ken Fyten made changes -
        Description The ace:expansionToggler component does not display a focus rectangle when it has focus, though it appears to be focussable via the Tab-key.

        In addition, when toggled/activated via the Space bar, it doesn't seem to consume the space key event, resulting in the browser scrolling down a page in response to the space key being pressed.
        When focus is set on the ace:expansionToggler component via the keyboard / Tab key, and space is pressed to toggle its state, the state toggles as expected, however, focus is then lost to the dataTable row instead of the originating toggle element. Ideally, focus would remain on the toggler such that space could be pressed again to toggle the state indefinitely.

        In addition, when toggled/activated via the Space bar, it doesn't seem to consume the space key event, resulting in the browser scrolling down a page in response to the space key being pressed.
        Ken Fyten made changes -
        Summary ace:expansionToggler not showing focus indicator or consuming space keypresses ace:expansionToggler keyboard activation issues
        Ken Fyten made changes -
        Fix Version/s 4.1 [ 11375 ]
        Fix Version/s 4.0 [ 11382 ]
        Ken Fyten made changes -
        Field Original Value New Value
        Assignee Arturo Zambrano [ artzambrano ]
        Fix Version/s 4.0 [ 11382 ]
        Priority Major [ 3 ] Minor [ 4 ]
        Assignee Priority P2 [ 10011 ]
        Ken Fyten created issue -

          People

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

            Dates

            • Created:
              Updated:
              Resolved: