Thank you for reporting this bug. We will make it our priority to review this report.
  1. #1
    Ext User
    Join Date
    Aug 2008
    Posts
    2
    Vote Rating
    0
    nascom is on a distinguished road

      0  

    Question [TENT-112][3.0.0] Grid tab key navigation

    [TENT-112][3.0.0] Grid tab key navigation


    Hi,

    were experiencing the Problem that in Firefox the Editor Grid Tab Order is not behaving the way it should.

    To Reproduce the Problem:

    Go to http://www.extjs.com/deploy/dev/exam...edit-grid.html.
    Click "Add Plant".
    In the "Common Name"-Column type whatever you want.
    Press TAB.
    Now in the "Light"-Column delete all the text by holding BACKSPACE.
    When the Column is empty type for Example: "s" (for Shade) and press TAB.

    Voila! You've just selected the edit-grid.js Link.

    This isn't supposed to happen, right?

    Note: This only happens with Firefox.

  2. #2
    Ext User
    Join Date
    Jul 2007
    Location
    Florida
    Posts
    9,996
    Vote Rating
    6
    mjlecomte will become famous soon enough mjlecomte will become famous soon enough

      0  

    Default


    Thanks for the report. keyboard navigation in the grid is being assessed.

  3. #3
    Ext User
    Join Date
    Aug 2008
    Posts
    2
    Vote Rating
    0
    nascom is on a distinguished road

      0  

    Question


    Quote Originally Posted by mjlecomte View Post
    Thanks for the report. keyboard navigation in the grid is being assessed.
    For us Keyboard Navigation is the most important thing.
    Is there any Hotfix or Workaround for this Issue?

  4. #4
    Ext User
    Join Date
    Jul 2007
    Location
    Florida
    Posts
    9,996
    Vote Rating
    6
    mjlecomte will become famous soon enough mjlecomte will become famous soon enough

      0  

    Default


    Quote Originally Posted by nascom View Post
    For us Keyboard Navigation is the most important thing.
    Is there any Hotfix or Workaround for this Issue?
    See http://extjs.com/forum/showthread.php?t=71015, including #4

  5. #5
    Sencha - Ext JS Dev Team evant's Avatar
    Join Date
    Apr 2007
    Location
    Sydney, Australia
    Posts
    17,004
    Vote Rating
    650
    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


    A fix for this has been added to SVN.
    Evan Trimboli
    Sencha Developer
    Twitter - @evantrimboli
    Don't be afraid of the source code!

  6. #6
    Sencha - Community Support Team mystix's Avatar
    Join Date
    Mar 2007
    Location
    Singapore
    Posts
    6,236
    Vote Rating
    5
    mystix will become famous soon enough

      0  

    Default


    Quote Originally Posted by evant View Post
    A fix for this has been added to SVN.
    just checked out SVN HEAD.
    the KeyNav#stopKeyUp() method was omitted from both the 2.x and 3.x branches.

  7. #7
    Sencha - Community Support Team mystix's Avatar
    Join Date
    Mar 2007
    Location
    Singapore
    Posts
    6,236
    Vote Rating
    5
    mystix will become famous soon enough

      0  

    Default


    additionally, in the 2.x branch (SVN HEAD)
    • the Ext.Editor#onRender() method is missing a check for this.swallowKeys:
      Code:
              if(this.swallowKeys){
                  this.field.el.swallowEvent([
                      'keypress', // *** Opera
                      'keydown'   // *** all other browsers
                  ]);
              }
    • the Ext.form.Field#initEvents() method needs to use the the new Ext.EventManager.useKeyDown flag:
      (it would also help if the original comment for the 10ms buffer was re-added to the codebase for both the 2.x and 3.x branches)
      Code:
          initEvents : function(){
              // this.el.on(Ext.isIE || (Ext.isWebKit && !Ext.isSafari2) ? "keydown" : "keypress", this.fireKey,  this);
              this.el.on(Ext.EventManager.useKeydown ? 'keydown' : 'keypress', this.fireKey, this);
              this.el.on("focus", this.onFocus,  this);
      
              // standardise buffer across all browsers + OS-es for consistent event order.
              // (the 10ms buffer for Editors fixes a weird FF/Win editor issue when changing OS window focus)
              this.el.on('blur', this.onBlur, this, this.inEditor ? {buffer:10} : null);
          },

  8. #8
    Sencha - Community Support Team mystix's Avatar
    Join Date
    Mar 2007
    Location
    Singapore
    Posts
    6,236
    Vote Rating
    5
    mystix will become famous soon enough

      0  

    Default


    spotted one more:

    there's a now defunct Ext.grid.GridPanel#onKeyDown() method in the 2.x branch.
    this should be removed to prevent any possible confusion and keep the 2.x / 3.x changes in sync.