-
Notifications
You must be signed in to change notification settings - Fork 49
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
feat(icon-button): change import button
desktop -> mobile
#1511
base: master
Are you sure you want to change the base?
Conversation
🦋 Changeset detectedLatest commit: 6b1d2c3 The changes in this PR will be included in the next version bump. This PR includes changesets to release 33 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 |
Bundle size report
|
Собрана новая демка. |
button
d->m import
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.
Мобильные и десктопные кнопки могут отличаться скруглением и цветом. Правки могут привести к непредвиденному поведению у того кто уже использует компонент на десктопе
@fulcanellee я не вижу в компоненте использование таких элементов кнопки. От кнопки тут только поведение/события. |
button
d->m importbutton
desktop -> mobile
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.
нужно подумать в сторону BaseButton
Опишите проблему
Замена универсального импорта кнопки на мобильную версию. Кнопка нужна только как функционал, а он между д/м одинаковый. Поэтому сплитить сам компонент icon-button на д/м (и все от него зависимые) не нужно
Внешний вид
Без изменений
https://core-ds.github.io/core-components/feat_ds_7758_35_cdbaa9e172a712deca7f8b2835383d33b9739997/?path=/story/iconbutton--icon-button