Looks like we can't reproduce the issue or there's a problem in the test case provided.
  1. #1
    Sencha User
    Join Date
    Oct 2012
    Posts
    6
    Vote Rating
    0
    Xeno is on a distinguished road

      0  

    Default [4.1.1] Ext.menu.KeyNav continues to swallow key events after clicking an item

    [4.1.1] Ext.menu.KeyNav continues to swallow key events after clicking an item


    REQUIRED INFORMATION


    Ext version tested:
    • Ext 4.1.1
    Browser versions tested against:
    • Bug present in
      • Chrome 23.0.1271.17 beta-m
      • Chrome 22.0.1229.79 m
      • Chrome 22.0.1229.92 m
    • Bug not present in
      • Firefox 15.01
      • Internet Explorer 9
    Description:

    When you dismiss a menu by clicking on a menu item, the KeyNav object tied to the menu continues to swallow key events. When dismissing the menu by clicking on the modal mask DIV this effect does not occur.

    Steps to reproduce the problem:
    • Go to http://jsfiddle.net/6sDeP/17/
    • Repeatedly press the 'up' arrow key
    • Note that the events are displayed for both keyUp and keyDown
    • Right click to open the Ext context menu.
    • Dismiss the menu by clicking a menu item
    • Repeatedly press the 'up' arrow key
    The result that was expected:
    • The events are displayed for both keyUp and keyDown
    The result that occurs instead:
    • Only the keyUp event appears to be handled
    Debugging already done:
    • A breakpoint in the Ext.menu.KeyNav "up" method is hit when the menu is no longer visible
    • Setting "enableKeyNav" to false for the menu fixes the problem
    Additional CSS used:
    • only default ext-all.css
    Operating System:
    • Windows 7

  2. #2
    Sencha - Senior Forum Manager mitchellsimoens's Avatar
    Join Date
    Mar 2007
    Location
    Gainesville, FL
    Posts
    37,647
    Vote Rating
    899
    mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute

      0  

    Default


    Looks like you are mixing jQuery in there, have you tried a Ext JS only test? Also, not able to reproduce using Chrome 22 (stable)
    Mitchell Simoens @SenchaMitch
    Sencha Inc, Senior Forum Manager
    ________________
    Check out my GitHub, lots of nice things for Ext JS 4 and Sencha Touch 2
    https://github.com/mitchellsimoens

    Think my support is good? Get more personalized support via a support subscription. https://www.sencha.com/store/

    Need more help with your app? Hire Sencha Services services@sencha.com

    Want to learn Sencha Touch 2? Check out Sencha Touch in Action that is in print!

    When posting code, please use BBCode's CODE tags.

  3. #3
    Sencha User
    Join Date
    Oct 2012
    Posts
    6
    Vote Rating
    0
    Xeno is on a distinguished road

      0  

    Default


    I have updated the JSFiddle in the original post with an all Ext version.

    Also I can reproduce the bug on Chrome 22.0.1229.79 m, and have added this to the original post.

  4. #4
    Sencha - Senior Forum Manager mitchellsimoens's Avatar
    Join Date
    Mar 2007
    Location
    Gainesville, FL
    Posts
    37,647
    Vote Rating
    899
    mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute

      0  

    Default


    I still cannot reproduce. I press up 5 times and I see 'keyUp - 9'. I then right-click to get the context menu and click on the first item. I then press up once and I then see 'keyUp - 11' as the keyDown was 10
    Mitchell Simoens @SenchaMitch
    Sencha Inc, Senior Forum Manager
    ________________
    Check out my GitHub, lots of nice things for Ext JS 4 and Sencha Touch 2
    https://github.com/mitchellsimoens

    Think my support is good? Get more personalized support via a support subscription. https://www.sencha.com/store/

    Need more help with your app? Hire Sencha Services services@sencha.com

    Want to learn Sencha Touch 2? Check out Sencha Touch in Action that is in print!

    When posting code, please use BBCode's CODE tags.

  5. #5
    Sencha User
    Join Date
    Oct 2012
    Posts
    6
    Vote Rating
    0
    Xeno is on a distinguished road

      0  

    Default


    I have added another version of Chrome we have confirmed the bug with

    We have also discovered that if you click on the "Result" window after clicking a menu item the problem does not occur. This means that the steps are now
    • Right click to bring up the context menu
    • Click on the menu item
    • Do not click the mouse again
    • Press the 'up' key again.

  6. #6
    Sencha User mberrie's Avatar
    Join Date
    Feb 2011
    Location
    Bangkok, Thailand
    Posts
    506
    Vote Rating
    14
    mberrie will become famous soon enough mberrie will become famous soon enough

      0  

    Default


    I can confirm the reported behavior on Chrome 22.0.1229.94 m on Win7/x64.

  7. #7
    Sencha - Ext JS Dev Team Animal's Avatar
    Join Date
    Mar 2007
    Location
    Notts/Redwood City
    Posts
    30,546
    Vote Rating
    64
    Animal is a jewel in the rough Animal is a jewel in the rough Animal is a jewel in the rough

      0  

    Default


    This is fixed in the latest code, and the problem should not occur in the upcoming releases.

  8. #8
    Sencha - Ext JS Dev Team evant's Avatar
    Join Date
    Apr 2007
    Location
    Sydney, Australia
    Posts
    17,167
    Vote Rating
    674
    evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute

      0  

    Default


    Verified this in the latest nightly, following the steps in post 5, both events are still handled after clicking the menu item.
    Evan Trimboli
    Sencha Developer
    Twitter - @evantrimboli
    Don't be afraid of the source code!