-
Notifications
You must be signed in to change notification settings - Fork 186
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(FixedLayout): revert to 1364 PR number #2472
Conversation
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 6ef1bab:
|
size-limit report 📦
|
👀 Styleguide deployedSee the styleguide for this PR at https://vkcom.github.io/VKUI/pull/2472/ |
e2e должны как раз такое отлавливать |
Согласен с Антоном. Не хватает скриншотного теста на такое отображение шапки и других FixedLayout'ов. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Исправление сломало `SplitLayout`, в котором в текущей версии важно, чтобы у `FixedLayout` было `left: auto`
Старое описание
Поправил центровку при фиксированной ширине.
Здесь #2414 (коммит 230b4dc) не учёл это.
Использовал старый добрый метод 😄
UPD
После изменения #2414 (коммит 230b4dc) в
v4.29.1
сломалась:центровка при фиксированной ширине
Демо
Было

Стало

[Bug][PanelHeader] Съезжает хеадер в многоколончатом интерфейсе #2475
поэтому ревертнул коммит 230b4dc
fix #2475