fix: show windows when the app is rendered

Signed-off-by: Grigorii K. Shartsev <[email protected]>

Author: ShGKme

src/authentication/authentication.window.js

@@ -8,7 +8,7 @@ const { BASE_TITLE, TITLE_BAR_HEIGHT } = require('../constants.js')
 const { applyContextMenu } = require('../app/applyContextMenu.js')
 const { getBrowserWindowIcon } = require('../shared/icons.utils.js')
 const { getAppConfig } = require('../app/AppConfig.ts')
-const { getScaledWindowSize } = require('../app/utils.ts')
+const { getScaledWindowSize, showWhenWindowMarkedReady } = require('../app/utils.ts')
 
 /**
  * @return {import('electron').BrowserWindow}
@@ -53,6 +53,8 @@ function createAuthenticationWindow() {
 
 	window.loadURL(AUTHENTICATION_WINDOW_WEBPACK_ENTRY)
 
+	showWhenWindowMarkedReady(window)
+
 	return window
 }
 

src/authentication/renderer/authentication.main.js

@@ -8,7 +8,10 @@ import '../../shared/assets/global.styles.css'
 import Vue from 'vue'
 import AuthenticationApp from './AuthenticationApp.vue'
 import { setupWebPage } from '../../shared/setupWebPage.js'
+import { markWindowReady } from '../../shared/markWindowReady.ts'
 
 await setupWebPage()
 
 new Vue(AuthenticationApp).$mount('#app')
+
+markWindowReady()

src/help/help.window.js

@@ -7,7 +7,7 @@ const { BASE_TITLE } = require('../constants.js')
 const { BrowserWindow } = require('electron')
 const { applyExternalLinkHandler } = require('../app/externalLinkHandlers.js')
 const { getBrowserWindowIcon } = require('../shared/icons.utils.js')
-const { getScaledWindowSize } = require('../app/utils.ts')
+const { getScaledWindowSize, showWhenWindowMarkedReady } = require('../app/utils.ts')
 
 /**
  *
@@ -41,9 +41,7 @@ function createHelpWindow(parentWindow) {
 
 	applyExternalLinkHandler(window)
 
-	window.on('ready-to-show', () => {
-		window.show()
-	})
+	showWhenWindowMarkedReady(window)
 
 	return window
 }

src/help/renderer/help.app.js

@@ -8,7 +8,10 @@ import '../../shared/assets/global.styles.css'
 import Vue from 'vue'
 import HelpApp from './HelpApp.vue'
 import { setupWebPage } from '../../shared/setupWebPage.js'
+import { markWindowReady } from '../../shared/markWindowReady.ts'
 
 await setupWebPage()
 
 new Vue(HelpApp).$mount('#app')
+
+markWindowReady()

src/main.js

@@ -19,6 +19,7 @@ const { createWelcomeWindow } = require('./welcome/welcome.window.js')
 const { installVueDevtools } = require('./install-vue-devtools.js')
 const { loadAppConfig, getAppConfig, setAppConfig } = require('./app/AppConfig.ts')
 const { triggerDownloadUrl } = require('./app/downloads.ts')
+const { showWhenWindowMarkedReady, waitWindowMarkedReady } = require('./app/utils.ts')
 
 /**
  * Parse command line arguments
@@ -144,7 +145,7 @@ app.whenReady().then(async () => {
 		// There is no window (possible on macOS) - create
 		if (!mainWindow || mainWindow.isDestroyed()) {
 			mainWindow = createMainWindow()
-			mainWindow.once('ready-to-show', () => mainWindow.show())
+			showWhenWindowMarkedReady(mainWindow)
 			return
 		}
 
@@ -233,7 +234,6 @@ app.whenReady().then(async () => {
 
 	mainWindow = createWelcomeWindow()
 	createMainWindow = createWelcomeWindow
-	mainWindow.once('ready-to-show', () => mainWindow.show())
 
 	ipcMain.once('appData:receive', async (event, appData) => {
 		const welcomeWindow = mainWindow
@@ -254,14 +254,15 @@ app.whenReady().then(async () => {
 			createMainWindow = createAuthenticationWindow
 		}
 
-		mainWindow.once('ready-to-show', () => {
-			// Do not show the main window if it is the Talk Window opened in the background
-			const isTalkWindow = createMainWindow === createTalkWindow
-			if (!isTalkWindow || !ARGUMENTS.openInBackground) {
-				mainWindow.show()
-			}
-			welcomeWindow.close()
-		})
+		await waitWindowMarkedReady(mainWindow)
+
+		welcomeWindow.close()
+
+		// Do not show the main window if it is the Talk Window opened in the background
+		const isTalkWindow = createMainWindow === createTalkWindow
+		if (!isTalkWindow || !ARGUMENTS.openInBackground) {
+			mainWindow.show()
+		}
 	})
 
 	let macDockBounceId
@@ -287,19 +288,18 @@ app.whenReady().then(async () => {
 
 	ipcMain.handle('authentication:openLoginWebView', async (event, serverUrl) => openLoginWebView(mainWindow, serverUrl))
 
-	ipcMain.handle('authentication:login', async () => {
+	ipcMain.handle('authentication:login', () => {
 		mainWindow.close()
 		mainWindow = createTalkWindow()
 		createMainWindow = createTalkWindow
-		mainWindow.once('ready-to-show', () => mainWindow.show())
+		showWhenWindowMarkedReady(mainWindow)
 	})
 
-	ipcMain.handle('authentication:logout', async (event) => {
+	ipcMain.handle('authentication:logout', async () => {
 		if (createMainWindow === createTalkWindow) {
 			await mainWindow.webContents.session.clearStorageData()
 			const authenticationWindow = createAuthenticationWindow()
 			createMainWindow = createAuthenticationWindow
-			authenticationWindow.once('ready-to-show', () => authenticationWindow.show())
 
 			mainWindow.destroy()
 			mainWindow = authenticationWindow
@@ -322,7 +322,7 @@ app.whenReady().then(async () => {
 		isInWindowRelaunch = true
 		mainWindow.destroy()
 		mainWindow = createMainWindow()
-		mainWindow.once('ready-to-show', () => mainWindow.show())
+		showWhenWindowMarkedReady(mainWindow)
 		isInWindowRelaunch = false
 	})
 
@@ -338,7 +338,7 @@ app.whenReady().then(async () => {
 			// dock icon is clicked and there are no other windows open.
 			// See window-all-closed event handler.
 			mainWindow = createMainWindow()
-			mainWindow.once('ready-to-show', () => mainWindow.show())
+			showWhenWindowMarkedReady(mainWindow)
 		}
 	})
 })

src/talk/renderer/talk.main.js

@@ -11,6 +11,7 @@ import './assets/overrides.css'
 import 'regenerator-runtime' // TODO: Why isn't it added on bundling
 import { initTalkHashIntegration } from './init.js'
 import { setupWebPage } from '../../shared/setupWebPage.js'
+import { markWindowReady } from '../../shared/markWindowReady.ts'
 import { createViewer } from './Viewer/Viewer.js'
 import { createDesktopApp } from './desktop.app.js'
 import { registerTalkDesktopSettingsSection } from './Settings/index.ts'
@@ -35,3 +36,5 @@ window.OCA.Talk.Desktop.talkRouter.value = window.OCA.Talk.instance.$router
 registerTalkDesktopSettingsSection()
 
 await import('./notifications/notifications.store.js')
+
+markWindowReady()

src/upgrade/renderer/upgrade.app.js

@@ -8,7 +8,10 @@ import '../../shared/assets/global.styles.css'
 import Vue from 'vue'
 import UpgradeApp from './UpgradeApp.vue'
 import { setupWebPage } from '../../shared/setupWebPage.js'
+import { markWindowReady } from '../../shared/markWindowReady.ts'
 
 await setupWebPage()
 
 new Vue(UpgradeApp).$mount('#app')
+
+markWindowReady()

src/upgrade/upgrade.window.js

@@ -7,7 +7,7 @@ const { BASE_TITLE } = require('../constants.js')
 const { BrowserWindow } = require('electron')
 const { applyExternalLinkHandler } = require('../app/externalLinkHandlers.js')
 const { getBrowserWindowIcon } = require('../shared/icons.utils.js')
-const { getScaledWindowSize } = require('../app/utils.ts')
+const { getScaledWindowSize, showWhenWindowMarkedReady } = require('../app/utils.ts')
 
 /**
  *
@@ -38,9 +38,7 @@ function createUpgradeWindow() {
 
 	applyExternalLinkHandler(window)
 
-	window.on('ready-to-show', () => {
-		window.show()
-	})
+	showWhenWindowMarkedReady(window)
 
 	return window
 }

src/welcome/welcome.js

@@ -9,6 +9,7 @@ import { appData } from '../app/AppData.js'
 import { refetchAppDataIfDirty } from '../app/appData.service.js'
 import { initGlobals } from '../shared/globals/globals.js'
 import { applyAxiosInterceptors } from '../shared/setupWebPage.js'
+import { markWindowReady } from '../shared/markWindowReady.ts'
 
 const quitButton = document.querySelector('.quit')
 quitButton.addEventListener('click', () => window.TALK_DESKTOP.quit())
@@ -20,6 +21,8 @@ window.TALK_DESKTOP.getOs().then(os => {
 	}
 })
 
+markWindowReady()
+
 appData.restore()
 
 initGlobals()

src/welcome/welcome.window.js

@@ -6,7 +6,7 @@
 const { BrowserWindow } = require('electron')
 const { getBrowserWindowIcon } = require('../shared/icons.utils.js')
 const { isMac } = require('../shared/os.utils.js')
-const { getScaledWindowSize } = require('../app/utils.ts')
+const { getScaledWindowSize, showWhenWindowMarkedReady } = require('../app/utils.ts')
 
 /**
  * @return {import('electron').BrowserWindow}
@@ -37,6 +37,8 @@ function createWelcomeWindow() {
 
 	window.loadURL(WELCOME_WINDOW_WEBPACK_ENTRY)
 
+	showWhenWindowMarkedReady(window)
+
 	return window
 }