Skip to content

Commit 76b22bd

Browse files
authored
Merge pull request #14885 from nextcloud/bugfix/14639/theming
Fix various theming issues on bright colors
2 parents 5ae9eda + 3860373 commit 76b22bd

File tree

2 files changed

+23
-4
lines changed

2 files changed

+23
-4
lines changed

apps/theming/css/theming.scss

Lines changed: 22 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,8 @@ $invert: luma($color-primary) > 0.6;
6868
a,
6969
label,
7070
p,
71-
#alternative-logins legend {
71+
#alternative-logins legend,
72+
.lost-password-container #lost-password {
7273
color: $color-primary-text;
7374
}
7475
input[type='checkbox'].checkbox--white + label:before {
@@ -175,8 +176,26 @@ input.primary,
175176
}
176177

177178
@if ($invert) {
178-
#body-login #submit-wrapper .icon-confirm-white {
179-
background-image: url('../../../core/img/actions/confirm.svg');
179+
#body-login {
180+
.body-login-container {
181+
background-color: $color-main-background;
182+
}
183+
#submit-wrapper .icon-confirm-white {
184+
background-image: url('../../../core/img/actions/confirm.svg');
185+
}
186+
.body-login-container {
187+
h2 {
188+
color: $color-main-text;
189+
}
190+
.icon-search.icon-white {
191+
// CSS variable is not used here since it is on the public page layout,
192+
// where the dark theme doesn't apply at the moment
193+
background-image: url('../../../core/img/actions/search.svg');
194+
}
195+
}
196+
}
197+
#body-public #header .icon-more-white {
198+
background-image: var(--icon-more-000);
180199
}
181200
}
182201

core/css/header.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -445,7 +445,7 @@ nav[role='navigation'] {
445445
text-overflow: initial;
446446
width: auto;
447447
overflow: hidden;
448-
background-color: var(--color-primary-element);
448+
background-color: var(--color-primary);
449449
padding: 0 5px;
450450
z-index: 2;
451451
}

0 commit comments

Comments
 (0)