Skip to content

Commit

Permalink
Fix drop shadow issue in Safari for menu and filter (Shopify#1212)
Browse files Browse the repository at this point in the history
  • Loading branch information
ludoboludo committed Jan 25, 2022
1 parent 182ebab commit 626128b
Show file tree
Hide file tree
Showing 5 changed files with 18 additions and 31 deletions.
7 changes: 1 addition & 6 deletions assets/base.css
Original file line number Diff line number Diff line change
Expand Up @@ -2543,12 +2543,7 @@ details[open] > .header__menu-item .icon-caret {
border-color: rgba(var(--color-foreground), var(--popup-border-opacity));
border-style: solid;
border-width: var(--popup-border-width);
filter: drop-shadow(
var(--popup-shadow-horizontal-offset)
var(--popup-shadow-vertical-offset)
var(--popup-shadow-blur-radius)
rgba(var(--color-shadow), var(--popup-shadow-opacity))
);
box-shadow: var(--popup-shadow-horizontal-offset) var(--popup-shadow-vertical-offset) var(--popup-shadow-blur-radius) rgba(var(--color-shadow), var(--popup-shadow-opacity));
}

.header__submenu.list-menu {
Expand Down
9 changes: 2 additions & 7 deletions assets/component-facets.css
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,7 @@

.facet-filters__field .select:after,
.facet-filters__field .select:before,
.mobile-facets__sort .select:after,
.mobile-facets__sort .select:after,
.mobile-facets__sort .select:before {
content: none;
}
Expand Down Expand Up @@ -238,12 +238,7 @@
border-style: solid;
border-color: rgba(var(--color-foreground), var(--popup-border-opacity));
border-radius: var(--popup-corner-radius);
filter: drop-shadow(
var(--popup-shadow-horizontal-offset)
var(--popup-shadow-vertical-offset)
var(--popup-shadow-blur-radius)
rgba(var(--color-shadow), var(--popup-shadow-opacity))
);
box-shadow: var(--popup-shadow-horizontal-offset) var(--popup-shadow-vertical-offset) var(--popup-shadow-blur-radius) rgba(var(--color-shadow), var(--popup-shadow-opacity));
background-color: rgb(var(--color-background));
position: absolute;
top: calc(100% + 0.5rem);
Expand Down
7 changes: 1 addition & 6 deletions assets/component-predictive-search.css
Original file line number Diff line number Diff line change
Expand Up @@ -10,12 +10,7 @@
z-index: 3;
border-bottom-right-radius: var(--popup-corner-radius);
border-bottom-left-radius: var(--popup-corner-radius);
filter: drop-shadow(
var(--popup-shadow-horizontal-offset)
var(--popup-shadow-vertical-offset)
var(--popup-shadow-blur-radius)
rgba(var(--color-shadow), var(--popup-shadow-opacity))
);
box-shadow: var(--popup-shadow-horizontal-offset) var(--popup-shadow-vertical-offset) var(--popup-shadow-blur-radius) rgba(var(--color-shadow), var(--popup-shadow-opacity));
}

.predictive-search--search-template {
Expand Down
7 changes: 1 addition & 6 deletions assets/disclosure.css
Original file line number Diff line number Diff line change
Expand Up @@ -32,12 +32,7 @@
z-index: 2;
background-color: rgb(var(--color-background));
border-radius: var(--popup-corner-radius);
filter: drop-shadow(
var(--popup-shadow-horizontal-offset)
var(--popup-shadow-vertical-offset)
var(--popup-shadow-blur-radius)
rgba(var(--color-shadow), var(--popup-shadow-opacity))
);
box-shadow: var(--popup-shadow-horizontal-offset) var(--popup-shadow-vertical-offset) var(--popup-shadow-blur-radius) rgba(var(--color-shadow), var(--popup-shadow-opacity));
}

.disclosure__item {
Expand Down
19 changes: 13 additions & 6 deletions assets/section-main-product.css
Original file line number Diff line number Diff line change
Expand Up @@ -839,12 +839,19 @@ a.product__text {
border-color: rgba(var(--color-foreground), var(--popup-border-opacity));
border-style: solid;
border-width: var(--popup-border-width);
filter: drop-shadow(
var(--popup-shadow-horizontal-offset)
var(--popup-shadow-vertical-offset)
var(--popup-shadow-blur-radius)
rgba(var(--color-foreground), var(--popup-shadow-opacity))
);
box-shadow: var(--popup-shadow-horizontal-offset) var(--popup-shadow-vertical-offset) var(--popup-shadow-blur-radius) rgba(var(--color-shadow), var(--popup-shadow-opacity));
}

.product-popup-modal__content.focused {
box-shadow: 0 0 0 0.3rem rgb(var(--color-background)),
0 0 0.5rem 0.4rem rgba(var(--color-foreground), 0.3),
var(--popup-shadow-horizontal-offset) var(--popup-shadow-vertical-offset) var(--popup-shadow-blur-radius) rgba(var(--color-shadow), var(--popup-shadow-opacity));
}

.product-popup-modal__content:focus-visible{
box-shadow: 0 0 0 0.3rem rgb(var(--color-background)),
0 0 0.5rem 0.4rem rgba(var(--color-foreground), 0.3),
var(--popup-shadow-horizontal-offset) var(--popup-shadow-vertical-offset) var(--popup-shadow-blur-radius) rgba(var(--color-shadow), var(--popup-shadow-opacity));
}

@media screen and (min-width: 750px) {
Expand Down

0 comments on commit 626128b

Please sign in to comment.