Merge pull request #154 from TechnologyAdvice/feature/table-sortable-…

…class Automatic sortable table class
Semantic-Org · Jan 28, 2016 · 8480aad · 8480aad
2 parents ab635b9 + 5988e56
commit 8480aad
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 0 deletions.
diff --git a/src/collections/Table/Table.js b/src/collections/Table/Table.js
@@ -82,6 +82,7 @@ export default class Table extends Component {
     const classes = classNames(
       'sd-table',
       'ui',
+      {sortable: !!this.props.onSortChange},
       this.props.className,
       'table'
     );

diff --git a/test/specs/collections/Table/Table-test.js b/test/specs/collections/Table/Table-test.js
@@ -106,4 +106,22 @@ describe('Table', () => {
         });
     });
   });
+
+  describe('sortable', () => {
+    describe('with onSortChange prop', () => {
+      it('has class "sortable"', () => {
+        render(<Table data={tableData} onSortChange={_.noop} />)
+          .findClass('sortable');
+      });
+    });
+
+    describe('without onSortChange prop', () => {
+      it('does not have class "sortable"', () => {
+        render(<Table data={tableData} />)
+          .findClass('sd-table')
+          .props.className
+          .should.not.include('sortable');
+      });
+    });
+  });
 });