feat(plasma-web): Added TabsController in order to use keyboard arrows
#687
Conversation
|
|
||
| return ( | ||
| <TabsContext.Provider value={{ refs }}> | ||
| <Tabs ref={listRef} stretch={stretch}> |
There was a problem hiding this comment.
Сейчас не совсем непонятно, почему у нас именно 2 компоненты - Tabs и TabsController, когда как у TabsController вроде как та же функциональность, но теперь есть возможность использовать стрелочки. Почему мы оставляем Tabs? И нормально ли, что у них api отличается - у TabsController(появляются items, itemsindex, onIndexChange)?
There was a problem hiding this comment.
Эти компоненты переименуются v2.0:
Tabs => TabsRoot
TabsController => Tabs
TabsRoot & TabItem будут использоваться для кастомной сборки
Tabs останется "умным" компонентом, с которым не нужно делать лишних настроек
fanisco
force-pushed
the
fanisco/feat-tabs-arrows
branch
2 times, most recently
from
749c51a
to
99e3e63
Compare
fanisco
changed the title
TabsController in order to use keyboard arrows
fanisco
force-pushed
the
fanisco/feat-tabs-arrows
branch
from
99e3e63
to
42aba12
Compare
fanisco
force-pushed
the
fanisco/feat-tabs-arrows
branch
from
42aba12
to
699d34b
Compare
|
Demo canvas app deployed! |
|
Documentation preview deployed! docs: http://plasma.sberdevices.ru/pr-687/ |
1 similar comment
|
Documentation preview deployed! docs: http://plasma.sberdevices.ru/pr-687/ |
fanisco
requested review from
LamaEats,
OgNellis,
VladislavPetyukevich and
Yeti-or
as code owners
📦 Published PR as canary version:
Canary Versions✨ Test out this PR locally via:
npm install @sberdevices/plasma-docs@0.1.0-canary.687.5f8b1e9d1c77ab9ade2e51de2ed471ca45a359c9.0 npm install @sberdevices/demo-canvas-app@0.23.0-canary.687.5f8b1e9d1c77ab9ade2e51de2ed471ca45a359c9.0 npm install @sberdevices/plasma-ui@1.46.0-canary.687.5f8b1e9d1c77ab9ade2e51de2ed471ca45a359c9.0 npm install @sberdevices/plasma-web@1.44.0-canary.687.5f8b1e9d1c77ab9ade2e51de2ed471ca45a359c9.0 npm install @sberdevices/extract-sb-docgen-info@0.1.0-canary.687.5f8b1e9d1c77ab9ade2e51de2ed471ca45a359c9.0 npm install @sberdevices/showcase@0.51.0-canary.687.5f8b1e9d1c77ab9ade2e51de2ed471ca45a359c9.0 # or yarn add @sberdevices/plasma-docs@0.1.0-canary.687.5f8b1e9d1c77ab9ade2e51de2ed471ca45a359c9.0 yarn add @sberdevices/demo-canvas-app@0.23.0-canary.687.5f8b1e9d1c77ab9ade2e51de2ed471ca45a359c9.0 yarn add @sberdevices/plasma-ui@1.46.0-canary.687.5f8b1e9d1c77ab9ade2e51de2ed471ca45a359c9.0 yarn add @sberdevices/plasma-web@1.44.0-canary.687.5f8b1e9d1c77ab9ade2e51de2ed471ca45a359c9.0 yarn add @sberdevices/extract-sb-docgen-info@0.1.0-canary.687.5f8b1e9d1c77ab9ade2e51de2ed471ca45a359c9.0 yarn add @sberdevices/showcase@0.51.0-canary.687.5f8b1e9d1c77ab9ade2e51de2ed471ca45a359c9.0