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

              Ascending order - Click to sort in descending order
              Ken Fyten created issue -
              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 made changes -
              Fix Version/s 4.1 [ 11375 ]
              Fix Version/s 4.0 [ 11382 ]
              Ken Fyten made changes -
              Summary ace:expansionToggler not showing focus indicator or consuming space keypresses ace:expansionToggler keyboard activation issues
              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.
              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
              Hide
              Permalink
              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.
              Arturo Zambrano made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              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.
              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
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: