Better handling loading terminal fonts (#6913)

* better handling loading terminal fonts timing issues, fix #5019

Signed-off-by: Roman <ixrock@gmail.com>

* refactoring, loading all terminal fonts to earliest stage app starting

Signed-off-by: Roman <ixrock@gmail.com>

* attempt to fix tests

Signed-off-by: Roman <ixrock@gmail.com>

Signed-off-by: Roman <ixrock@gmail.com>

src/renderer/components/app.scss

@@ -232,22 +232,6 @@ iframe {
   }
 }
 
-#fonts-preloading {
-  > span {
-    position: absolute;
-    visibility: hidden;
-    height: 0;
-
-    &:before {
-      width: 0;
-      display: block;
-      overflow: hidden;
-      content: "text-example"; // some text required to start applying/rendering font in document
-      font-family: inherit; // font-family must be specified via style="" (see: template.html)
-    }
-  }
-}
-
 // app's common loading indicator, displaying on the route transitions
 #loading {
   position: absolute;

src/renderer/components/dock/terminal/create-terminal.injectable.ts

@@ -9,7 +9,8 @@ import type { TabId } from "../dock/store";
 import type { TerminalApi } from "../../../api/terminal-api";
 import terminalSpawningPoolInjectable from "./terminal-spawning-pool.injectable";
 import terminalConfigInjectable from "../../../../common/user-store/terminal-config.injectable";
-import terminalCopyOnSelectInjectable from "../../../../common/user-store/terminal-copy-on-select.injectable";
+import terminalCopyOnSelectInjectable
+  from "../../../../common/user-store/terminal-copy-on-select.injectable";
 import isMacInjectable from "../../../../common/vars/is-mac.injectable";
 import openLinkInBrowserInjectable from "../../../../common/utils/open-link-in-browser.injectable";
 import xtermColorThemeInjectable from "../../../themes/terminal-colors.injectable";

src/renderer/components/dock/terminal/terminal-fonts.global-override-for-injectable.ts

@@ -0,0 +1,14 @@
+/**
+ * Copyright (c) OpenLens Authors. All rights reserved.
+ * Licensed under MIT License. See LICENSE in root directory for more information.
+ */
+import { getGlobalOverride } from "../../../../common/test-utils/get-global-override";
+import { preloadAllTerminalFontsInjectable } from "./terminal-fonts.injectable";
+
+export default getGlobalOverride(preloadAllTerminalFontsInjectable, () => {
+  return {
+    id: "",
+    async run() {
+    },
+  };
+});

src/renderer/components/dock/terminal/terminal-fonts.injectable.ts

@@ -0,0 +1,78 @@
+/**
+ * Copyright (c) OpenLens Authors. All rights reserved.
+ * Licensed under MIT License. See LICENSE in root directory for more information.
+ */
+
+import { getInjectable } from "@ogre-tools/injectable";
+import { beforeFrameStartsFirstInjectionToken } from "../../../before-frame-starts/tokens";
+import RobotoMono from "../../../fonts/Roboto-Mono-nerd.ttf"; // patched font with icons
+import AnonymousPro from "../../../fonts/AnonymousPro-Regular.ttf";
+import IBMPlexMono from "../../../fonts/IBMPlexMono-Regular.ttf";
+import JetBrainsMono from "../../../fonts/JetBrainsMono-Regular.ttf";
+import RedHatMono from "../../../fonts/RedHatMono-Regular.ttf";
+import SourceCodePro from "../../../fonts/SourceCodePro-Regular.ttf";
+import SpaceMono from "../../../fonts/SpaceMono-Regular.ttf";
+import UbuntuMono from "../../../fonts/UbuntuMono-Regular.ttf";
+
+export const terminalFontsInjectable = getInjectable({
+  id: "terminalFontsInjectable",
+
+  instantiate() {
+    return new Map([
+      ["RobotoMono", RobotoMono],
+      ["Anonymous Pro", AnonymousPro],
+      ["IBM Plex Mono", IBMPlexMono],
+      ["JetBrains Mono", JetBrainsMono],
+      ["Red Hat Mono", RedHatMono],
+      ["Source Code Pro", SourceCodePro],
+      ["Space Mono", SpaceMono],
+      ["Ubuntu Mono", UbuntuMono],
+    ]);
+  },
+});
+
+
+export const preloadTerminalFontInjectable = getInjectable({
+  id: "preloadTerminalFontInjectable",
+
+  instantiate(di) {
+    const terminalFonts = di.inject(terminalFontsInjectable);
+
+    return async function (fontFamily: string): Promise<void> {
+      const fontBundledPath = terminalFonts.get(fontFamily);
+      const fontLoaded = document.fonts.check(`10px ${fontFamily}`);
+
+      if (fontLoaded || !fontBundledPath) return;
+
+      const font = new FontFace(fontFamily, `url(${fontBundledPath})`);
+
+      document.fonts.add(font);
+      await font.load();
+    };
+  },
+
+  causesSideEffects: true,
+});
+
+export const preloadAllTerminalFontsInjectable = getInjectable({
+  id: "preloadAllTerminalFontsInjectable",
+
+  instantiate(di) {
+    const terminalFonts = di.inject(terminalFontsInjectable);
+    const preloadFont = di.inject(preloadTerminalFontInjectable);
+
+    return {
+      id: "preload-all-terminal-fonts",
+
+      async run() {
+        await Promise.allSettled(
+          Array.from(terminalFonts.keys()).map(preloadFont),
+        );
+      },
+    };
+  },
+
+  injectionToken: beforeFrameStartsFirstInjectionToken,
+
+  causesSideEffects: true,
+});

