DataGrid Header taking focus from other elements when refreshed

[leelofgren]

If a header column in dojox.grid.DataGrid? had focus before leaving the grid for another element, then that other element triggered a refresh of the DataGrid?, the DataGrid? will pull focus back. Environment: dojo 1.4.1, firefox 3.6.2 on Windows 7. Suspected cause: DataGrid?.focus._colHeadFocusIdx and DataGrid?.focus._colHeadNode are used to return focus to a head cell when columns are destroyed and re-created. the DataGrid? onBlur does not clear these variables so a refresh of the grid causes it to think that it was caused by the header and mistakenly returns focus. Workaround: If the grid doesn't have focus, set [DataGridObject?].focus._colHeadFocusIdx=null and [DataGridObject?].focus._colHeadNode=null before refreshing the grid. What I was doing: I wrote a web page where I have a search textbox that filters the DataGrid? through [DataGridObject?].setQuery() as letters are typed. If I sort a column before typing letters into my search box, the text box it looses focus to the grid header.

[leelofgren]

simple example file based on DataGrid? example from dojocampus.org

[leelofgren]

In sample page: If prior focus was not on a header row, you can type multiple letters into text box. Click any header row to sort, now you can only type 1 character before focus shifts back to header cell. Add grid4.focus._colHeadFocusIdx=null;grid4.focus._colHeadNode=null; to the refresh_grid function before the grid4.setQuery() and it will now work correctly.

[Sam Reynolds]

Proposed fix for problem

[Sam Reynolds]

I've attached a patch that appears to work for us. I would welcome any feedback.

Thanks, Sam

[fabioginzel]

+1

[bill]

DojoX Grid and EnhancedGrid are deprecated in favor of ​dgrid and ​gridx.

You should upgrade your code to use one of those two grids.

We will consider patches to the old DojoX Grid code though.