Combinations import done #1
Conversation
Основные фичи:Открытие Сторибуков
|
|
Чтобы протестировать аддон, нужно запустить с ним локальный Сторибук Это делается в пакете |
Drag'n'dropПеретаскивание компонентов на холст: Предварительно, функция должны быть включена в тулбаре: |
Генерация вариантовГенерация вариантов на основе выбраных пропсов Набор пропсов производится в панели: Как и drag'n'drop, фича должна быть включена в тулбаре: |
Импорт вариантовВарианты можно импортировать нажатием кнопки: |
packages/storybook-addon/src/components/CombinePanel/CombinePanel.tsx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Было бы неплохо, если бы в README было описано как это подключить, типо:
npm install storybook-to-figma-addon// .storybook/main.js
module.exports = {
// ...
addons: ["storybook-to-figma-addon"],
};Хотя бы самый минимум. Ну из мелочей - было бы здорово запустить prettier, чтобы привести код к единообразному виду. В целом есть еще места, где можно было бы типы получше описать, но это мелочи
|
btw, а чего ты не conventional commits используешь? |
packages/figma-plugin/src/modules/importFunctions/importVariants.ts
Outdated
Show resolved
Hide resolved
packages/figma-plugin/src/modules/importFunctions/importVariants.ts
Outdated
Show resolved
Hide resolved
packages/storybook-addon/src/components/DragWrapper/DragWrapper.tsx
Outdated
Show resolved
Hide resolved
|
@axaxaman подскажи у нас тут есть автоматика для |
Тут у нас нет никаких роботов, руками только |
| type : RequestType.GET, | ||
| timestamp: Date.now(), | ||
| payload: key, | ||
| } as FigmaStorageGetRequest; |
There was a problem hiding this comment.
А здесь и ниже точно нужен каст? Тут тайпскрипт без проблем должен типы резолвить
| } | ||
| } | ||
|
|
||
| async function sendRequestAndExpectResult(request : FigmaRequest) : Promise<any | undefined> { | ||
| async function sendRequestAndExpectResult(request: FigmaStorageRequest): Promise<unknown | undefined> { |
There was a problem hiding this comment.
А это по идее можно сделать женериком тогда каст на 26 строчке можно будет убрать
| window.parent.postMessage({ pluginMessage: request }, "https://www.figma.com"); | ||
| } | ||
|
|
||
| function createPromiseForRequestResult(request: FigmaRequest): Promise<any> { | ||
| function createPromiseForRequestResult(request: FigmaStorageRequest): Promise<any> { | ||
| return new Promise((resolve, reject) => { | ||
| window.addEventListener('message', (e) => { |
There was a problem hiding this comment.
А тут кажется есть проблема с тем что ивент листенеры не удаляются. Вообще я бы подумал над тем чтоб ивентлистенер создавался только один раз
P.S responce -> response
| this.storybooks = await this.figmaStorage.getItem("storybooks") as Storybook[]; | ||
| this.selectedId = await this.figmaStorage.getItem("selectedId") as string; |
There was a problem hiding this comment.
По идее тут тоже можно сделать дженерик чтоб не кастить, как по мне так красивее
No description provided.