diff --git a/src/components/Dialog/ButtonClose/ButtonClose.scss b/src/components/Dialog/ButtonClose/ButtonClose.scss index ab48918c09..84f2110321 100644 --- a/src/components/Dialog/ButtonClose/ButtonClose.scss +++ b/src/components/Dialog/ButtonClose/ButtonClose.scss @@ -1,14 +1,8 @@ -@use '../../variables'; +@use '../variables'; -$block: '.#{variables.$ns}dialog-btn-close'; - -#{$block} { +#{variables.$closeBtnBlock} { position: absolute; - // Offset from icon to the edges should be 20px (36px is Button's size, 8px is half of Button's padding) - // Icon size = 20px - // Button size = 36px - // Offset = 20 - ((36 - 20) / 2) = 14 - inset-block-start: 14px; - inset-inline-end: 14px; + inset-block-start: 12px; + inset-inline-end: 16px; z-index: 1; } diff --git a/src/components/Dialog/Dialog.scss b/src/components/Dialog/Dialog.scss index 2818ea0397..3e47b05712 100644 --- a/src/components/Dialog/Dialog.scss +++ b/src/components/Dialog/Dialog.scss @@ -1,15 +1,16 @@ -@use '../variables'; +@use './variables'; -$block: '.#{variables.$ns}dialog'; - -#{$block} { +#{variables.$block} { --_--side-padding: 32px; + --_--block-start-padding: 20px; + --_--block-end-padding: 24px; --_--close-button-space: 0px; position: relative; display: flex; flex-direction: column; width: var(--g-dialog-width, var(--_--width)); + padding-block: var(--_--block-start-padding) var(--_--block-end-padding); &_has-scroll { overflow-y: auto; @@ -32,7 +33,15 @@ $block: '.#{variables.$ns}dialog'; } } - &_has-close { - --_--close-button-space: 24px; + &:has(#{variables.$closeBtnBlock}) { + --_--close-button-space: 26px; + } + + &:has(#{variables.$headerBlock}) { + padding-block-start: 0; + } + + &:has(#{variables.$footerBlock}) { + padding-block-end: 0; } } diff --git a/src/components/Dialog/Dialog.tsx b/src/components/Dialog/Dialog.tsx index d1d34a171f..31bb371bcb 100644 --- a/src/components/Dialog/Dialog.tsx +++ b/src/components/Dialog/Dialog.tsx @@ -138,16 +138,7 @@ export function Dialog({ qa={qa} disableHeightTransition > -