Skip to content

Commit ea65fbd

Browse files
Merge pull request #1642 from CleverCloud/cc-dialog/fix-responsive-close-btn
`cc-dialog`: fix close button positioning
2 parents d492147 + eed38b7 commit ea65fbd

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

src/components/cc-dialog/cc-dialog.js

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -204,16 +204,16 @@ export class CcDialog extends LitElement {
204204
border-radius: var(--cc-border-radius-default, 0.25em);
205205
box-shadow: 2px 4px 8px 0 rgb(0 0 0 / 12%);
206206
box-sizing: border-box;
207-
padding: var(--cc-dialog-padding);
207+
padding: var(--cc-dialog-padding, var(--default-dialog-padding));
208208
width: min(var(--cc-dialog-width, 38em), 80%);
209209
210-
--cc-dialog-padding: var(--cc-dialog-padding-xl);
210+
--default-dialog-padding: var(--cc-dialog-padding-xl);
211211
}
212212
213213
/* stylelint-disable-next-line media-feature-range-notation */
214214
@media screen and (max-width: 25em) {
215215
dialog {
216-
--cc-dialog-padding: var(--cc-dialog-padding-sm);
216+
--default-dialog-padding: var(--cc-dialog-padding-sm);
217217
}
218218
}
219219
@@ -230,6 +230,7 @@ export class CcDialog extends LitElement {
230230
231231
.dialog-close {
232232
--close-btn-size: 2em;
233+
--close-btn-offset: calc(var(--cc-dialog-padding, var(--default-dialog-padding)) / 1.5);
233234
234235
align-items: center;
235236
background: none;
@@ -244,7 +245,7 @@ export class CcDialog extends LitElement {
244245
margin-top: calc(var(--close-btn-size) * -1);
245246
position: sticky;
246247
top: 0;
247-
transform: translate(2.75em, -2.75em);
248+
transform: translate(var(--close-btn-offset), calc(var(--close-btn-offset) * -1));
248249
width: var(--close-btn-size);
249250
}
250251

0 commit comments

Comments
 (0)