ICEfaces
  1. ICEfaces
  2. ICE-10042

ace:autoCompleteEntry - Keyboard arrow key navigation issue in IE 11

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Minor Minor
          • Resolution: Fixed
          • Affects Version/s: 4.0.BETA, EE-3.3.0.GA_P02
          • Fix Version/s: EE-3.3.0.GA_P02, 4.0
          • Component/s: ACE-Components, Sample Apps
          • Labels:
            None
          • Environment:
            Tomcat 7, Icefaces 4 trunk, Icefaces ee-3.3.0 maintenance branch r 41200. ACE components and Showcase, IE 11 only.
          • Assignee Priority:
            P2

            Description

            IE 11 does not allow keyboard up/down arrow navigation in autoCompleteEntry demos. Functions properly otherwise. No errors occur. Not an issue in any other browsers.

            To reproduce:
            Build/Deploy showcase
            Navigate to ace:autoCompleteEntry > Overview.
            Type a letter into the input field.
            Attempt to use up or down keyboard arrows to navigate through the list.

              Activity

              Ascending order - Click to sort in descending order
              Liana Munroe created issue -
              Ken Fyten made changes -
              Field Original Value New Value
              Assignee Arturo Zambrano [ artzambrano ]
              Fix Version/s EE-3.3.0.GA_P02 [ 11371 ]
              Fix Version/s 4.0 [ 11382 ]
              Affects Version/s 4.0.BETA [ 10770 ]
              Affects Version/s EE-3.3.0.GA_P02 [ 11371 ]
              Priority Major [ 3 ] Minor [ 4 ]
              Assignee Priority P2 [ 10011 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #41209 Mon May 26 18:02:32 MDT 2014 art.zambrano ICE-10042 modified browser detection to correctly identify IE11 and thus register the keypress handler on the keydown event, as with the other IE browsers, which is necessary to obtain correct key codes; removed special IE11 handling in isCharacterCode() function, since with this fix all key codes are the same as with other browsers
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/autocompleteentry/autocompleteentry.js
              Hide
              Permalink
              Arturo Zambrano added a comment -

              Committed fix to 4.0 trunk at revision 41209 and to 3.3 EE maintenance branch at revision 41208. The problem was that IE11 wasn't being detected correctly, and the main event handler was being registered on the keyrpess event, while it should have been the keydown event, as with the other IE browsers. This was necessary so that special keys, such as the arrow keys, could be registered by the browser and fire the event handler. This change also fixes the original issue in ICE-10032, where the key codes in the event object were different. It seems that in IE and Webkit browsers the key codes are slightly different between the keydown and keypress events. So, the fix committed for ICE-10032 was reverted, since it's no longer necessary.

              Show
              Arturo Zambrano added a comment - Committed fix to 4.0 trunk at revision 41209 and to 3.3 EE maintenance branch at revision 41208. The problem was that IE11 wasn't being detected correctly, and the main event handler was being registered on the keyrpess event, while it should have been the keydown event, as with the other IE browsers. This was necessary so that special keys, such as the arrow keys, could be registered by the browser and fire the event handler. This change also fixes the original issue in ICE-10032, where the key codes in the event object were different. It seems that in IE and Webkit browsers the key codes are slightly different between the keydown and keypress events. So, the fix committed for ICE-10032 was reverted, since it's no longer necessary.
              Arturo Zambrano made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              Liana Munroe added a comment -

              Confirmed fixed Icefaces ee-3.3.0 maintenance branch and Icefaces 4 trunk r 41219.Tomcat 7. IE 11

              Show
              Liana Munroe added a comment - Confirmed fixed Icefaces ee-3.3.0 maintenance branch and Icefaces 4 trunk r 41219.Tomcat 7. IE 11
              Ken Fyten made changes -
              Security Private [ 10001 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Arturo Zambrano
                  Reporter:
                  Liana Munroe
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: