-
Notifications
You must be signed in to change notification settings - Fork 198
[Deprecated]: Задепрекейтить FixedLayout #2674
Copy link
Copy link
Closed
Copy link
Labels
category:layoutcmp:fixed-layout (deprecated)deprecated by https://github.com/VKCOM/VKUI/issues/2674deprecated by https://github.com/VKCOM/VKUI/issues/2674researchtechnical tasktype:feature
Milestone
Description
Список задач на основе фильтра по issues.
Задачи, связанные с компонентом:
- [Bug] FixedLayout в положении top не считает отступ при скролле. Компонент начинает "висеть в воздухе" #1589
- [Bug] FixedLayout/PanelHeader скачет при swipeback после overscroll #1612
- [Bug][FixedLayout] Сломана центровка внутри
View>Panelна устройствах с бровкой вlandscapeрежиме #2422 - [Bug]: FixedLayout влечет за собой reflow #5414
- [Bug][PanelHeader]: Некорректная позиция after на первом рендере #6588
- [Bug]: PanelHeader: включать боковые отступы на основе area-insets в fixed режиме только на iOS платформе. #8115
Проблема
В рамках PR #9252 изучил возможность заменить в FixedLayout свойство position: fixed на position: sticky – вкратце, так сделать не можем, т.к. на position: fixed уже много чего завязано. Это будет breaking change без возможности покрыть его кодмодом, требующий от пользователей перелопатить вёрстку, в частности, в местах использования компонента с vertical="bottom", потому что сейчас можно не беспокоится о расположении компонента в потоке и его могут хоть где рендерить.
<View activePanel="root">
<Panel id="root">
<PanelHeader>Header</PanelHeader>
<FixedLayout vertical="bottom">Footer</FixedLayout>
<Group>Body</Group>
</Panel>
</View>Решение
Здесь проще будет всё же избавиться от FixedLayout в VKUI v11.
Что для этого нужно сделать сейчас:
- Реализовать задачу [Feature][Layout]: Расширить свойство
positionзначениемsticky#9253, чтобы появилась альтернатива в видеBoxсposition="sticky". - Отметить компонент
FixedLayoutкак@deprecated- комментарий, что удаляем в VKUI v10, используйте
Boxсposition: sticky; warnOnceс тем же комментарием
- комментарий, что удаляем в VKUI v10, используйте
- Добавить в документациях (
website/иFixedLayout.stories.tsx) инфомрацию про депрекейт и то, как переписать его наBox.
Далее отдельными задачами заменить использование FixedLayout внутри библиотеки:
- Удалить из
PullToRefresh. - Удалить из
PanelHeaderContext– использоватьPopover. - Избавиться от применения стилей
vkuiInternalFixedLayout--iosиvkuiInternalFixedLayout--vertical-bottom(сейчас используется вTabs).
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
category:layoutcmp:fixed-layout (deprecated)deprecated by https://github.com/VKCOM/VKUI/issues/2674deprecated by https://github.com/VKCOM/VKUI/issues/2674researchtechnical tasktype:feature
Type
Projects
Status
✅ Done