[cla] [patch] dojox.Grid loses scroll state if columns are re-ordered

[Karl Tiedt]

One of our teams has reported this issue with a very simple fix.

Basically if you go to any grid page, scroll down in the grid, and then start to drag a column header to re-order them... even if you cancel it... the grid will re-render and lose scroll state.

Attached is a patch again 1.7.2 just to show the fix they have been using.

Evan, do you see reason why this wouldn't be a good fix to make?

[Karl Tiedt]

patch against 1.7.2 (just to illustrate the simple fix)

[Evan]

It's a good fix, working nicely on all browsers. My understanding is the major diff between grid.render() and grid.update() is whether the scrollTop will be retained.

[Evan]

In [29050]:

Fixes #15557 !strict, change to grid.update() for retaining scrollTop after column moved, also applied to 1.7.x branch, thanks ktiedt!