stops opening a pending tab if current tab is in review mode already

app/assets/javascripts/ide/components/commit_sidebar/list_item.vue

@@ -23,8 +23,10 @@ export default {
   methods: {
     ...mapActions(['discardFileChanges', 'updateViewer', 'openPendingTab']),
     openFileInEditor(file) {
-      return this.updateViewer('diff').then(() => {
-        this.openPendingTab(file);
+      return this.openPendingTab(file).then(changeViewer => {
+        if (changeViewer) {
+          this.updateViewer('diff');
+        }
       });
     },
   },

app/assets/javascripts/ide/stores/actions/file.js

@@ -138,12 +138,18 @@ export const discardFileChanges = ({ state, commit }, path) => {
   eventHub.$emit(`editor.update.model.content.${file.path}`, file.raw);
 };
 
-export const openPendingTab = ({ commit, dispatch, state }, file) => {
+export const openPendingTab = ({ commit, getters, dispatch, state }, file) => {
+  if (getters.activeFile && getters.activeFile.path === file.path && state.viewer === 'diff') {
+    return false;
+  }
+
   commit(types.ADD_PENDING_TAB, file);
 
   dispatch('scrollToTab');
 
   router.push(`/project/${file.projectId}/tree/${state.currentBranchId}/`);
+
+  return true;
 };
 
 export const removePendingTab = ({ commit }, file) => {