Opened 11 years ago
Closed 8 years ago
#11867 closed defect (patchwelcome)
FocusManager tries to focus a node that is not visible
Reported by: | tanneman | Owned by: | Bryan Forbes |
---|---|---|---|
Priority: | high | Milestone: | tbd |
Component: | DojoX Grid | Version: | 1.5 |
Keywords: | FocusManager DataGrid | Cc: | |
Blocked By: | Blocking: |
Description
The call this.grid.domNode.focus() in _FocusManager._delayedHeaderFocus can result in an error in IE (Can't move focus to the control because it is invisible).
I have attached an html file that exposes the bug. The steps to reproduce are in the attached html file. Just click on the buttons :).
Steps to reproduce
- Create a DataGrid? with selectable set tot true.
- Fill the table with rows and make sure you get a vertical scrollbar.
- Move the focus on one of the column header nodes (click on it)
- Hide the table (set style display to none)
- fill the table (or update the rowCount)
Expected
- No problems
Actual
- A script error occurs (on IE 6-8)
Attachments (2)
Change History (6)
Changed 11 years ago by
Attachment: | expose-bug.html added |
---|
Changed 11 years ago by
Attachment: | FocusManagerPatch.js added |
---|
comment:1 Changed 10 years ago by
Owner: | changed from bryanforbes to Bryan Forbes |
---|
comment:2 Changed 10 years ago by
comment:3 Changed 8 years ago by
comment:4 Changed 8 years ago by
Resolution: | → patchwelcome |
---|---|
Status: | new → closed |
Note: See
TracTickets for help on using
tickets.
(In [25473]) Refs #11867 add a safe check as - if(!cell){return} to avoid exceptions when cell is undefined, backport to 1.6/1.5 branches