[table] fix wrapText on EditableCell component (#2701)

* fixes #2700: adds wrapText prop to Cell component * lint test * typo wrapsText should be wrapText in test descr * make selector more specific
palantir · Aug 1, 2018 · 2c419d4 · 2c419d4 · blueprint-bot · Aug 1, 2018
1 parent 9a9dd7d
commit 2c419d4
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/packages/table/src/cell/editableCell.tsx b/packages/table/src/cell/editableCell.tsx
@@ -136,6 +136,7 @@ export class EditableCell extends React.Component<IEditableCellProps, IEditableC
         return (
             <Cell
                 {...spreadableProps}
+                wrapText={wrapText}
                 truncated={false}
                 interactive={interactive}
                 cellRef={this.refHandlers.cell}

diff --git a/packages/table/test/editableCellTests.tsx b/packages/table/test/editableCellTests.tsx
@@ -126,4 +126,16 @@ describe("<EditableCell>", () => {
         elem.find("input").simulate("blur");
         expect(onChangeSpy.firstCall.args).to.deep.equal([CHANGED_VALUE, ROW_INDEX, COLUMN_INDEX]);
     });
+
+    it("defaults to no wrapText", () => {
+        const elem = mount(<EditableCell />);
+
+        expect(elem.find(`.${Classes.TABLE_NO_WRAP_TEXT}`).children().length).to.equal(1);
+    });
+
+    it("wraps text when wrapText is true", () => {
+        const elem = mount(<EditableCell wrapText={true} />);
+
+        expect(elem.find(`.${Classes.TABLE_NO_WRAP_TEXT}`).children().length).to.equal(0);
+    });
 });