GerHobbelt/SlickGrid — pull request #3 by beardedlinuxgeek (2014-04-28 23:33:58+00:00)
Title: RowSelectionModel:selectActiveRow is broken

slick.grid.js:setActiveCellInternal() used to call

trigger(self.onActiveCellChanged, getActiveCell());

but now it calls

trigger(self.onActiveCellChanged, {activeCell: getActiveCell(),prevActiveCell: prevActiveCell,editMode:opt_editMode,}, e);

which means the data sent to slick.rowselectionmodel.js:handleActiveCellChange() has changed from

Object {row: 4, cell: 4, grid: SlickGrid}

to

Object {activeCell: Object, prevActiveCell: null, editMode: false, grid: SlickGrid}

slick.grid.js:setActiveCellInternal() is manually adding add a "active-row" CSS class to the cell's parent (i.e. the row), but the row is never actually selected in the RowSelectionModel. This patch fixes that.