From b3510aa2b47c3609810cbce2e4b2115b97291d0f Mon Sep 17 00:00:00 2001
From: Bruno Windels <brunow@matrix.org>
Date: Tue, 23 Jun 2020 17:54:17 +0200
Subject: [PATCH] remove css vars when switching theme

---
 src/theme.js | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/src/theme.js b/src/theme.js
index da70dc7bb1..7e0eaabf11 100644
--- a/src/theme.js
+++ b/src/theme.js
@@ -35,6 +35,16 @@ export function enumerateThemes() {
     return Object.assign({}, customThemeNames, BUILTIN_THEMES);
 }
 
+function clearCustomTheme() {
+    // remove all css variables, we assume these are there because of the custom theme
+    const inlineStyleProps = Object.values(document.body.style);
+    for (const prop of inlineStyleProps) {
+        if (prop.startsWith("--")) {
+            document.body.style.removeProperty(prop);
+        }
+    }
+}
+
 
 function setCustomThemeVars(customTheme) {
     const {style} = document.body;
@@ -97,6 +107,7 @@ export async function setTheme(theme) {
         const themeWatcher = new ThemeWatcher();
         theme = themeWatcher.getEffectiveTheme();
     }
+    clearCustomTheme();
     let stylesheetName = theme;
     if (theme.startsWith("custom-")) {
         const customTheme = getCustomTheme(theme.substr(7));