src/renderer/components/fonts.scss

@@ -44,58 +44,3 @@
   font-display: block;
   src: url("../fonts/MaterialIcons-Regular.ttf") format("truetype");
 }
-
-
-// Terminal fonts (monospaced)
-// Source: https://fonts.google.com/?category=Monospace
-@font-face {
-  font-family: "Anonymous Pro";
-  src: local("Anonymous Pro"), url("../fonts/AnonymousPro-Regular.ttf") format("truetype");
-  font-display: block;
-}
-
-@font-face {
-  font-family: "IBM Plex Mono";
-  src: local("IBM Plex Mono"), url("../fonts/IBMPlexMono-Regular.ttf") format("truetype");
-  font-display: block;
-}
-
-@font-face {
-  font-family: "JetBrains Mono";
-  src: local("JetBrains Mono"), url("../fonts/JetBrainsMono-Regular.ttf") format("truetype");
-  font-display: block;
-}
-
-@font-face {
-  font-family: "Red Hat Mono";
-  src: local("Red Hat Mono"), url("../fonts/RedHatMono-Regular.ttf") format("truetype");
-  font-display: block;
-}
-
-
-@font-face {
-  font-family: "Source Code Pro";
-  src: local("Source Code Pro"), url("../fonts/SourceCodePro-Regular.ttf") format("truetype");
-  font-display: block;
-}
-
-@font-face {
-  font-family: "Space Mono";
-  src: local("Space Mono"), url("../fonts/SpaceMono-Regular.ttf") format("truetype");
-  font-display: block;
-}
-
-@font-face {
-  font-family: "Ubuntu Mono";
-  src: local("Ubuntu Mono"), url("../fonts/UbuntuMono-Regular.ttf") format("truetype");
-  font-display: block;
-}
-
-// Patched RobotoMono font with icons
-// RobotoMono Windows Compatible for using in terminal
-// https://github.com/ryanoasis/nerd-fonts/tree/master/patched-fonts/RobotoMono
-@font-face {
-  font-family: "RobotoMono";
-  src: local("RobotoMono"), url("../fonts/Roboto-Mono-nerd.ttf") format("truetype");
-  font-display: block;
-}

src/renderer/template.html

@@ -5,17 +5,6 @@
   </head>
   <body>
     <div id="app"></div>
-    <div id="fonts-preloading">
-      <span style="font-family: 'Material Icons'" />
-      <span style="font-family: 'RobotoMono'" />
-      <span style="font-family: 'Anonymous Pro'" />
-      <span style="font-family: 'IBM Plex Mono'" />
-      <span style="font-family: 'JetBrains Mono'" />
-      <span style="font-family: 'Red Hat Mono'" />
-      <span style="font-family: 'Source Code Pro'" />
-      <span style="font-family: 'Space Mono'" />
-      <span style="font-family: 'Ubuntu Mono'" />
-    </div>
     <div id="terminal-init"></div>
   </body>
 </html>