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: Доки чеклист code review & css-переменные #121

Merged
merged 3 commits into from
Jul 15, 2022

Conversation

EGNKupava
Copy link
Contributor

🚀

  • Изменил компонент для отображения переменных. Теперь он умеет сворачиваться и отображать миксины.
  • Добавил доку для code review.
  • Поправил доку по css переменным.

@core-ds-bot
Copy link
Collaborator

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

@core-ds-bot
Copy link
Collaborator

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

@@ -31,69 +33,109 @@ import shadowsIndigo from '!!raw-loader!./shadows-indigo.css';
```css
@import '@alfalab/core-components/vars';
```
Для удобства использования все CSS-переменные экспортируются в виде JS-переменных:

Все CSS-переменные можно также экспортировать в виде JS-переменных:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Думаю, это можно куда-то в конец)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Я сейчас использую styled-components и прям удобно сверху доков брать этот импорт

@SiebenSieben
Copy link
Contributor

@Oladii есть что причесать в доках, запиши в бэклог, пожалуйста)

@reme3d2y
Copy link
Contributor

reme3d2y commented Jul 5, 2022

Чуть-чуть воздуха бы добавить в оформление чек-листов)) @Oladii

image

Иначе переменные в сборке не будут работать.
- При использовании других компонент из библиотеки необходимо следить,
чтобы импорт начинался с `@alfalab`.
> При использовании автоимпорта, иногда происходит ошибка ипорта из локальной папки `../component/src`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Оформление сломалось

@core-ds-bot
Copy link
Collaborator

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

@EGNKupava EGNKupava merged commit 46ad375 into master Jul 15, 2022
@EGNKupava EGNKupava deleted the docs/vars-mixins branch July 15, 2022 12:46
@core-ds-bot
Copy link
Collaborator

🎉 This PR is included in version 28.3.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants