fix(ModalPage): account for prev expandable value (#6637)

* fix(ModalPage): apply DOM changes before height calculation * fix(ModalPage): apply DOM changes before height calculation
VKCOM · Feb 28, 2024 · c05368c · c05368c
1 parent 3dd9c1c
commit c05368c
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/packages/vkui/src/components/ModalRoot/ModalRoot.tsx b/packages/vkui/src/components/ModalRoot/ModalRoot.tsx
@@ -669,6 +669,7 @@ function initPageModal(modalState: ModalsStateEntry) {
   const bottomInsetHeight = bottomInset?.offsetHeight || 0;
   const contentHeight = contentElementHeight + bottomInsetHeight;
   let prevTranslateY = modalState.translateY;
+  let prevExpandable = modalState.expandable;
 
   modalState.expandable =
     contentHeight > (contentElement?.clientHeight ?? 0) || modalState.settlingHeight === 100;
@@ -708,11 +709,10 @@ function initPageModal(modalState: ModalsStateEntry) {
     hiddenRange = [translateY + 25, translateY + 100];
   }
 
+  // Свойство expandable может измениться из-за высоты контента, в таком случае на всю высоту не разворачиваем
+  const shouldExpand = prevExpandable && modalState.expandable;
   // Если модалка может открываться на весь экран, и новый сдвиг больше предыдущего, то откроем её на весь экран
-  if (
-    (modalState.expandable && translateY > (prevTranslateY ?? 100)) ||
-    modalState.settlingHeight === 100
-  ) {
+  if ((shouldExpand && translateY > (prevTranslateY ?? 100)) || modalState.settlingHeight === 100) {
     translateY = 0;
   }