#8811 closed defect (fixed)
[PATCH][CCLA]: IE Specific: Selecting blank area of a grid and hitting arrow key to nav throws error
Reported by: | Jared Jurkiewicz | Owned by: | Bryan Forbes |
---|---|---|---|
Priority: | high | Milestone: | 1.3 |
Component: | DojoX Grid | Version: | 1.3.0b2 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description (last modified by )
This is visible with current test files: dojox/grid/tests/test_data_grid_empty.html
Click in the blank area under the header.
Hit an arrow key
IE throws error: this.cell.index is null or not an object
Doesn't occur on other browsers.
Attachments (1)
Change History (5)
comment:1 Changed 12 years ago by
Description: | modified (diff) |
---|
Changed 12 years ago by
Attachment: | _FocusManager.patch added |
---|
comment:2 Changed 12 years ago by
Summary: | IE Specific: Selecting blank area of a grid and hitting arrow key to nav throws error → [PATCH][CCLA]: IE Specific: Selecting blank area of a grid and hitting arrow key to nav throws error |
---|
Added a patch that just wraps several places in _FocusManager with a check on this.cell before it tries to make use of it. This avoids the error seen in IE when the grid has focus but no cell was selected, then an arrow key is hit. The grid simply doesn't try to do anything now (no cell selected).
Let me know if this is good enough and I'll commit it.
comment:3 Changed 12 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Fixed in changeset: http://bugs.dojotoolkit.org/changeset/16942
comment:4 Changed 12 years ago by
Milestone: | tbd → 1.3 |
---|
batch move of tickets marked 'tbd' fixed in the 1.3 timeframe
Tentative patch that adds in checks on this.cell before it is used in several functions.