Skip to content

Commit 92b0aea

Browse files
authored
fix(ModalPage): rm CSS padding prop in desktop (#8790)
Заменил безопасные отступы с `padding` на ограничение `max-block-size`/`max-inline-size` через `calc(<value> - 16 * 2)`. В связи с этим удалил лишний брейкпоинт `@media (max-height: 672px)`, который до этого рассчитывал на `padding-block` если экран `> 672px`.
1 parent 5dfd67a commit 92b0aea

File tree

1 file changed

+9
-14
lines changed

1 file changed

+9
-14
lines changed

packages/vkui/src/components/ModalPage/ModalPage.module.css

Lines changed: 9 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -30,29 +30,24 @@
3030

3131
/* Desktop */
3232
.hostDesktop {
33-
--vkui_internal_ModalPage--desktopSafeInsetBlock: calc(var(--vkui--spacing_size_2xl) * 2);
33+
--vkui_internal_ModalPage--desktopStretchedSize: calc(100% - var(--vkui--spacing_size_2xl) * 2);
3434
--vkui_internal_ModalPage--desktopMaxWidth: 100%;
3535
--vkui_internal_ModalPage--desktopMaxHeight: 640px;
3636
--vkui_internal_ModalPage--userHeight: auto;
3737

3838
position: relative;
3939
box-sizing: content-box;
40-
max-inline-size: var(--vkui_internal_ModalPage--desktopMaxWidth);
41-
max-block-size: var(--vkui_internal_ModalPage--desktopMaxHeight);
42-
padding-block: var(--vkui_internal_ModalPage--desktopSafeInsetBlock);
43-
padding-inline: var(--vkui--spacing_size_xl);
40+
max-inline-size: min(
41+
var(--vkui_internal_ModalPage--desktopMaxWidth),
42+
var(--vkui_internal_ModalPage--desktopStretchedSize)
43+
);
44+
max-block-size: min(
45+
var(--vkui_internal_ModalPage--desktopMaxHeight),
46+
var(--vkui_internal_ModalPage--desktopStretchedSize)
47+
);
4448
margin-block: auto;
4549
}
4650

47-
/* Неизвестный брейкпоинт */
48-
@media (max-height: 672px) {
49-
.hostDesktop {
50-
--vkui_internal_ModalPage--desktopMaxHeight: calc(
51-
100% - var(--vkui_internal_ModalPage--desktopSafeInsetBlock) * 2
52-
);
53-
}
54-
}
55-
5651
.hostDesktopMaxWidthS {
5752
--vkui_internal_ModalPage--desktopMaxWidth: var(--vkui--size_popup_small--regular);
5853
}

0 commit comments

Comments
 (0)