From d4c8606f432ba7cb762e4eca102ee09ca3f728bb Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Javi=20Mart=C3=ADn?= Date: Sat, 29 Oct 2022 13:51:14 +0200 Subject: [PATCH] Simplify brand-background mixin We don't need the color parameter anymore since we can now use a more generic mixin for any background, like brand-secondary. --- app/assets/stylesheets/mixins/colors.scss | 4 ++-- app/assets/stylesheets/mixins/forms.scss | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/app/assets/stylesheets/mixins/colors.scss b/app/assets/stylesheets/mixins/colors.scss index eff3b260d..c7811bbbf 100644 --- a/app/assets/stylesheets/mixins/colors.scss +++ b/app/assets/stylesheets/mixins/colors.scss @@ -7,8 +7,8 @@ } } -@mixin brand-background($color: $brand) { - @include background-with-text-contrast($color); +@mixin brand-background { + @include background-with-text-contrast($brand); } @mixin brand-text { diff --git a/app/assets/stylesheets/mixins/forms.scss b/app/assets/stylesheets/mixins/forms.scss index 3a6b85df1..212c6c79a 100644 --- a/app/assets/stylesheets/mixins/forms.scss +++ b/app/assets/stylesheets/mixins/forms.scss @@ -14,7 +14,7 @@ $icon-size-with-padding: $icon-size + $padding-right; $polygon-size: $icon-size / 2; @include background-till-left-of-screen; - @include brand-background($brand-secondary); + @include background-with-text-contrast($brand-secondary); border: $border-width solid $brand-secondary; border-bottom-right-radius: rem-calc(12); border-top-right-radius: rem-calc(12);