Merge 664a2f7 into 484e2a6

microsoft · May 1, 2019 · 266efe2 · 266efe2
2 parents 484e2a6 + 664a2f7
commit 266efe2
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/packages/app/main/src/appMenuBuilder.spec.ts b/packages/app/main/src/appMenuBuilder.spec.ts
@@ -343,7 +343,7 @@ describe('AppMenuBuilder', () => {
     expect(editMenuTemplate).toHaveLength(7);
 
     const viewMenuTemplate = appMenuTemplate[2].submenu;
-    expect(viewMenuTemplate).toHaveLength(6);
+    expect(viewMenuTemplate).toHaveLength(7);
 
     const convoMenuTemplate = appMenuTemplate[3].submenu;
     expect(convoMenuTemplate).toHaveLength(1);
@@ -413,7 +413,7 @@ describe('AppMenuBuilder', () => {
 
   it('should initialize and update the debugMenu item', async () => {
     const viewMenu = await AppMenuBuilder.initViewMenu();
-    expect(viewMenu.submenu[5]).toEqual({
+    expect(viewMenu.submenu[6]).toEqual({
       checked: false,
       click: jasmine.any(Function),
       id: 'debugMode',
@@ -422,11 +422,11 @@ describe('AppMenuBuilder', () => {
     });
 
     mockGetApplicationMenu = () => ({
-      getMenuItemById: () => viewMenu.submenu[5],
+      getMenuItemById: () => viewMenu.submenu[6],
     });
 
     AppMenuBuilder.updateDebugModeViewMenuItem(DebugMode.Sidecar);
-    expect(viewMenu.submenu[5]).toEqual({
+    expect(viewMenu.submenu[6]).toEqual({
       checked: true,
       click: jasmine.any(Function),
       id: 'debugMode',

diff --git a/packages/app/main/src/appMenuBuilder.ts b/packages/app/main/src/appMenuBuilder.ts
@@ -437,6 +437,7 @@ export class AppMenuBuilder {
         { role: 'zoomout' },
         { type: 'separator' },
         { role: 'togglefullscreen' },
+        { role: 'toggledevtools' },
         {
           type: 'checkbox',
           checked: debugMode === DebugMode.Sidecar,