VKCOM / VKUI

VKUI – это набор React-компонентов, с помощью которых можно создавать интерфейсы, внешне неотличимые от наших iOS и Android приложений.
https://vkcom.github.io/VKUI/
MIT License
1.03k stars 186 forks source link

[Bug][CustomScrollView]: не пересчитываются скроллы при изменении детей #7753

Open Semigradsky opened 1 week ago

Semigradsky commented 1 week ago

Описание

Пример: https://codesandbox.io/p/sandbox/hopeful-brown-9w5ffd

У .vkuiCustomScrollView__box-content высота 100%, соответственно при добавлении/удалении детей он не ресайзится, и пересчёт скролла не происходит.

Версия

6.7.2

В каких браузерах воспроизводится проблема?

No response

Шаги воспроизведения

No response

Ожидаемое поведение

Если убрать height: 100%, то дети перестанут быть растянуты на всю высоту.

Кажется проблему можно решить если для .vkuiCustomScrollView__box добавить display: grid. Тогда у .vkuiCustomScrollView__box-content можно убрать height: 100% и width: 100%, он и так будет растянут как элемент грида. И при добавлении детей сверх своей высоты он будет увеличиваться.

Скриншоты

No response

Пример с воспроизведением

https://codesandbox.io/p/sandbox/hopeful-brown-9w5ffd

Semigradsky commented 1 week ago

cc @EldarMuhamethanov =)

EldarMuhamethanov commented 1 week ago

Скорее всего с версии v7.0.0 перейдем на нативный скролл(#7699) так, что проблемы такой больше не возникнет

Semigradsky commented 1 week ago

Звучит многообещающе) Но хотелось бы в v6.x иметь исправленное, кажется не так просто будет на v7 перейти