Selecting grid row causes unexpected scrolling

[guest]

Steps to reproduce:

1. Open e.g. dojox/grid/tests/test_edit.html (also other tests have the problem)
2. Drag the scrollbar about halfway down
3. Click on a row somewhere in the middle of the visible area

Bug: The selected row jumps to the top/bottom of visible area, with some flickering.

Environment:

• FF2, win xp
• SVN HEAD, currently R11920.

[sorvell]

(In [12060]) !strict addresses #5469

[sorvell]

Should be addressed by [12060].

When a selection is made, we no longer try to focus the cell in the old selection before focusing the the cell in the new selection. This was causing bogus scrolling.

[guest]

I think that this is still a problem in IE, but not in Firefox. I can't reproduce it with the dojox/grid/tests/test_edit.html file that the original reporter used, but I can reproduce it with the file that I attached (index.html).

Open this file in IE and click on any of the cells. The browser will immediately scroll so that the grid is at the top of the browser and the text above (the word "Title") will scroll off. It does not seem to do this in Firefox.

[guest]

grid edit example in small screen

[guest]

There's more cases where clicking on grid causes unexpected scrolling. The problem can be seen also in grid/tests/test_edit.html It also happens in a grid where one area (cell) is scrollable.

I've seen the problem in IE7 only, Firefox works fine. Using SVN head r13358, win xp.

First resize and scroll test_edit.html as show in grid_test_edit_screenshot.png Now, clicking almost anywhere in grid causes unexpected scrolling. - Clicking e.g. "message" column causes the horizontal scrollbar to jump left - Clicking in rightmost "status" column causes jump to the right - Double clicking in rightmost "amount" or "status" column activates editor, but the scrollbar jumps and the editor is only partially visible. - etc

[BryanForbes]

(In [13793]) refs #6353 !strict

• Moved dojox.grid.VirtualGrid? to dojox.grid._Grid.
• Renamed DataGrid?._getItem to DataGrid?.getItem.
• Added "defaultValue" property to cells and set it to '...'.
• Added _Grid._fetch() (stub function).
• Changed _Grid to work with asyncronous stores more efficiently.

refs #6545

• Changed "itemAttr" on cells to "field".
• Added "sortDesc" attribute to cells.

fixes #6746, #5469

• Fixed horizontal scrolling issue in FF2/3.