Skip to content
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

chore: storybook 7 #669

Merged
merged 1 commit into from
May 17, 2023
Merged

chore: storybook 7 #669

merged 1 commit into from
May 17, 2023

Conversation

v-gevak
Copy link
Contributor

@v-gevak v-gevak commented May 15, 2023

Что изменилось:

  • Стори теперь на mdx2. Изменений немного, в наших сторях только пришлось комменты заменить с <!-- на {/*, заменить <React.Fragment> на <> и строки обернуть в курли скобки (проблема с кирилицей, нужна писать вот так - title={'заголовок'}, чтобы получить эту строку на выходе). Ну и стори теперь не падают из-за пустых строк.
  • Manager теперь собирается с помощью esbuild, очень быстро, но повлиять на его конфиг невозможно. Соответственно там никакого postcss и прочего нет и быть не может
  • Переключение Docs/Canvas переехало в левую сайд-панель
  • Пермалинки не работают с docs. Работают только с Canvas.
  • Для рендера md-разметки нужно использовать специальный Markdown враппер. Было - <Readme />, стало <Markdown>{Readme}</Markdown>
  • Кириллица теперь корректно отображается в урле.
  • Плагин storybook-addons-live-examples пришлось доработать, новый mdx2-loader перестал парсить метаинформацию(live, mobileOnly, mobileWidth и т.п), теперь это делается через дополнительный плагин remark-mdx-code-meta, который нужно подключать в webpack конфиге (см patchWebpackConfig). И мету теперь нужно передавать как jsx-аттрибут. Было mobileWidth=640, стало mobileWidth={640}

Дополнительно:

  • Скрины теперь делаются на месте. Без копирования результат на отдельную страницу, как это было раньше.
  • Styrene шрифты сохранены в проект, из-за них моргали тесты, так как они не всегда успевают прогрузиться
  • Сборка сторибука переведена на modern-версии компонентов

С чем осталось разобраться:

  • Метрика. Пока не работает совсем
  • Исправить все ссылки по проекту.
  • Разобраться с css sourceMaps. В devTools браузера ссылки ведут на неверный css-файл

@changeset-bot
Copy link

changeset-bot bot commented May 15, 2023

⚠️ No Changeset found

Latest commit: 0384161

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@core-ds-bot
Copy link
Collaborator

Собрана новая демка.

@coveralls
Copy link

coveralls commented May 15, 2023

Pull Request Test Coverage Report for Build 5003304751

  • 2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 80.414%

Totals Coverage Status
Change from base Build 4983998705: 0.0%
Covered Lines: 7937
Relevant Lines: 8959

💛 - Coveralls

@v-gevak v-gevak force-pushed the chore/storybook-7 branch 3 times, most recently from d6ccd73 to 0f362f3 Compare May 16, 2023 08:29
@v-gevak v-gevak force-pushed the chore/storybook-7 branch 3 times, most recently from c2d655c to 66fadd0 Compare May 16, 2023 13:18
@Valeri8888

This comment was marked as resolved.

@Valeri8888

This comment was marked as resolved.

@v-gevak v-gevak force-pushed the chore/storybook-7 branch 2 times, most recently from 85431e9 to 1466cd3 Compare May 17, 2023 08:33
@v-gevak

This comment was marked as resolved.

@v-gevak

This comment was marked as resolved.

@reme3d2y
Copy link
Contributor

Сейчас даркмод в темах site и corp сломался :(

image image

@core-ds core-ds deleted a comment from core-ds-bot May 17, 2023
@core-ds core-ds deleted a comment from core-ds-bot May 17, 2023
@core-ds core-ds deleted a comment from core-ds-bot May 17, 2023
@core-ds core-ds deleted a comment from core-ds-bot May 17, 2023
@core-ds core-ds deleted a comment from core-ds-bot May 17, 2023
@core-ds core-ds deleted a comment from core-ds-bot May 17, 2023
@core-ds core-ds deleted a comment from core-ds-bot May 17, 2023
@core-ds core-ds deleted a comment from core-ds-bot May 17, 2023
@core-ds-bot
Copy link
Collaborator

Собрана новая демка.

@v-gevak
Copy link
Contributor Author

v-gevak commented May 17, 2023

Сейчас даркмод в темах site и corp сломался :(

Починил

@core-ds-bot
Copy link
Collaborator

Собрана новая демка.

@v-gevak v-gevak merged commit c9da2ca into master May 17, 2023
5 of 7 checks passed
@v-gevak v-gevak deleted the chore/storybook-7 branch May 17, 2023 12:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants