chore: Replace remote for hiding window (#652)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Afonso Jorge Ramos <[email protected]>

Author: 3 people

main.js

@@ -67,6 +67,7 @@ menubarApp.on('ready', () => {
   });
 
   ipcMain.on('reopen-window', () => menubarApp.showWindow());
+  ipcMain.on('hide-window', () => menubarApp.hideWindow());
   ipcMain.on('app-quit', () => menubarApp.app.quit());
   ipcMain.on('update-icon', (_, arg) => {
     if (!menubarApp.tray.isDestroyed()) {

src/utils/auth.ts

@@ -1,7 +1,6 @@
-import { generateGitHubAPIUrl } from './helpers';
-
-const { BrowserWindow } = require('@electron/remote');
+import { BrowserWindow } from '@electron/remote';
 
+import { generateGitHubAPIUrl } from './helpers';
 import { apiRequest, apiRequestAuth } from '../utils/api-requests';
 import { AuthResponse, AuthState, AuthTokenResponse } from '../types';
 import { Constants } from '../utils/constants';

src/utils/comms.test.ts

@@ -1,14 +1,12 @@
+import { ipcRenderer, shell } from 'electron';
+import remote from '@electron/remote';
 import {
   updateTrayIcon,
   openExternalLink,
   setAutoLaunch,
   restoreSetting,
 } from './comms';
 
-const { ipcRenderer, shell } = require('electron');
-
-const remote = require('@electron/remote');
-
 describe('utils/comms.ts', () => {
   beforeEach(function () {
     jest.spyOn(ipcRenderer, 'send');

src/utils/comms.ts

@@ -1,5 +1,5 @@
-const { ipcRenderer, shell } = require('electron');
-const remote = require('@electron/remote');
+import { ipcRenderer, shell } from 'electron';
+import remote from '@electron/remote';
 
 export function openExternalLink(url: string): void {
   shell.openExternal(url);

src/utils/notifications.ts

@@ -1,6 +1,4 @@
 import { ipcRenderer } from 'electron';
-const remote = require('@electron/remote');
-
 import { openInBrowser } from '../utils/helpers';
 import { updateTrayIcon } from './comms';
 import { Notification } from '../typesGithub';
@@ -84,7 +82,7 @@ export const raiseNativeNotification = (
 
   nativeNotification.onclick = function () {
     if (notifications.length === 1) {
-      remote.getCurrentWindow().hide();
+      ipcRenderer.send('hide-window');
       openInBrowser(notifications[0], accounts);
     } else {
       ipcRenderer.send('reopen-window');