Skip to content

Commit 28b206a

Browse files
committed
feat(pie-modal): DSW-2172 replace media query to include-media in pie-modal
1 parent 8c8529a commit 28b206a

File tree

1 file changed

+15
-15
lines changed

1 file changed

+15
-15
lines changed

packages/components/pie-modal/src/modal.scss

+15-15
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,11 @@
11
@use '@justeattakeaway/pie-css/scss' as p;
2+
@use '@justeattakeaway/pie-css/scss/settings' as *;
23
@use '@justeat/pie-design-tokens/dist/jet.scss' as dt;
34
@use 'dialog-polyfill/dist/dialog-polyfill.css';
45

56
.c-modal {
67
// Custom Property Declarations
78
// These are defined here instead of :host to encapsulate them inside Shadow DOM
8-
$breakpoint-wide: 768px;
9-
109
--modal-size-s: 450px;
1110
--modal-size-m: 600px;
1211
--modal-size-l: 1080px;
@@ -23,7 +22,8 @@
2322

2423
// We need to override the icon sizes at different screen sizes regardless of size prop passed in
2524
pie-icon-button {
26-
@media (width < $breakpoint-wide) {
25+
26+
@include media('<md') {
2727
--btn-dimension: 40px;
2828
}
2929
}
@@ -50,7 +50,7 @@
5050
--modal-margin-large: var(--dt-spacing-j);
5151
--modal-margin-block: var(--modal-margin-small);
5252

53-
@media (min-width: $breakpoint-wide) {
53+
@include media('>md') {
5454
--modal-margin-block: var(--modal-margin-large);
5555
}
5656

@@ -69,7 +69,7 @@
6969
&.c-modal--small {
7070
--modal-max-inline-size: var(--modal-size-s);
7171

72-
@media (min-width: $breakpoint-wide) {
72+
@include media('>md') {
7373
--modal-margin-block: var(--modal-margin-large);
7474
}
7575
}
@@ -87,7 +87,7 @@
8787
// Fullscreen styles
8888
&.c-modal--large,
8989
&.c-modal--medium.c-modal--fullWidthBelowMid {
90-
@media (width < $breakpoint-wide) {
90+
@include media('<md') {
9191
--modal-margin-block: var(--modal-margin-none);
9292
--modal-border-radius: var(--dt-radius-rounded-none);
9393
--modal-block-size: 100%;
@@ -110,7 +110,7 @@
110110

111111
&.c-modal--large,
112112
&.c-modal--fullWidthBelowMid.c-modal--medium {
113-
@media (width < $breakpoint-wide) {
113+
@include media('<md') {
114114
margin-block-start: var(--dt-spacing-none);
115115
max-block-size: 100%;
116116
}
@@ -133,13 +133,13 @@
133133
gap: var(--modal-button-spacing);
134134
padding: var(--modal-footer-padding);
135135

136-
@media (min-width: $breakpoint-wide) {
136+
@include media('>md') {
137137
--modal-footer-padding: var(--dt-spacing-e);
138138
}
139139

140140
&.c-modal-footer--stackedActions {
141141
// TODO: Move breakpoint sizes into shared CSS component utilities
142-
@media (width < $breakpoint-wide) {
142+
@include media('<md') {
143143
flex-direction: column;
144144
}
145145
}
@@ -167,7 +167,7 @@
167167
margin-inline-end: var(--dt-spacing-d);
168168
margin-block: 14px; // This is deliberately not a custom property
169169

170-
@media (min-width: $breakpoint-wide) {
170+
@include media('>md') {
171171
--modal-header-font-size: calc(var(--dt-font-heading-m-size--wide) * 1px);
172172
--modal-header-font-line-height: calc(var(--dt-font-heading-m-line-height--wide) * 1px);
173173
margin-inline-start: var(--dt-spacing-e);
@@ -180,15 +180,15 @@
180180
.c-modal-backBtn + .c-modal-heading {
181181
margin-inline-start: var(--dt-spacing-b);
182182

183-
@media (min-width: $breakpoint-wide) {
183+
@include media('>md') {
184184
margin-inline-start: var(--dt-spacing-c);
185185
}
186186
}
187187

188188
&.c-modal--dismissible .c-modal-heading {
189189
margin-inline-end: var(--dt-spacing-d);
190190

191-
@media (min-width: $breakpoint-wide) {
191+
@include media('>md') {
192192
margin-inline-end: var(--dt-spacing-e);
193193
}
194194
}
@@ -200,7 +200,7 @@
200200
margin-inline-start: var(--dt-spacing-b);
201201
margin-inline-end: var(--dt-spacing-none);
202202

203-
@media (min-width: $breakpoint-wide) {
203+
@include media('>md') {
204204
margin-block-start: var(--dt-spacing-c);
205205
margin-block-end: var(--dt-spacing-c);
206206
margin-inline-start: var(--dt-spacing-c);
@@ -215,7 +215,7 @@
215215
margin-inline-start: var(--dt-spacing-none);
216216
margin-inline-end: var(--dt-spacing-b);
217217

218-
@media (min-width: $breakpoint-wide) {
218+
@include media('>md') {
219219
margin-block-start: var(--dt-spacing-c);
220220
margin-block-end: var(--dt-spacing-c);
221221
margin-inline-start: var(--dt-spacing-none);
@@ -234,7 +234,7 @@
234234
--modal-content-min-block-size: var(--dt-spacing-j);
235235

236236

237-
@media (min-width: $breakpoint-wide) {
237+
@include media('>md') {
238238
--modal-content-padding-inline: var(--dt-spacing-e);
239239
}
240240

0 commit comments

Comments
 (0)