diff --git a/apps/theming/css/theming.scss b/apps/theming/css/theming.scss
index b1b8896d0c..35341159f1 100644
--- a/apps/theming/css/theming.scss
+++ b/apps/theming/css/theming.scss
@@ -7,6 +7,17 @@
@return (0.2126 * $-local-red + 0.7152 * $-local-green + 0.0722 * $-local-blue) / 255;
}
+$has-custom-logo: variable_exists('theming-logo-mime') and $theming-logo-mime != '';
+$invert: luma($color-primary) > 0.6;
+
+@if ($has-custom-logo == false) {
+ @if ($invert) {
+ $image-logo: url('../../../../svg/core/logo/logo/000000?v=1');
+ } @else {
+ $image-logo: url('../../../../svg/core/logo/logo/ffffff?v=1');
+ }
+}
+
.nc-theming-main-background {
background-color: $color-primary;
}
@@ -19,7 +30,7 @@
color: $color-primary-text;
}
-@if (luma($color-primary) > 0.6) {
+@if ($invert) {
#appmenu:not(.inverted) svg {
filter: invert(1);
}
@@ -104,7 +115,8 @@
}
/* override styles for login screen in guest.css */
-@if variable_exists('theming-logo-mime') and $theming-logo-mime != '' {
+@if ($has-custom-logo) {
+ // custom logo
#theming-preview-logo,
#header .logo {
background-size: contain;
@@ -113,6 +125,14 @@
#body-login #header .logo {
margin-bottom: 22px;
}
+} @else {
+ // default logo
+ @if ($invert) {
+ #theming-preview-logo,
+ #header .logo {
+ opacity: .6;
+ }
+ }
}
@if variable_exists('theming-background-mime') and $theming-background-mime != '' {
@@ -157,7 +177,7 @@ input.primary,
color: $color-primary-text;
}
-@if (luma($color-primary) > 0.6) {
+@if ($invert) {
#body-login #submit-wrapper .icon-confirm-white {
background-image: url('../../../core/img/actions/confirm.svg');
}
diff --git a/apps/theming/lib/ImageManager.php b/apps/theming/lib/ImageManager.php
index 46598673f1..75c978f2ad 100644
--- a/apps/theming/lib/ImageManager.php
+++ b/apps/theming/lib/ImageManager.php
@@ -80,9 +80,6 @@ class ImageManager {
}
switch ($key) {
- case 'logo-blue':
- // the blue logo is only available as svg
- return $this->urlGenerator->getAbsoluteURL('svg/core/logo/logo/0082C9') . '?v=' . $cacheBusterCounter;
case 'logo':
case 'logoheader':
case 'favicon':
diff --git a/apps/theming/lib/ThemingDefaults.php b/apps/theming/lib/ThemingDefaults.php
index 7a26b46516..0573f7b84d 100644
--- a/apps/theming/lib/ThemingDefaults.php
+++ b/apps/theming/lib/ThemingDefaults.php
@@ -33,7 +33,6 @@
namespace OCA\Theming;
-
use OCP\App\AppPathNotFoundException;
use OCP\App\IAppManager;
use OCP\Files\NotFoundException;
@@ -275,7 +274,7 @@ class ThemingDefaults extends \OC_Defaults {
'theming-favicon-mime' => "'" . $this->config->getAppValue('theming', 'faviconMime') . "'"
];
- $variables['image-logo'] = "url('". $this->getLogoUrl() ."')";
+ $variables['image-logo'] = "url('".$this->imageManager->getImageUrl('logo')."')";
$variables['image-logoheader'] = "'".$this->imageManager->getImageUrl('logoheader')."'";
$variables['image-favicon'] = "'".$this->imageManager->getImageUrl('favicon')."'";
$variables['image-login-background'] = "url('".$this->imageManager->getImageUrl('background')."')";
@@ -300,24 +299,6 @@ class ThemingDefaults extends \OC_Defaults {
return $variables;
}
- /**
- * Returns the logo url.
- * If there is a custom logo, it just returns it.
- * For the default logo it returns the white or blue one depending on the color luminance.
- *
- * @return string
- */
- private function getLogoUrl() {
- $logoMime = $this->config->getAppValue('theming', 'logoMime');
- $primaryColor = $this->getColorPrimary();
- $luminance = $this->util->calculateLuminance($primaryColor);
- if ($logoMime === '' & $luminance > 0.8) {
- return $this->imageManager->getImageUrl('logo-blue', true);
- } else {
- return $this->imageManager->getImageUrl('logo', true);
- }
- }
-
/**
* Check if the image should be replaced by the theming app
* and return the new image location then
diff --git a/apps/theming/tests/ThemingDefaultsTest.php b/apps/theming/tests/ThemingDefaultsTest.php
index 3da39fbbf2..68435dd148 100644
--- a/apps/theming/tests/ThemingDefaultsTest.php
+++ b/apps/theming/tests/ThemingDefaultsTest.php
@@ -632,66 +632,6 @@ class ThemingDefaultsTest extends TestCase {
$this->assertEquals(['foo'=>'bar'], $this->template->getScssVariables());
}
- /**
- * Provides test data for the get logo scss variable test.
- *
- * @return array
- */
- public function provideTestGetImageLogoScssVariableTestData(): array {
- return [
- // default logo
- ['', '#000000', 0.0, 'logo'],
- ['', '#cccccc', 0.8, 'logo'],
- ['', '#dddddd', 0.81, 'logo-blue'],
- ['', '#ffffff', 1.0, 'logo-blue'],
-
- // custom logo
- ['image/png', '#000000', 0.0, 'logo'],
- ['image/png', '#cccccc', 0.8, 'logo'],
- ['image/png', '#dddddd', 0.81, 'logo'],
- ['image/png', '#ffffff', 1.0, 'logo'],
- ];
- }
-
- /**
- * Tests chat the logo url scss variable has the expected value
- * depending on color and custom logo presence.
- *
- * @dataProvider provideTestGetImageLogoScssVariableTestData
- * @param string $themingLogoMime The custom logo mime type
- * @param string $primaryColor The primary theme color
- * @param float $luminance The calculated luminance
- * @param string $expected The expected requested logo
- * @return void
- */
- public function testGetImageLogoScssVariable(
- string $themingLogoMime,
- string $primaryColor,
- float $luminance,
- string $expected
- ) {
- $this->config->expects($this->at(5))
- ->method('getAppValue')
- ->with('theming', 'logoMime')
- ->willReturn($themingLogoMime);
- $this->config->expects($this->at(6))
- ->method('getAppValue')
- ->with('theming', 'color', $this->defaults->getColorPrimary())
- ->willReturn($primaryColor);
-
- $this->util
- ->method('calculateLuminance')
- ->with($primaryColor)
- ->willReturn($luminance);
-
- $this->imageManager->expects($this->at(0))
- ->method('getImageUrl')
- ->with($expected)
- ->willReturn('custom-logo?v=0');
-
- $this->template->getScssVariables();
- }
-
public function testGetScssVariables() {
$this->config->expects($this->at(0))->method('getAppValue')->with('theming', 'cachebuster', '0')->willReturn('0');
$this->config->expects($this->at(1))->method('getAppValue')->with('theming', 'logoMime', false)->willReturn('jpeg');
@@ -699,13 +639,10 @@ class ThemingDefaultsTest extends TestCase {
$this->config->expects($this->at(3))->method('getAppValue')->with('theming', 'logoheaderMime', false)->willReturn('jpeg');
$this->config->expects($this->at(4))->method('getAppValue')->with('theming', 'faviconMime', false)->willReturn('jpeg');
- $this->config->expects($this->at(5))->method('getAppValue')->with('theming', 'logoMime', false)->willReturn('jpeg');
+ $this->config->expects($this->at(5))->method('getAppValue')->with('theming', 'color', null)->willReturn($this->defaults->getColorPrimary());
$this->config->expects($this->at(6))->method('getAppValue')->with('theming', 'color', $this->defaults->getColorPrimary())->willReturn($this->defaults->getColorPrimary());
-
- $this->config->expects($this->at(7))->method('getAppValue')->with('theming', 'color', null)->willReturn($this->defaults->getColorPrimary());
+ $this->config->expects($this->at(7))->method('getAppValue')->with('theming', 'color', $this->defaults->getColorPrimary())->willReturn($this->defaults->getColorPrimary());
$this->config->expects($this->at(8))->method('getAppValue')->with('theming', 'color', $this->defaults->getColorPrimary())->willReturn($this->defaults->getColorPrimary());
- $this->config->expects($this->at(9))->method('getAppValue')->with('theming', 'color', $this->defaults->getColorPrimary())->willReturn($this->defaults->getColorPrimary());
- $this->config->expects($this->at(10))->method('getAppValue')->with('theming', 'color', $this->defaults->getColorPrimary())->willReturn($this->defaults->getColorPrimary());
$this->util->expects($this->any())->method('invertTextColor')->with($this->defaults->getColorPrimary())->willReturn(false);
$this->util->expects($this->any())->method('elementColor')->with($this->defaults->getColorPrimary())->willReturn('#aaaaaa');
diff --git a/core/img/logo/logo.svg b/core/img/logo/logo.svg
index dda265dfa1..dbc3e7f3f8 100644
--- a/core/img/logo/logo.svg
+++ b/core/img/logo/logo.svg
@@ -1,71 +1 @@
-
-
+