Merge pull request #53 from bcmi-labs/PROEDITOR-27

 PROEDITOR-27: Try to preserve the window.

Author: kittaakos

arduino-ide-electron/package.json

@@ -39,11 +39,11 @@
                     "editor.autoSave": "on"
                 }
             }
-        }
-    },
-    "generator": {
-        "config": {
-            "preloadTemplate": "<div class='theia-preload' style='background-color: rgb(237, 241, 242);'></div>"
+        },
+        "generator": {
+            "config": {
+                "preloadTemplate": "<div class='theia-preload' style='background-color: rgb(237, 241, 242);'></div>"
+            }
         }
     }
 }

arduino-ide-extension/src/browser/arduino-frontend-contribution.tsx

@@ -19,7 +19,7 @@ import { WorkspaceService } from '@theia/workspace/lib/browser/workspace-service
 import { SketchFactory } from './sketch-factory';
 import { ArduinoToolbar } from './toolbar/arduino-toolbar';
 import { EditorManager, EditorMainMenu } from '@theia/editor/lib/browser';
-import { ContextMenuRenderer, OpenerService, Widget, StatusBar, ShellLayoutRestorer, StatusBarAlignment } from '@theia/core/lib/browser';
+import { ContextMenuRenderer, OpenerService, Widget, StatusBar, ShellLayoutRestorer, StatusBarAlignment, LabelProvider } from '@theia/core/lib/browser';
 import { OpenFileDialogProps, FileDialogService } from '@theia/filesystem/lib/browser/file-dialog';
 import { FileSystem, FileStat } from '@theia/filesystem/lib/common';
 import { ArduinoToolbarContextMenu } from './arduino-file-menu';
@@ -113,6 +113,9 @@ export class ArduinoFrontendContribution implements TabBarToolbarContribution, C
     @inject(ShellLayoutRestorer)
     protected readonly layoutRestorer: ShellLayoutRestorer;
 
+    @inject(LabelProvider)
+    protected readonly labelProvider: LabelProvider;
+
     protected boardsToolbarItem: BoardsToolBarItem | null;
     protected wsSketchCount: number = 0;
 
@@ -375,9 +378,21 @@ export class ArduinoFrontendContribution implements TabBarToolbarContribution, C
     }
 
     protected async openSketchFilesInNewWindow(uri: string) {
-        const location = new URL(window.location.href);
-        location.searchParams.set('sketch', uri);
-        this.windowService.openNewWindow(location.toString());
+        const url = new URL(window.location.href);
+        const currentSketch = url.searchParams.get('sketch');
+        // Nothing to do if we want to open the same sketch which is already opened.
+        if (!!currentSketch && new URI(currentSketch).toString() === new URI(uri).toString()) {
+            this.messageService.info(`The '${this.labelProvider.getLongName(new URI(uri))}' is already opened.`);
+            // NOOP.
+            return;
+        }
+        // Preserve the current window if the `sketch` is not in the `searchParams`.
+        url.searchParams.set('sketch', uri);
+        if (!currentSketch) {
+            setTimeout(() => window.location.href = url.toString(), 100);
+            return;
+        }
+        this.windowService.openNewWindow(url.toString());
     }
 
     async openSketchFiles(uri: string) {

electron/packager/index.js

@@ -66,7 +66,7 @@
     //-------------------------------------------------------------------------------------------------+
     // Rebuild the extension with the copied `yarn.lock`. It is a must to use the same Theia versions. |
     //-------------------------------------------------------------------------------------------------+
-    exec(`yarn --cwd ${path('..', workingCopy)}`, 'Building the Arduino Theia extensions');
+    exec(`yarn --network-timeout 1000000 --cwd ${path('..', workingCopy)}`, 'Building the Arduino Theia extensions');
     // Collect all unused dependencies by the backend. We have to remove them from the electron app.
     // The `bundle.js` already contains everything we need for the frontend.
     // We have to do it before changing the dependencies to `local-path`.
@@ -98,8 +98,8 @@ ${fs.readFileSync(path('..', 'build', 'package.json')).toString()}
     //-------------------------------------------------------------------------------------------+
     // Install all private and public dependencies for the electron application and build Theia. |
     //-------------------------------------------------------------------------------------------+
-    exec(`yarn --cwd ${path('..', 'build')}`, 'Installing dependencies');
-    exec(`yarn --cwd ${path('..', 'build')} build${release ? ':release' : ''}`, 'Building the Arduino-PoC application');
+    exec(`yarn --network-timeout 1000000 --cwd ${path('..', 'build')}`, 'Installing dependencies');
+    exec(`yarn --network-timeout 1000000 --cwd ${path('..', 'build')} build${release ? ':release' : ''}`, 'Building the Arduino-PoC application');
 
     //------------------------------------------------------------------------------+
     // Create a throw away dotenv file which we use to feed the builder with input. |
@@ -115,7 +115,7 @@ ${fs.readFileSync(path('..', 'build', 'package.json')).toString()}
     //-----------------------------------+
     // Package the electron application. |
     //-----------------------------------+
-    exec(`yarn --cwd ${path('..', 'build')} package`, `Packaging your Arduino-PoC application`);
+    exec(`yarn --network-timeout 1000000 --cwd ${path('..', 'build')} package`, `Packaging your Arduino-PoC application`);
     echo(`🎉  Success. Your application is at: ${path('..', 'build', 'dist')}`);
 
     restore();