From a94feb28b6259f7c1a88ff59b6dda08eaca41de3 Mon Sep 17 00:00:00 2001
From: jorg-vr <jorg.vr@live.be>
Date: Thu, 29 Feb 2024 10:22:13 +0100
Subject: [PATCH] Fix iframe theme switch from system theme

---
 app/assets/javascripts/state/Theme.ts | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/app/assets/javascripts/state/Theme.ts b/app/assets/javascripts/state/Theme.ts
index fe32cda1c1..084d85a91d 100644
--- a/app/assets/javascripts/state/Theme.ts
+++ b/app/assets/javascripts/state/Theme.ts
@@ -14,11 +14,17 @@ class ThemeState extends State {
     @stateProperty computedStyle: CSSStyleDeclaration = getComputedStyle(document.documentElement);
 
     // the theme option selected by the user
-    get selectedTheme(): ThemeOption {
+    public get selectedTheme(): ThemeOption {
         return this._selectedTheme;
     }
 
-    set selectedTheme(theme: ThemeOption) {
+    public set selectedTheme(theme: ThemeOption) {
+        // update the theme of all iframes that have a theme parameter with the current theme
+        Array.from(document.getElementsByTagName("iframe")).forEach(iframe => {
+            if (getURLParameter("theme", iframe.src) === this._selectedTheme) {
+                iframe.src = updateURLParameter(iframe.src, "theme", theme);
+            }
+        });
         this._selectedTheme = theme;
         this.theme = theme === "system" ? this.systemTheme : theme;
     }
@@ -29,23 +35,17 @@ class ThemeState extends State {
     }
 
     // The theme that is currently applied to the page
-    get theme(): Theme {
+    public get theme(): Theme {
         return this._theme;
     }
 
-    set theme(theme: Theme) {
+    private set theme(theme: Theme) {
         // update the page theme
         document.documentElement.setAttribute("data-bs-theme", theme);
         // update the theme of all elements that have a data-bs-theme attribute with the current theme
         document.querySelectorAll(`[data-bs-theme="${this._theme}"]`).forEach(element => {
             element.setAttribute("data-bs-theme", theme);
         });
-        // update the theme of all iframes that have a theme parameter with the current theme
-        Array.from(document.getElementsByTagName("iframe")).forEach(iframe => {
-            if (getURLParameter("theme", iframe.src) === this._theme) {
-                iframe.src = updateURLParameter(iframe.src, "theme", theme);
-            }
-        });
         this._theme = theme;
         this.computedStyle = getComputedStyle(document.documentElement);
     }