From 6e174012965742fb80851b2749847a5bdcf00d2c Mon Sep 17 00:00:00 2001
From: tanmoyopenroot <tanmoy.openroot@gmail.com>
Date: Sat, 3 Aug 2019 18:10:28 +0530
Subject: [PATCH] use tabindex prop in <Hotkeys and add test

---
 packages/table/src/cell/editableCell.tsx  |  4 +++-
 packages/table/test/editableCellTests.tsx | 18 +++++++++++++++++-
 2 files changed, 20 insertions(+), 2 deletions(-)

diff --git a/packages/table/src/cell/editableCell.tsx b/packages/table/src/cell/editableCell.tsx
index 99e0427973..3b388e91ba 100644
--- a/packages/table/src/cell/editableCell.tsx
+++ b/packages/table/src/cell/editableCell.tsx
@@ -190,8 +190,10 @@ export class EditableCell extends React.Component<IEditableCellProps, IEditableC
     }
 
     public renderHotkeys() {
+        const { tabIndex } = this.props;
+
         return (
-            <Hotkeys>
+            <Hotkeys tabIndex={tabIndex}>
                 <Hotkey
                     key="edit-cell"
                     label="Edit the currently focused cell"
diff --git a/packages/table/test/editableCellTests.tsx b/packages/table/test/editableCellTests.tsx
index 3d03e72886..75d4a0bf9f 100644
--- a/packages/table/test/editableCellTests.tsx
+++ b/packages/table/test/editableCellTests.tsx
@@ -21,7 +21,7 @@ import * as sinon from "sinon";
 
 import { Classes } from "@blueprintjs/core";
 import * as TableClasses from "../src/common/classes";
-import { EditableCell } from "../src/index";
+import { Cell, EditableCell } from "../src/index";
 import { CellType, expectCellLoading } from "./cellTestUtils";
 
 describe("<EditableCell>", () => {
@@ -35,6 +35,22 @@ describe("<EditableCell>", () => {
         expectCellLoading(editableCellHarness.first().getDOMNode(), CellType.BODY_CELL);
     });
 
+    it("renders cell with default tabIndex as zero", () => {
+        const tabIndex = 0;
+        const elem = mount(<EditableCell value="test-value-5000" />);
+        const cellInstance = elem.find(Cell).instance() as Cell;
+
+        expect(cellInstance.props.tabIndex).to.equal(tabIndex);
+    });
+
+    it("renders cell with tabIndex", () => {
+        const tabIndex = 1;
+        const elem = mount(<EditableCell tabIndex={tabIndex} value="test-value-5000" />);
+        const cellInstance = elem.find(Cell).instance() as Cell;
+
+        expect(cellInstance.props.tabIndex).to.equal(tabIndex);
+    });
+
     it("renders new value if props.value changes", () => {
         const VALUE_1 = "foo";
         const VALUE_2 = "bar";