Merge pull request #1245 from Michael-Hanley/feature/KeyUpSort

Adds onKeyUp event to header for sort

Author: AllenFang

packages/react-bootstrap-table2/src/header-cell.js

@@ -77,6 +77,12 @@ class HeaderCell extends eventDelegater(React.Component) {
 
     if (sort) {
       const customClick = cellAttrs.onClick;
+      cellAttrs.onKeyUp = (e) => {
+        if (e.key === 'Enter') {
+          onSort(column);
+          if (_.isFunction(customClick)) customClick(e);
+        }
+      };
       cellAttrs.onClick = (e) => {
         onSort(column);
         if (_.isFunction(customClick)) customClick(e);

packages/react-bootstrap-table2/test/header-cell.test.js

@@ -394,11 +394,25 @@ describe('HeaderCell', () => {
       expect(wrapper.find('th').prop('onClick')).toBeDefined();
     });
 
+    it('should have onKeyUp event on header cell', () => {
+      expect(wrapper.find('th').prop('onKeyUp')).toBeDefined();
+    });
+
     it('should trigger onSort callback when click on header cell', () => {
       wrapper.find('th').simulate('click');
       expect(onSortCallBack.callCount).toBe(1);
     });
 
+    it('should trigger onSort callback when keyup Enter on header cell', () => {
+      wrapper.find('th').simulate('keyup', { key: 'Enter' });
+      expect(onSortCallBack.callCount).toBe(1);
+    });
+
+    it('should not trigger onSort callback when keyup key is not Enter on header cell', () => {
+      wrapper.find('th').simulate('keyup', { key: 'test-key' });
+      expect(onSortCallBack.callCount).toBe(0);
+    });
+
     describe('and sorting prop is false', () => {
       it('header should render SortSymbol as default', () => {
         expect(wrapper.find(SortSymbol).length).toBe(1);