Merge pull request quantopian#65 from blink1073/fix-remove-row

Fix remove row behavior
jdavidheiser · Mar 6, 2016 · e1190ae · e1190ae
2 parents 1b8defb + 85e4a2a
commit e1190ae
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/qgrid/qgridjs/qgrid.widget.js b/qgrid/qgridjs/qgrid.widget.js
@@ -142,7 +142,7 @@ define([path + "widgets/js/widget", path + "widgets/js/manager"], function(widge
             sgrid.onCellChange.subscribe(function(e, args) {
                 var column = columns[args.cell].name;
                 var id = args.grid.getDataItem(args.row).slick_grid_id;
-                var row = Number(id.replace('row', ''))
+                var row = Number(id.replace('row', ''));
                 var msg = {'row': row, 'column': column,
                            'value': args.item[column], 'type': 'cell_change'};
                 that.send(msg);
@@ -173,7 +173,8 @@ define([path + "widgets/js/widget", path + "widgets/js/manager"], function(widge
                 }
                 var data = sgrid.getData().getItem(cell.row);
                 grid.data_view.deleteItem(data.slick_grid_id);
-                msg = {'type': 'remove_row', 'row': cell.row, 'id': data.id};
+                var row = Number(data.slick_grid_id.replace('row', ''));
+                msg = {'type': 'remove_row', 'row': row, 'id': data.id};
                 this.updateSize();
                 this.send(msg);