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

docs(): create new documentation #3469

Open
wants to merge 269 commits into
base: 5.x
Choose a base branch
from
Open

docs(): create new documentation #3469

wants to merge 269 commits into from

Conversation

SchwJ
Copy link
Member

@SchwJ SchwJ commented Jul 15, 2024

Проблема

Старая документация устарела и имеет ряд проблем.

Решение

Решено делать новую документацию на базе Storybook.

Текущий деплой здесь: https://ui.gitlab-pages.kontur.host/storybook-documentation/

Что реализовано:

  1. React UI

1.1. Страница для компонента (содержание)
1.1.1. Обновлены описания компонентов
1.1.2. Добавлены ссылки на Гайды и Figma (где они есть)
1.1.3. Базовый пример контрола
1.1.4. Таблица пропов. Берутся также пропы компонента, от которого наследуемся. Обновлены описания пропов
1.1.5. Истории (примеры использования компонента). С возможностью посмотреть код.

1.2. Информационные страницы
1.2.1. Вынесены информационные страницы и отдельно страницы версионирования
1.2.2. Собрана Versions из старых версий (данные берутся из json и из задеплоенных версий в гитлабе)

1.3. Группировка в левом меню: компоненты разделены на логические группы, созданы группы версионирования и информационных страниц. Группы отсортированы в порядке, который мне кажется наиболее удобным.

1.4. Переключения локали, темы и фиче-флагов -- доступны только в дочернем сторибуке

1.5. Для каждой страницы собирается содержание


  1. React UI Validations

2.1. Страницы переведены на mdx

2.2. Примеры сделаны на страницах с помощью + jsx static ... , чтобы была возможность и "потыкать" и код посмотреть. (Потому что при использовании одних Story нет возможности посмотреть код. Предположительно из-за фильтра для mdx в filteredStorybooksWebpackRules)

2.3. Переключение фиче-флагов -- доступно только в дочернем сторибуке


  1. Царь-storybook

3.1. Информационные страницы (вынесены из react-ui и переведены на mdx)
3.1.1. Accessibility
3.1.2. Contributing
3.1.3. Ecosystem

3.2. Подключены сторибуки React UI и React UI Validations

Ссылки

Чек-лист перед запросом ревью

  1. Добавлены тесты на все изменения
    ⬜ unit-тесты для логики
    ✅ скриншоты для верстки и кросс-браузерности
    ⬜ нерелевантно

  2. Добавлена (обновлена) документация
    ✅ styleguidist для пропов и примеров использования компонентов
    ✅ jsdoc для утилит и хелперов
    ⬜ комментарии для неочевидных мест в коде
    ✅ прочие инструкции (README.md, contributing.md и др.)
    ⬜ нерелевантно

  3. Изменения корректно типизированы
    ✅ без использования any (см. PR 2856)
    ⬜ нерелевантно

  4. Прочее
    ❌ все тесты и линтеры на CI проходят -- не проходят 4 теста MaskedInput
    ✅ в коде нет лишних изменений
    ✅ заголовок PR кратко и доступно отражает суть изменений (он попадет в changelog)

SchwJ and others added 7 commits September 26, 2024 13:57
…ok-7

# Conflicts:
#	packages/react-ui/.storybook/preview.tsx
#	packages/react-ui/components/Button/__stories__/Button.stories.tsx
#	packages/react-ui/components/ComboBox/ComboBox.tsx
#	packages/react-ui/components/CurrencyLabel/CurrencyLabel.tsx
#	packages/react-ui/components/DateInput/DateInput.tsx
#	packages/react-ui/components/Dropdown/Dropdown.tsx
#	packages/react-ui/components/MenuFooter/MenuFooter.tsx
#	packages/react-ui/components/MenuHeader/MenuHeader.tsx
#	packages/react-ui/components/Paging/Paging.tsx
#	packages/react-ui/components/ScrollContainer/ScrollContainer.tsx
#	packages/react-ui/components/Toggle/Toggle.tsx
#	packages/react-ui/components/TokenInput/TokenInput.tsx
…ok-7

# Conflicts:
#	packages/react-ui/components/Button/Button.md
#	packages/react-ui/components/Button/Button.tsx
#	packages/react-ui/components/Link/Link.md
#	packages/react-ui/components/Link/Link.tsx
#	packages/react-ui/components/MaskedInput/MaskedInput.md
#	packages/react-ui/components/MaskedInput/MaskedInput.tsx
#	packages/react-ui/components/MaskedInput/__stories__/MaskedInput.stories.tsx
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants