minor fixes

lib/frappe-datatable.js

@@ -626,7 +626,7 @@ var DEFAULT_OPTIONS = {
   addCheckboxColumn: true,
   enableClusterize: true,
   enableLogs: false,
-  takeAvailableSpace: true
+  takeAvailableSpace: false
 };
 
 var DataTable = function () {
@@ -809,6 +809,16 @@ var DataTable = function () {
 
       _dom2.default.style(this.bodyScrollable, { width: width + 'px' });
     }
+  }, {
+    key: 'getColumn',
+    value: function getColumn(colIndex) {
+      return this.datamanager.getColumn(colIndex);
+    }
+  }, {
+    key: 'getCell',
+    value: function getCell(colIndex, rowIndex) {
+      return this.datamanager.getCell(colIndex, rowIndex);
+    }
   }, {
     key: 'getColumnHeaderElement',
     value: function getColumnHeaderElement(colIndex) {
@@ -1393,6 +1403,9 @@ var CellManager = function () {
     value: function focusCell($cell) {
       if (!$cell) return;
 
+      // don't focus if already editing cell
+      if ($cell === this.$editingCell) return;
+
       var _$$data3 = _dom2.default.data($cell),
           colIndex = _$$data3.colIndex,
           isHeader = _$$data3.isHeader;
@@ -2291,6 +2304,7 @@ var RowManager = function () {
     this.instance = instance;
     this.options = this.instance.options;
     this.wrapper = this.instance.wrapper;
+    this.bodyScrollable = this.instance.bodyScrollable;
     this.datamanager = this.instance.datamanager;
 
     this.bindEvents();

src/cellmanager.js

@@ -175,6 +175,9 @@ export default class CellManager {
   focusCell($cell) {
     if (!$cell) return;
 
+    // don't focus if already editing cell
+    if ($cell === this.$editingCell) return;
+
     const { colIndex, isHeader } = $.data($cell);
 
     if (this.isStandardCell(colIndex) || isHeader) {

src/datatable.js

@@ -24,7 +24,7 @@ const DEFAULT_OPTIONS = {
   addCheckboxColumn: true,
   enableClusterize: true,
   enableLogs: false,
-  takeAvailableSpace: true
+  takeAvailableSpace: false
 };
 
 export default class DataTable {
@@ -207,6 +207,14 @@ export default class DataTable {
     $.style(this.bodyScrollable, { width: width + 'px' });
   }
 
+  getColumn(colIndex) {
+    return this.datamanager.getColumn(colIndex);
+  }
+
+  getCell(colIndex, rowIndex) {
+    return this.datamanager.getCell(colIndex, rowIndex);
+  }
+
   getColumnHeaderElement(colIndex) {
     return this.columnmanager.getColumnHeaderElement(colIndex);
   }

src/rowmanager.js

@@ -5,6 +5,7 @@ export default class RowManager {
     this.instance = instance;
     this.options = this.instance.options;
     this.wrapper = this.instance.wrapper;
+    this.bodyScrollable = this.instance.bodyScrollable;
     this.datamanager = this.instance.datamanager;
 
     this.bindEvents();