Skip to content

Commit ae87910

Browse files
committed
fix for mleibman #329
1 parent 1ffcd5e commit ae87910

File tree

1 file changed

+8
-5
lines changed

1 file changed

+8
-5
lines changed

slick.grid.js

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2550,8 +2550,10 @@ if (typeof Slick === "undefined") {
25502550
if (e.isImmediatePropagationStopped()) {
25512551
return;
25522552
}
2553-
2554-
if ((activeCell != cell.cell || activeRow != cell.row) && canCellBeActive(cell.row, cell.cell)) {
2553+
2554+
// this optimisation causes trouble - MLeibman #329
2555+
//if ((activeCell != cell.cell || activeRow != cell.row) && canCellBeActive(cell.row, cell.cell)) {
2556+
if (canCellBeActive(cell.row, cell.cell)) {
25552557
if (!getEditorLock().isActive() || getEditorLock().commitCurrentEdit()) {
25562558
scrollRowIntoView(cell.row, false);
25572559
setActiveCellInternal(getCellNode(cell.row, cell.cell));
@@ -2776,10 +2778,11 @@ if (typeof Slick === "undefined") {
27762778
} else {
27772779
activeRow = activeCell = null;
27782780
}
2779-
2780-
if (activeCellChanged) {
2781+
2782+
// this optimisation causes trouble - MLeibman #329
2783+
//if (activeCellChanged) {
27812784
trigger(self.onActiveCellChanged, getActiveCell());
2782-
}
2785+
//}
27832786
}
27842787

27852788
function clearTextSelection() {

0 commit comments

Comments
 (0)