From e8938df1986c4b52fc1f5c9afe4cc4078ea86e4c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Julius=20H=C3=A4rtl?= Date: Wed, 29 Aug 2018 16:46:12 +0200 Subject: [PATCH] Move SCSS variable fetching before the variables.scss to properly calculate color values MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Julius Härtl --- apps/accessibility/lib/Controller/AccessibilityController.php | 2 +- core/css/variables.scss | 4 ++-- lib/private/Template/SCSSCacher.php | 4 ++-- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/apps/accessibility/lib/Controller/AccessibilityController.php b/apps/accessibility/lib/Controller/AccessibilityController.php index 0a1d091d1c..8f1ffa452a 100644 --- a/apps/accessibility/lib/Controller/AccessibilityController.php +++ b/apps/accessibility/lib/Controller/AccessibilityController.php @@ -149,8 +149,8 @@ class AccessibilityController extends Controller { try { $css .= $scss->compile( $imports . - '@import "variables.scss";' . $this->getInjectedVariables() . + '@import "variables.scss";' . '@import "css-variables.scss";' ); } catch (ParserException $e) { diff --git a/core/css/variables.scss b/core/css/variables.scss index b6bffea973..34a105fccc 100644 --- a/core/css/variables.scss +++ b/core/css/variables.scss @@ -38,8 +38,8 @@ $color-main-background: #fff !default; $color-background-dark: nc-darken($color-main-background, 7%) !default; $color-background-darker: nc-darken($color-main-background, 14%) !default; -$color-primary: #0082c9; -$color-primary-text: #ffffff; +$color-primary: #0082c9 !default; +$color-primary-text: #ffffff !default; // do not use nc-darken/lighten in case of overriding because // primary-text is independent of color-main-text $color-primary-text-dark: darken($color-primary-text, 7%) !default; diff --git a/lib/private/Template/SCSSCacher.php b/lib/private/Template/SCSSCacher.php index 19734355d2..c4f89a9c63 100644 --- a/lib/private/Template/SCSSCacher.php +++ b/lib/private/Template/SCSSCacher.php @@ -284,9 +284,9 @@ class SCSSCacher { try { $compiledScss = $scss->compile( '$webroot: \'' . $this->getRoutePrefix() . '\';' . + $this->getInjectedVariables() . '@import "variables.scss";' . '@import "functions.scss";' . - $this->getInjectedVariables() . '@import "' . $fileNameSCSS . '";'); } catch (ParserException $e) { $this->logger->error($e, ['app' => 'core']); @@ -349,7 +349,7 @@ class SCSSCacher { } $variables = ''; foreach ($this->defaults->getScssVariables() as $key => $value) { - $variables .= '$' . $key . ': ' . $value . ';'; + $variables .= '$' . $key . ': ' . $value . ' !default;'; } // check for valid variables / otherwise fall back to defaults