Merge pull request #731 from metasfresh/dev-725

Sidelist closes when doubleclick #725
metasfresh · May 11, 2017 · 1dd8642 · 1dd8642
2 parents 8277471 + 1f63040
commit 1dd8642
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 11 deletions.
diff --git a/src/components/app/DocumentList.js b/src/components/app/DocumentList.js
@@ -396,7 +396,7 @@ class DocumentList extends Component {
 
     redirectToDocument = (id) => {
         const {
-            dispatch, isModal, windowType, isSideListShow, closeSideList
+            dispatch, isModal, windowType, isSideListShow
         } = this.props;
         const {page, viewId, sort} = this.state;
 
@@ -406,9 +406,7 @@ class DocumentList extends Component {
 
         dispatch(push('/window/' + windowType + '/' + id));
 
-        if(isSideListShow) {
-            closeSideList();
-        }else{
+        if(!isSideListShow){
             // Caching last settings
             dispatch(setPagination(page, windowType));
             dispatch(setSorting(sort, windowType));

diff --git a/src/components/table/Table.js b/src/components/table/Table.js
@@ -321,7 +321,7 @@ class Table extends Component {
         const {selected, rows} = this.state;
 
         const {
-            onDoubleClick, closeOverlays, open
+            onDoubleClick, closeOverlays
         } = this.props;
 
         const selectRange = e.shiftKey;
@@ -377,14 +377,10 @@ class Table extends Component {
                    onDoubleClick(selected[selected.length-1]);
                 }
 
-                if(open) {
-                    closeOverlays();
-                }
                 break;
             case 'Escape':
-                if(open){
-                    closeOverlays();
-                }
+                closeOverlays();
+
                 break;
             case 'Tab': {
                 e.preventDefault();