feat(plasma-web/b2c/new-hope): Add Radiobox component with new design and core
#775
Conversation
neretin-trike
requested review from
Yeti-or,
Yakutoc and
kayman233
as code owners
neretin-trike
force-pushed
the
neretinaa/add-new-hope-radiobox
branch
from
233cb1a to
b4b87f0
Compare
|
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-775/ |
neretin-trike
force-pushed
the
neretinaa/add-new-hope-radiobox
branch
from
b4b87f0 to
c404921
Compare
|
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-775/ |
|
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-775/ |
|
Component performance testing Result: 🟢 Success Check out report in job artifacts! |
|
|
||
| export const RadioGroup = React.forwardRef<HTMLInputElement, RadioGroupProps>(({ children, ...rest }, ref) => { | ||
| return ( | ||
| <Base ref={ref} {...rest}> |
There was a problem hiding this comment.
[Уточнение]
А вот у <Base/> какое будет displayName в react dev tools?
There was a problem hiding this comment.
скорее всего RadioGroup
p.s. проверил, вроде да
|
@neretin-trike Хотел спросить, а почему screenshots показывает diff? |
как я понял это из-за того, что раньше у чебокса / радибокоса была рамка того же цвета, что и заливка, а они рендерятся в браузерах немного по-другому. Но с появлением новых вьюх (primary, secondary и т.д.), так больше оставлять было нельзя, т.к. используемые там цвета прозрачные и при наложении друг на друга (бэграунд + рамка) выглядили криво. С рамкой как у нас сейчас в мастере: Без рамки (свойство |
| */ | ||
| export const Radiobox = RadioboxComponent; | ||
|
|
||
| export { RadioGroup } from '@salutejs/plasma-new-hope/styled-components'; |
There was a problem hiding this comment.
а с ним что-то ещё нужно будет сделать? Просто сейчас я его сделал таким же, как в plasma-hope
| `; | ||
|
|
||
| // NOTE: Необходимое переопределение токенов из компонента Checkbox т.к. используются его части | ||
| const mappingOverride = ` |
There was a problem hiding this comment.
чет конечно выглядит как забор в котором очень просто ошибиться =/
хотя идей как улучшить пока нет
neretin-trike
force-pushed
the
neretinaa/add-new-hope-radiobox
branch
from
c404921 to
3344961
Compare
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| value={item.value} | ||
| label={item.label} | ||
| disabled={item.disabled} | ||
| checked={value[item.value]} |
Check warning
Code scanning / Semgrep
Semgrep Finding: gitlab.eslint.detect-object-injection Warning
| value={item.value} | ||
| label={item.label} | ||
| disabled={item.disabled} | ||
| checked={value[item.value]} |
Check warning
Code scanning / Semgrep
Semgrep Finding: gitlab.eslint.detect-object-injection Warning
|
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-775/ |
|
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-775/ |
|
Component performance testing Result: 🟢 Success Check out report in job artifacts! |
|
🚀 This PR is included in version: @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected] 🚀 |
2 similar comments
|
🚀 This PR is included in version: @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected] 🚀 |
|
🚀 This PR is included in version: @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected] 🚀 |
|
🚀 This PR is included in version: @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected] 🚀 |
2 similar comments
|
🚀 This PR is included in version: @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected] 🚀 |
|
🚀 This PR is included in version: @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected] 🚀 |
Добавлена реализация компонента
Radioboxв новой архитектуре.Добавлены сборки компонента для новых тем plasma_web и plasma_b2c с новыми пропсами:
view,disabled,focused.Добавлены сборки компонента для старых библиотек
@salutejs/plasma-web,@salutejs/plasma-b2c.Исправлено поведение, при котором при фокусе на ссылку внутри описания, рамка обрезалась.
Удалён
CheckboxиRadioboxиз@salutejs/plasma-hope.#587
🐤 Download canary assets:
📦 Published PR as canary version:
Canary Versions✨ Test out this PR locally via:
Version
Published prerelease version:
@salutejs/[email protected]@salutejs/[email protected]@salutejs/[email protected]@salutejs/[email protected]@salutejs/[email protected]@salutejs/[email protected]@salutejs/[email protected]@salutejs/[email protected]@salutejs/[email protected]@salutejs/[email protected]Changelog
Release Notes
plasma-web: Pass
refto DropdownItem (#769)Добавлена возможность использовать свойство
refдля компонентаDropdownItemplasma-infra: Update version (#770)
Выключили sync package-lock's файлов для приватных пакетов.
What/why Changed
При попытке установить приватный пакет, получаем ошибку 404 - Not found.
Поэтому принято решение не запускать
npm i --package-lock-onlyдля приватнных пакетов.Добавлен флаг
--legacy-peer-depsчто бы исключить ошибки:plasma-infra: Update private packages version with @auto-it (#734)
Включили обновление версий для всех пакетов, включая приватные.
What/why Changed
Сделали патчи для
@auto-it/coreи@auto-it/npmчто бы можно было включить в обработку все пакеты.Как оказалось в
@auto-it/coreв методеgetLernaPackages(packages/core/src/utils/get-lerna-packages.ts)была логика фильтрующая пакеты по полюprivate:C помощью
patch-packageсделал патчи для изменения этой логики.В этом commit из песочнице можно посмотреть как это работает.
Запуск локально проекта
auto next --dry-run(Report what command will do but do not actually do anything)при изменениях в пакете
plasma-web:Так же откатили/удалили все что было связанно с ручным обновлением версий в приватных пакетах.
🚀 Enhancement
Linkcomponent with new design and core #696 (@neretin-trike)@salutejs/plasma-hopeRadioboxcomponent #775 (@neretin-trike)Spinnercomponent with new design and core #736 (@neretin-trike)Linkcomponent with new design and core #696 (@neretin-trike)refto DropdownItem #769 (@Yakutoc)@salutejs/plasma-webRadioboxcomponent with new design and core #775 (@neretin-trike)Checkboxfrom plasma-new-hope and update snapshots #754 (@neretin-trike)Spinnercomponent with new design and core #736 (@neretin-trike)Linkcomponent with new design and core #696 (@neretin-trike)@salutejs/plasma-b2cRadioboxcomponent with new design and core #775 (@neretin-trike)Checkboxcomponent with new design and core #754 (@neretin-trike)Spinnercomponent with new design and core #736 (@neretin-trike)Linkcomponent with new design and core #696 (@neretin-trike)@salutejs/plasma-new-hopeRadioboxcomponent with new design and core #775 (@neretin-trike)Checkboxcomponent with new design and core #754 (@neretin-trike)Spinnercomponent with new design and core #736 (@neretin-trike)Linkcomponent with new design and core #696 (@neretin-trike)@salutejs/plasma-core,@salutejs/plasma-new-hopeSpinnercomponent with new design and core #736 (@neretin-trike)@salutejs/plasma-b2c,@salutejs/plasma-core,@salutejs/plasma-hope,@salutejs/plasma-icons,@salutejs/plasma-temple,@salutejs/plasma-ui,@salutejs/plasma-web,@salutejs/plasma-sb-utils@salutejs/plasma-b2c,@salutejs/plasma-core,@salutejs/plasma-web@salutejs/plasma-b2c,@salutejs/plasma-core,@salutejs/plasma-hope,@salutejs/plasma-web@salutejs/plasma-core,@salutejs/plasma-hope,@salutejs/plasma-web@salutejs/plasma-tokens@salutejs/plasma-tokens-utils@salutejs/plasma-tokens,@salutejs/plasma-tokens-utils@salutejs/plasma-ui🐛 Bug Fix
@salutejs/plasma-b2c,@salutejs/plasma-core,@salutejs/plasma-web@salutejs/plasma-b2c,@salutejs/plasma-core,@salutejs/plasma-hope,@salutejs/plasma-web@salutejs/plasma-core,@salutejs/plasma-hope@salutejs/plasma-b2c,@salutejs/plasma-core,@salutejs/plasma-hope,@salutejs/plasma-icons,@salutejs/plasma-temple,@salutejs/plasma-ui,@salutejs/plasma-web,@salutejs/plasma-sb-utils@salutejs/plasma-coredev@salutejs/plasma-b2c,@salutejs/plasma-new-hope,@salutejs/plasma-web@salutejs/plasma-b2c,@salutejs/plasma-core,@salutejs/plasma-hope,@salutejs/plasma-icons,@salutejs/plasma-new-hope,@salutejs/plasma-temple,@salutejs/plasma-ui,@salutejs/plasma-web,@salutejs/plasma-sb-utils@salutejs/plasma-b2c,@salutejs/plasma-hope,@salutejs/plasma-new-hope,@salutejs/plasma-web@salutejs/plasma-b2c,@salutejs/plasma-core,@salutejs/plasma-hope,@salutejs/plasma-icons,@salutejs/plasma-temple,@salutejs/plasma-ui,@salutejs/plasma-web,@salutejs/plasma-sb-utils@salutejs/plasma-b2c,@salutejs/plasma-hope,@salutejs/plasma-web@salutejs/plasma-b2c,@salutejs/plasma-core,@salutejs/plasma-hope,@salutejs/plasma-icons,@salutejs/plasma-temple,@salutejs/plasma-tokens-b2b,@salutejs/plasma-tokens-b2c,@salutejs/plasma-tokens-web,@salutejs/plasma-tokens,@salutejs/plasma-ui,@salutejs/plasma-web,@salutejs/plasma-cy-utils,@salutejs/plasma-sb-utils,@salutejs/plasma-tokens-utils@salutejs/plasma-b2c,@salutejs/plasma-core,@salutejs/plasma-hope,@salutejs/plasma-icons,@salutejs/plasma-temple,@salutejs/plasma-tokens-web,@salutejs/plasma-ui,@salutejs/plasma-web,@salutejs/plasma-sb-utils,@salutejs/plasma-tokens-utils@salutejs/plasma-new-hope@salutejs/plasma-temple,@salutejs/plasma-uiAuthors: 7