Skip to content

Commit 68d93cf

Browse files
fix(theming): replace elementColor dark value with better contrast #8c8c8c
Signed-off-by: John Molakvoæ <[email protected]>
1 parent 4a7420f commit 68d93cf

File tree

4 files changed

+6
-8
lines changed

4 files changed

+6
-8
lines changed

apps/theming/css/default.css

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -63,14 +63,12 @@
6363
--color-primary-light: #e5f0f5;
6464
--color-primary-light-text: #002a41;
6565
--color-primary-light-hover: #dbe5ea;
66-
--color-primary-element-text-dark: #ededed;
6766
--color-primary-element: #006aa3;
68-
--color-primary-element-default-hover: #329bd3;
69-
--color-primary-element-text: #ffffff;
7067
--color-primary-element-hover: #3287b5;
68+
--color-primary-element-text: #ffffff;
7169
--color-primary-element-light: #e5f0f5;
72-
--color-primary-element-light-text: #002a41;
7370
--color-primary-element-light-hover: #dbe5ea;
71+
--color-primary-element-light-text: #002a41;
7472
--color-primary-element-text-dark: #ededed;
7573
--gradient-primary-background: linear-gradient(40deg, var(--color-primary) 0%, var(--color-primary-hover) 100%);
7674
--image-background-default: url('/apps/theming/img/background/kamil-porembinski-clouds.jpg');

apps/theming/lib/Util.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ public function elementColor($color, bool $brightBackground = true) {
9191

9292
if (!$brightBackground && $luminance < 0.2) {
9393
// If the color is too dark in dark mode, we fall back to a brighter gray
94-
return '#555555';
94+
return '#8c8c8c';
9595
}
9696

9797
return $color;

apps/theming/tests/CapabilitiesTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ public function dataGetCapabilities() {
111111
'color-text' => '#ffffff',
112112
'color-element' => '#000000',
113113
'color-element-bright' => '#000000',
114-
'color-element-dark' => '#555555',
114+
'color-element-dark' => '#8c8c8c',
115115
'logo' => 'http://localhost/logo5',
116116
'background' => '#000000',
117117
'background-plain' => true,
@@ -127,7 +127,7 @@ public function dataGetCapabilities() {
127127
'color-text' => '#ffffff',
128128
'color-element' => '#000000',
129129
'color-element-bright' => '#000000',
130-
'color-element-dark' => '#555555',
130+
'color-element-dark' => '#8c8c8c',
131131
'logo' => 'http://localhost/logo5',
132132
'background' => '#000000',
133133
'background-plain' => true,

apps/theming/tests/UtilTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ public function testElementColorDefault() {
112112

113113
public function testElementColorOnDarkBackground() {
114114
$elementColor = $this->util->elementColor("#000000", false);
115-
$this->assertEquals('#555555', $elementColor);
115+
$this->assertEquals('#8c8c8c', $elementColor);
116116
}
117117

118118
public function testElementColorOnBrightBackground() {

0 commit comments

Comments
 (0)