always show the correct sketchbook at ide start-up

Author: Alberto Iannaccone

Diff for: arduino-ide-extension/src/browser/arduino-ide-frontend-module.ts

@@ -420,9 +420,8 @@ export default new ContainerModule((bind, unbind, isBound, rebind) => {
   bind(SerialService)
     .toDynamicValue((context) => {
       const connection = context.container.get(WebSocketConnectionProvider);
-      const client = context.container.get<SerialServiceClient>(
-        SerialServiceClient
-      );
+      const client =
+        context.container.get<SerialServiceClient>(SerialServiceClient);
       return connection.createProxy(SerialServicePath, client);
     })
     .inSingletonScope();
@@ -486,11 +485,12 @@ export default new ContainerModule((bind, unbind, isBound, rebind) => {
     .inSingletonScope();
   rebind(TheiaEditorWidgetFactory).to(EditorWidgetFactory).inSingletonScope();
   rebind(TabBarToolbarFactory).toFactory(
-    ({ container: parentContainer }) => () => {
-      const container = parentContainer.createChild();
-      container.bind(TabBarToolbar).toSelf().inSingletonScope();
-      return container.get(TabBarToolbar);
-    }
+    ({ container: parentContainer }) =>
+      () => {
+        const container = parentContainer.createChild();
+        container.bind(TabBarToolbar).toSelf().inSingletonScope();
+        return container.get(TabBarToolbar);
+      }
   );
   bind(OutputWidget).toSelf().inSingletonScope();
   rebind(TheiaOutputWidget).toService(OutputWidget);
@@ -655,15 +655,13 @@ export default new ContainerModule((bind, unbind, isBound, rebind) => {
 
   // Enable the dirty indicator on uncloseable widgets.
   rebind(TabBarRendererFactory).toFactory((context) => () => {
-    const contextMenuRenderer = context.container.get<ContextMenuRenderer>(
-      ContextMenuRenderer
-    );
+    const contextMenuRenderer =
+      context.container.get<ContextMenuRenderer>(ContextMenuRenderer);
     const decoratorService = context.container.get<TabBarDecoratorService>(
       TabBarDecoratorService
     );
-    const iconThemeService = context.container.get<IconThemeService>(
-      IconThemeService
-    );
+    const iconThemeService =
+      context.container.get<IconThemeService>(IconThemeService);
     return new TabBarRenderer(
       contextMenuRenderer,
       decoratorService,
@@ -723,7 +721,7 @@ export default new ContainerModule((bind, unbind, isBound, rebind) => {
   bindViewContribution(bind, SketchbookWidgetContribution);
   bind(FrontendApplicationContribution).toService(SketchbookWidgetContribution);
   bind(WidgetFactory).toDynamicValue(({ container }) => ({
-    id: 'arduino-sketchbook-widget',
+    id: SketchbookWidget.ID,
     createWidget: () => container.get(SketchbookWidget),
   }));
 

Diff for: arduino-ide-extension/src/browser/theia/core/application-shell.ts

@@ -16,6 +16,7 @@ import { Sketch } from '../../../common/protocol';
 import { SaveAsSketch } from '../../contributions/save-as-sketch';
 import { SketchesServiceClientImpl } from '../../../common/protocol/sketches-service-client-impl';
 import { nls } from '@theia/core/lib/common';
+import { SketchbookWidget } from '../../widgets/sketchbook/sketchbook-widget';
 
 @injectable()
 export class ApplicationShell extends TheiaApplicationShell {
@@ -31,6 +32,18 @@ export class ApplicationShell extends TheiaApplicationShell {
   @inject(ConnectionStatusService)
   protected readonly connectionStatusService: ConnectionStatusService;
 
+  async setLayoutData(
+    layoutData: TheiaApplicationShell.LayoutData
+  ): Promise<void> {
+    layoutData.activeWidgetId = SketchbookWidget.ID;
+    layoutData.leftPanel?.items?.forEach((item) => {
+      if (item?.widget?.id === SketchbookWidget.ID) item.expanded = true;
+      else item.expanded = false;
+    });
+
+    super.setLayoutData(layoutData);
+  }
+
   protected track(widget: Widget): void {
     super.track(widget);
     if (widget instanceof OutputWidget) {

Diff for: arduino-ide-extension/src/browser/theia/core/shell-layout-restorer.ts

@@ -1,15 +1,19 @@
 import { injectable } from 'inversify';
-import { FrontendApplication } from '@theia/core/lib/browser/frontend-application';
 import { ShellLayoutRestorer as TheiaShellLayoutRestorer } from '@theia/core/lib/browser/shell/shell-layout-restorer';
+import { inject } from '@theia/core/shared/inversify';
+import { ApplicationShell } from '@theia/core/lib/browser/shell/application-shell';
 
 @injectable()
 export class ShellLayoutRestorer extends TheiaShellLayoutRestorer {
+  @inject(ApplicationShell)
+  protected readonly shell: ApplicationShell;
+
   // Workaround for https://github.com/eclipse-theia/theia/issues/6579.
-  async storeLayoutAsync(app: FrontendApplication): Promise<void> {
+  async storeLayoutAsync(): Promise<void> {
     if (this.shouldStoreLayout) {
       try {
         this.logger.info('>>> Storing the layout...');
-        const layoutData = app.shell.getLayoutData();
+        const layoutData = this.shell.getLayoutData();
         const serializedLayoutData = this.deflate(layoutData);
         await this.storageService.setData(
           this.storageKey,
@@ -23,7 +27,7 @@ export class ShellLayoutRestorer extends TheiaShellLayoutRestorer {
     }
   }
 
-  async restoreLayout(app: FrontendApplication): Promise<boolean> {
+  async restoreLayout(): Promise<boolean> {
     this.logger.info('>>> Restoring the layout state...');
     const serializedLayoutData = await this.storageService.getData<string>(
       this.storageKey
@@ -49,7 +53,7 @@ export class ShellLayoutRestorer extends TheiaShellLayoutRestorer {
       });
     }
 
-    await app.shell.setLayoutData(layoutData);
+    await this.shell.setLayoutData(layoutData);
     this.logger.info('<<< The layout has been successfully restored.');
     return true;
   }

Diff for: arduino-ide-extension/src/browser/widgets/sketchbook/sketchbook-commands.ts

@@ -32,4 +32,8 @@ export namespace SketchbookCommands {
     id: 'arduino-sketchbook--show-files',
     label: 'Contextual menu',
   };
+
+  export const SKETCHBOOK_TOGGLE_VIEW: Command = {
+    id: 'arduino-sketchbook-widget:toggle',
+  };
 }

Diff for: arduino-ide-extension/src/browser/widgets/sketchbook/sketchbook-widget-contribution.ts

@@ -68,13 +68,13 @@ export class SketchbookWidgetContribution
 
   constructor() {
     super({
-      widgetId: 'arduino-sketchbook-widget',
+      widgetId: SketchbookWidget.ID,
       widgetName: SketchbookWidget.LABEL,
       defaultWidgetOptions: {
         area: 'left',
         rank: 1,
       },
-      toggleCommandId: 'arduino-sketchbook-widget:toggle',
+      toggleCommandId: SketchbookCommands.SKETCHBOOK_TOGGLE_VIEW.id,
       toggleKeybinding: 'CtrlCmd+Shift+B',
     });
   }
@@ -191,7 +191,7 @@ export class SketchbookWidgetContribution
 
     // unregister main menu action
     registry.unregisterMenuAction({
-      commandId: 'arduino-sketchbook-widget:toggle',
+      commandId: SketchbookCommands.SKETCHBOOK_TOGGLE_VIEW.id,
     });
 
     registry.registerMenuAction(SKETCHBOOK__CONTEXT__MAIN_GROUP, {

Diff for: arduino-ide-extension/src/browser/widgets/sketchbook/sketchbook-widget.tsx

@@ -11,6 +11,7 @@ import { nls } from '@theia/core/lib/common';
 @injectable()
 export class SketchbookWidget extends BaseWidget {
   static LABEL = nls.localize('arduino/sketch/titleSketchbook', 'Sketchbook');
+  static ID = 'arduino-sketchbook-widget';
 
   @inject(SketchbookTreeWidget)
   protected readonly localSketchbookTreeWidget: SketchbookTreeWidget;
@@ -19,7 +20,7 @@ export class SketchbookWidget extends BaseWidget {
 
   constructor() {
     super();
-    this.id = 'arduino-sketchbook-widget';
+    this.id = SketchbookWidget.ID;
     this.title.caption = SketchbookWidget.LABEL;
     this.title.label = SketchbookWidget.LABEL;
     this.title.iconClass = 'fa fa-arduino-folder';