feat(gap): added numeric size values #1037
Conversation
Valeri8888
requested review from
reme3d2y,
v-gevak,
EGNKupava and
SiebenSieben
as code owners
🦋 Changeset detectedLatest commit: 5609ef4 The changes in this PR will be included in the next version bump. This PR includes changesets to release 4 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Valeri8888
force-pushed
the
feat/gap_new_size
branch
2 times, most recently
from
553e162
to
cea51ce
Compare
Pull Request Test Coverage Report for Build 7527918536
💛 - Coveralls |
Valeri8888
force-pushed
the
feat/gap_new_size
branch
from
cea51ce
to
1f95dc6
Compare
|
Собрана новая демка. |
Valeri8888
force-pushed
the
feat/gap_new_size
branch
from
1f95dc6
to
52ca293
Compare
| '6xl': 'size-72', | ||
| '7xl': 'size-96', | ||
| '8xl': 'size-128', | ||
| 0: 'size-0', |
There was a problem hiding this comment.
А ноль зачем? Зачем юзать компонент, если отступ не нужен? Непонятно
There was a problem hiding this comment.
ну может надо по условию менять размер c 0 на какой-то
я не знаю
@Oladii
There was a problem hiding this comment.
вот я затевал их именно для кейсов где мы знаем что отступ тут можно настраивать, но он может быть нулевым.
There was a problem hiding this comment.
Если отступ может быть нулевым, то лучше не рендерить компонент, чем устанавливать у него нулевую ширину.
{gapSize && <Gap size={gapSize} />}There was a problem hiding this comment.
Не думаю, что кто-то этим будет пользоваться
There was a problem hiding this comment.
Я бы оставил, в том же SDUI может быть удобнее тупо кинуть в компонент значение, чем добавлять еще доп. условие + могут быть проблемы с типами, если у них будет 0, а у нас нет
|
Собрана новая демка. |
| '6xl': 'size-72', | ||
| '7xl': 'size-96', | ||
| '8xl': 'size-128', | ||
| 0: 'size-0', |
There was a problem hiding this comment.
Я бы оставил, в том же SDUI может быть удобнее тупо кинуть в компонент значение, чем добавлять еще доп. условие + могут быть проблемы с типами, если у них будет 0, а у нас нет
Valeri8888
force-pushed
the
feat/gap_new_size
branch
2 times, most recently
from
978eb97
to
5609ef4
Compare
|
Собрана новая демка. |
No description provided.