feat(plasma-ui): add labels for time pickers #749
Conversation
|
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-749/ |
|
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-749/ |
|
Component performance testing Result: 🟢 Success Check out report in job artifacts! |
Yakutoc
force-pushed
the
feat-ui-picker-labels
branch
from
1f178f4 to
5e83e5c
Compare
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
|
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-749/ |
|
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-749/ |
| color: ${whiteTertiary}; | ||
| ${footnote1}; | ||
| font-weight: normal; | ||
| font-size: ${({ size = DEFAULT_PICKER_SIZE }) => labelFontSize[size]}; |
Check warning
Code scanning / Semgrep
Semgrep Finding: gitlab.eslint.detect-object-injection
There was a problem hiding this comment.
кажется это не кореектно в этом месте
There was a problem hiding this comment.
@Yeti-or Ты про то что он кинул алерт?
There was a problem hiding this comment.
ну либо я не понял идею, типа можно заполютить object если в size прописать что-то не то
но это точно нам надо править?
There was a problem hiding this comment.
Нам точно править не нужно. Скорее нужно скорректировать правила.
There was a problem hiding this comment.
А ты понял где прочитать пример эксплоита?
There was a problem hiding this comment.
|
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-749/ |
|
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-749/ |
|
Component performance testing Result: 🟢 Success Check out report in job artifacts! |
Yakutoc
force-pushed
the
feat-ui-picker-labels
branch
from
d029f9f to
20b6374
Compare
Yakutoc
changed the title
|
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-749/ |
|
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-749/ |
|
Component performance testing Result: 🟢 Success Check out report in job artifacts! |
Yakutoc
requested review from
Yeti-or,
neretin-trike and
kayman233
as code owners
| color: ${whiteTertiary}; | ||
| ${footnote1}; | ||
| font-weight: normal; | ||
| font-size: ${({ size = DEFAULT_PICKER_SIZE }) => labelFontSize[size]}; |
There was a problem hiding this comment.
кажется это не кореектно в этом месте
Yakutoc
force-pushed
the
feat-ui-picker-labels
branch
from
20b6374 to
21c7e32
Compare
|
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-749/ |
|
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-749/ |
|
Component performance testing Result: 🟢 Success Check out report in job artifacts! |
Yakutoc
force-pushed
the
feat-ui-picker-labels
branch
from
21c7e32 to
c6cf110
Compare
|
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-749/ |
|
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-749/ |
|
Component performance testing Result: 🟢 Success Check out report in job artifacts! |
Yakutoc
force-pushed
the
feat-ui-picker-labels
branch
from
c6cf110 to
4ce30bd
Compare
|
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-749/ |
|
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-749/ |
|
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] 🚀 |
Release Notes
Добавлены
labelдля компонентаTimePicker.What/why Changed
Что бы было понятно с чем пользователь имеет дело было решено добавить label.
Стало
🐤 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]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-ui@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🐛 Bug Fix
@salutejs/plasma-b2c,@salutejs/plasma-hope,@salutejs/plasma-web@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-hope,@salutejs/plasma-web@salutejs/plasma-b2c,@salutejs/plasma-hope,@salutejs/plasma-new-hope,@salutejs/plasma-web@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-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-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