diff options
| author |  Kristóf Marussy <kristof@marussy.com> | 2021-05-29 21:09:41 +0200 |
|---|---|---|
| committer |  Vijay Raghavan Aravamudhan <vraravam@users.noreply.github.com> | 2021-05-30 04:56:58 +0530 |
| commit | 38e90556cd8c5d51dc2af74cf69ddc8386a98730 (patch) | |
| tree | 789e3c7b6359b885eaa9dc02bf90648eff727a28 /src/styles/tabs.scss | |
| parent | Bump ws from 6.2.1 to 7.4.6 (#1463) (diff) | |
| download | ferdium-app-38e90556cd8c5d51dc2af74cf69ddc8386a98730.tar.gz ferdium-app-38e90556cd8c5d51dc2af74cf69ddc8386a98730.tar.zst ferdium-app-38e90556cd8c5d51dc2af74cf69ddc8386a98730.zip | |
Fix color adjustment confusion
SCSS functions apply absolute change to the color lightness values, but
the 'color' npm package does relative changes. This lead to inconsistent
behavior between the default and custom accent colors.
See also https://github.com/Qix-/color/issues/53#issuecomment-656590710
We use relative adjustment for buttons. For the service switcher in
light mode, we use relative adjustment instead, because absolute
adjustment made the buttons too hard to see (#235).
Diffstat (limited to 'src/styles/tabs.scss')
| -rw-r--r-- | src/styles/tabs.scss | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/styles/tabs.scss b/src/styles/tabs.scss index fb2f1c32a..31a239387 100644 --- a/src/styles/tabs.scss +++ b/src/styles/tabs.scss | |||
| @@ -33,7 +33,8 @@ | |||
| 33 | width: $theme-sidebar-width; | 33 | width: $theme-sidebar-width; |
| 34 | 34 | ||
| 35 | &.is-active { | 35 | &.is-active { |
| 36 | background: lighten($theme-brand-primary, 35%); | 36 | background: change-color($theme-brand-primary, |
| 37 | $lightness: min(lightness($theme-brand-primary) * 1.35, 100)); | ||
| 37 | border-left-width: 4px; | 38 | border-left-width: 4px; |
| 38 | border-left-style: solid; | 39 | border-left-style: solid; |
| 39 | color: $theme-brand-primary; | 40 | color: $theme-brand-primary; |