feat(Group): Add Group.ExpandedContent to ignore group inner paddings #7396
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
size-limit report 📦
|
|
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. |
e2e tests |
👀 Docs deployed
Commit 5c664ce |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #7396 +/- ##
=======================================
Coverage 95.02% 95.03%
=======================================
Files 378 379 +1
Lines 11205 11215 +10
Branches 3674 3675 +1
=======================================
+ Hits 10648 10658 +10
Misses 557 557
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
mendrew
force-pushed
the
mendrew/7279/Group-ignore-inner-paddings
branch
from
c1d604f
to
0b2d8fa
Compare
old
В доке вот такое заметил, что теперь как 
понял почему #7395 (comment) |
mendrew
force-pushed
the
mendrew/7279/Group-ignore-inner-paddings
branch
from
4228b5f
to
d9b464f
Compare
mendrew
force-pushed
the
mendrew/5758/Group/add-submodules
branch
from
6e4033b
to
8c53f92
Compare
mendrew
force-pushed
the
mendrew/7279/Group-ignore-inner-paddings
branch
from
d9b464f
to
01cefc4
Compare
andrey-medvedev-vk
dismissed
inomdzhon’s stale review
The base branch was changed.
andrey-medvedev-vk
force-pushed
the
mendrew/7279/Group-ignore-inner-paddings
branch
from
01cefc4
to
5c664ce
Compare
inomdzhon
approved these changes
Aug 30, 2024
SevereCloud
approved these changes
Aug 30, 2024
andrey-medvedev-vk
deleted the
mendrew/7279/Group-ignore-inner-paddings
branch
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Описание
Добавили подкомпонент для игнорирования вертикальных и горизонтальных отступов
Group.Можно игнорировать либо вертикальные, либо горизонтальные отступы.
Возможно, что имеет смысл игнорировать сразу все отступы, но это можно сделать отдельным вариантом свойства
direction.Изменения
Group.Потребовалось, чтобы упростить понимание реализации
Group.ExpandedContent.Так как
Group.ExpandedContentвсегда будет находится внутриGroupи на него всегда будут влиять внутренние отступыGroup, будет логично внутренние отступы компенсировать за счёт отрицательных марджинов.Проще всего взять текущее значение паддинга, умножить на -1 и подставить в
marginGroup.ExpandedContent.Но чтобы это сделать надо где-то хранить текущее значение паддинга.
Так как паддинг по горизонтали и вертикали меняется относительно режима
mode = 'card' | 'plain'уGroup, то логично хранить паддинг в двух отдельных css-переменных:--vkui_internal--Group_padding_inlineи--vkui_internal--Group_padding_block.Можно один раз прописать
а дальше менять только css-переменные, в зависимости от режима.
К сожалению, не удалось избавиться от сss-переменной
--vkui_internal--Group_padding_size, которая используется для задания отступов в режимеcard.Тем не менее удалось встроить её в схему с основными переменными.
Переменная теперь имеет более специфичное имя:
--vkui_internal--Group_card_mode_padding_size.