-
Notifications
You must be signed in to change notification settings - Fork 20
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
PLASMA-3619: add 'hover' trigger to Tooltip #1625
Conversation
Theme Builder app deployed! https://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-1625/ |
Documentation preview deployed! website:https://plasma.sberdevices.ru/pr/pr-1625/ |
a9ac5ab
to
b6bf167
Compare
Думаю, нужна еще дока по новому функционалу и тесты в b2c |
Заполни описание по шаблону, пожалуйста |
packages/plasma-new-hope/src/components/Tooltip/Tooltip.types.ts
Outdated
Show resolved
Hide resolved
packages/plasma-new-hope/src/components/Popover/Popover.styles.ts
Outdated
Show resolved
Hide resolved
доку добавлю, тесты там по ссылке с web |
b6bf167
to
e8458b1
Compare
Core
Tooltip, Popover
hover
What/why changed
Добавлено внутреннее состояния для тултипа: isOpen
Прокинуты свойства в Popover: trigger, closeOnEsc, closeOnOverlayClick. Так же, невидимый блок между target и контентом
📦 Published PR as canary version:
Canary Versions
✨ Test out this PR locally via: