From 7ae2dee1b2c627a98e58b5c1fe11d6051f1b95c5 Mon Sep 17 00:00:00 2001 From: "mr. M" Date: Sun, 27 Oct 2024 20:32:20 +0100 Subject: [PATCH] Refactor ZenGradientGenerator class to use constructor instead of init method --- src/ZenGradientGenerator.mjs | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/src/ZenGradientGenerator.mjs b/src/ZenGradientGenerator.mjs index 65eb7d1..7f4992e 100644 --- a/src/ZenGradientGenerator.mjs +++ b/src/ZenGradientGenerator.mjs @@ -219,7 +219,7 @@ } this.panel.querySelector('.zen-theme-picker-gradient').appendChild(dot); if (!fromWorkspace) { - this.onDarkModeChange(null, true); + this.updateCurrentWorkspace(true); } } @@ -295,7 +295,7 @@ onDotMouseUp(event) { if (event.button === 2) { - if (this.numberOfDots < 2 || !event.target.classList.contains('zen-theme-picker-dot')) { + if (!event.target.classList.contains('zen-theme-picker-dot')) { return; } event.target.remove(); @@ -352,7 +352,9 @@ getGradient(colors) { const themedColors = this.themedColors(colors); - if (themedColors.length === 1) { + if (themedColors.length === 0) { + return "var(--zen-themed-toolbar-bg)"; + } else if (themedColors.length === 1) { return this.getSingleRGBColor(themedColors[0]); } return `linear-gradient(${this.currentRotation}deg, ${themedColors.map(color => this.getSingleRGBColor(color)).join(', ')})`; @@ -417,9 +419,9 @@ return; } - browser.gZenThemePicker.currentOpacity = workspaceTheme.opacity || 0.5; - browser.gZenThemePicker.currentRotation = workspaceTheme.rotation || 45; - browser.gZenThemePicker.currentTexture = workspaceTheme.texture || 0; + browser.gZenThemePicker.currentOpacity = workspaceTheme.opacity ?? 0.5; + browser.gZenThemePicker.currentRotation = workspaceTheme.rotation ?? 45; + browser.gZenThemePicker.currentTexture = workspaceTheme.texture ?? 0; browser.gZenThemePicker.numberOfDots = workspaceTheme.gradientColors.length;