Skip to content

Commit

Permalink
Merge pull request #42294 from nextcloud/backport/42283/stable28
Browse files Browse the repository at this point in the history
[stable28] fix: Ensure correct max contrast color is used when blurred background is used
  • Loading branch information
emoral435 authored Dec 15, 2023
2 parents cccac2a + 8d1d5f3 commit 42793ca
Show file tree
Hide file tree
Showing 11 changed files with 17 additions and 9 deletions.
3 changes: 3 additions & 0 deletions apps/user_status/src/UserStatus.vue
Original file line number Diff line number Diff line change
Expand Up @@ -176,6 +176,9 @@ export default {
<style lang="scss" scoped>
.user-status-menu-item {
// Ensure the maxcontrast color is set for the background
--color-text-maxcontrast: var(--color-text-maxcontrast-background-blur, var(--color-main-text));
width: auto;
min-width: 44px;
height: 44px;
Expand Down
2 changes: 1 addition & 1 deletion core/css/apps.css

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion core/css/apps.css.map

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 2 additions & 0 deletions core/css/apps.scss
Original file line number Diff line number Diff line change
Expand Up @@ -121,6 +121,8 @@ kbd {
#app-navigation:not(.vue) {
// We use fixed variable for the pill style as we have larger containers around nested list entries
--border-radius-pill: calc(var(--default-clickable-area) / 2);
// Ensure the maxcontrast color is set for the background
--color-text-maxcontrast: var(--color-text-maxcontrast-background-blur, var(--color-main-text));

width: variables.$navigation-width;
z-index: 500;
Expand Down
Loading

0 comments on commit 42793ca

Please sign in to comment.