From 3616b23e61a8c7a7d7a23dbb720cd3746155511c Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Tue, 27 Sep 2022 18:08:16 +0000 Subject: [PATCH] Bug 30952: (follow-up) Use variables for primary and secondary greens _variables.scss currently contains a variable, $background-color-primary, which isn't used. I think we should use that variable anywhere the color is used, and use SASS color calculation to generate the secondary color. The two colors can be compared in the search header: primary for the background, secondary for the submit button color. To test, apply the patch and rebuild the staff interface CSS. Test various pages in the staff interface to confirm that the colors look the same as before: Search header, tab colors, "Last patron" button, etc. --- koha-tmpl/intranet-tmpl/prog/css/src/_header.scss | 2 +- koha-tmpl/intranet-tmpl/prog/css/src/_variables.scss | 1 + koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss | 8 ++++---- 3 files changed, 6 insertions(+), 5 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/css/src/_header.scss b/koha-tmpl/intranet-tmpl/prog/css/src/_header.scss index 250065523f..8e786dbb04 100644 --- a/koha-tmpl/intranet-tmpl/prog/css/src/_header.scss +++ b/koha-tmpl/intranet-tmpl/prog/css/src/_header.scss @@ -180,7 +180,7 @@ a.navbar-toggle { input[type="submit"], button[type="submit"] { height: 31px; - background-color: #71B443; + background-color: $background-color-secondary; color: white; border: 0; text-shadow: unset; diff --git a/koha-tmpl/intranet-tmpl/prog/css/src/_variables.scss b/koha-tmpl/intranet-tmpl/prog/css/src/_variables.scss index 41966bfd6f..42b53140f8 100644 --- a/koha-tmpl/intranet-tmpl/prog/css/src/_variables.scss +++ b/koha-tmpl/intranet-tmpl/prog/css/src/_variables.scss @@ -1,5 +1,6 @@ $green-text-color: #006100; $background-color-primary: #408540; +$background-color-secondary: lighten(saturate(adjust-hue($background-color-primary, -24), 9), 9); // Copied from Bootstrap 5 without system-ui because of // https://infinnie.github.io/blog/2017/systemui.html diff --git a/koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss b/koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss index 8689c3c149..eb11e60c29 100644 --- a/koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss +++ b/koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss @@ -78,7 +78,7 @@ a { i, img { text-align: center; color: $green-text-color; - border: solid 3px #71B443; + border: solid 3px $background-color-secondary; border-radius: 50%; background-color: transparent; width: 40px; @@ -2792,7 +2792,7 @@ td.bundle { } .ui-tabs-nav { li { - background: #71B443; + background: $background-color-secondary; border: 0; margin-right: .4em; border-radius: 4px 4px 0 0; @@ -2938,7 +2938,7 @@ td.bundle { position: relative; top: 1px; white-space: nowrap; - background: #71B443; + background: $background-color-secondary; &.active, &:hover { font-weight: normal; @@ -3552,7 +3552,7 @@ code { .nav-tabs { > li { > a { - background-color: #71B443; + background-color: $background-color-secondary; color: #111; line-height: 1.42857143; margin-right: .4em; -- 2.30.2