Fixes close tabs to the left/right

when you have multiple tab pages Resolves #9789 Auditors: @bsclifton Test Plan:
brave · Jul 14, 2017 · 96d3a14 · 96d3a14
1 parent 2cdb625
commit 96d3a14
Show file tree

Hide file tree

Showing 4 changed files with 17,927 additions and 12 deletions.
diff --git a/app/renderer/reducers/frameReducer.js b/app/renderer/reducers/frameReducer.js
@@ -186,21 +186,33 @@ const frameReducer = (state, action, immutableAction) => {
       })
       break
     case windowConstants.WINDOW_CLOSE_OTHER_FRAMES:
-      const currentIndex = frameStateUtil.getIndexByTabId(state, action.tabId)
-      if (currentIndex === -1) {
-        return
-      }
+      {
+        const currentIndex = frameStateUtil.getIndexByTabId(state, action.tabId)
+        if (currentIndex === -1) {
+          break
+        }
 
-      state.get('frames').forEach((frame, i) => {
-        if (!frame.get('pinnedLocation') &&
-          ((i < currentIndex && action.isCloseLeft) || (i > currentIndex && action.isCloseRight))) {
-          if (frame) {
-            appActions.tabCloseRequested(frame.get('tabId'))
+        let tabs = []
+
+        state.get('frames').forEach((frame, i) => {
+          if (!frame.get('pinnedLocation') &&
+            ((i < currentIndex && action.isCloseLeft) || (i > currentIndex && action.isCloseRight))) {
+            if (frame) {
+              tabs.push(frame.get('tabId'))
+              appActions.tabCloseRequested(frame.get('tabId'))
+            }
           }
-        }
-      })
+        })
 
+        // TODO(nejc) this can be simplified when states are merged
+        const newFrames = state.get('frames').filter(frame => !tabs.includes(frame.get('tabId')))
+        let newState = state.set('frames', newFrames)
+        newState = frameStateUtil.updateTabPageIndex(newState, action.tabId)
+        const index = newState.getIn(['ui', 'tabs', 'tabPageIndex'], 0)
+        state = state.setIn(['ui', 'tabs', 'tabPageIndex'], index)
+      }
       break
+
     case windowConstants.WINDOW_CLOSE_FRAME:
       state = closeFrame(state, action)
       break