Skip to content

Commit

Permalink
Fix regression: modals lack backdrop
Browse files Browse the repository at this point in the history
  • Loading branch information
tnajdek committed Apr 25, 2024
1 parent d1a5966 commit 4cb0097
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 4 deletions.
2 changes: 1 addition & 1 deletion src/scss/components/_library.scss
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@
right: 0;
bottom: 0;
left: 0;
background-color: $modal-backdrop-bg;
background-color: var(--color-backdrop);
z-index: $z-index-search-backdrop
}

Expand Down
4 changes: 3 additions & 1 deletion src/scss/components/modal/_react-modal.scss
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
right: 0;
bottom: 0;
left: 0;
background-color: $modal-backdrop-bg;
background-color: var(--color-backdrop);
overflow-y: auto;
-webkit-overflow-scrolling: touch; // Momentum scrolling for iOS 10–
z-index: $z-index-modal;
Expand All @@ -39,6 +39,8 @@
}

.modal {
border: var(--material-border-quarternary);

@include bp-up(sm) {
width: $modal-md;
margin: $space-xl auto;
Expand Down
3 changes: 1 addition & 2 deletions src/scss/themes/_common.scss
Original file line number Diff line number Diff line change
Expand Up @@ -222,12 +222,11 @@ $menu-devider: $shade-2;
$menu-no-results-color: $shade-5;

// Drag layer
$creator-drag-preview-shadow: rgba($shade-10, 0.2);
$creator-drag-preview-shadow: var(--color-drag-preview-shadow);
$items-drag-indicator-color: $item-active-color;
$items-drag-indicator-bg: var(--accent-blue-lighter);

// Modal
$modal-backdrop-bg: rgba($shade-10, 0.4);
$modal-content-bg: $body-bg;
$modal-content-bg-touch: $shade-1;
$modal-header-bg: $shade-0;
Expand Down
2 changes: 2 additions & 0 deletions src/scss/themes/_dark.scss
Original file line number Diff line number Diff line change
Expand Up @@ -77,10 +77,12 @@ $-colors: (
color-button-active-bg: #f0f0f0,
color-button-shadow: rgba(0, 0, 0, 0.2),
color-button-shadow-lighter: rgba(0, 0, 0, 0.06),
color-drag-preview-shadow: rgba(0, 0, 0, 0.2),
tag-color-border: rgba(0, 0, 0, 0.08),
color-text-disabled: #9a9a9a,
color-nav-sidebar-border: #252525,
color-identifier-badge-hover-border: #b9b8b5,
color-backdrop: rgba(0, 0, 0, 0.6),
);

@media (prefers-color-scheme: dark) {
Expand Down
2 changes: 2 additions & 0 deletions src/scss/themes/_light.scss
Original file line number Diff line number Diff line change
Expand Up @@ -79,10 +79,12 @@ $-colors: (
color-button-active-bg: #f0f0f0,
color-button-shadow: rgba(0, 0, 0, 0.2),
color-button-shadow-lighter: rgba(0, 0, 0, 0.06),
color-drag-preview-shadow: rgba(0, 0, 0, 0.2),
tag-color-border: rgba(0, 0, 0, 0.08),
color-text-disabled: #656565, // lighten($text-color, 33%);
color-nav-sidebar-border: #252525,
color-identifier-badge-hover-border: #b9b8b5,
color-backdrop: rgba(0, 0, 0, 0.4),
);

@media (prefers-color-scheme: light) {
Expand Down

0 comments on commit 4cb0097

Please sign in to comment.