Skip to content

add control variables - #65

Open
koloskof wants to merge 1 commit into
masterfrom
card-24268059
Open

add control variables#65
koloskof wants to merge 1 commit into
masterfrom
card-24268059

Conversation

@koloskof

Copy link
Copy Markdown
Member

Выкристаллизовался финальный список переменных контролов. Имеет максимальную гибкость, обкатан на Лего компонентс в нескольких Яндексовых Темах.

Comment thread package.json
{
"name": "whitepaper-bem",
"version": "2.2.0",
"version": "2.3.0",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Давайте так не делать в пул-реквестах. Давайте отправлять PR в ветку с новой версией и уже в ней менять. А то в каждом PR новая версия. Во-первых, конфликты будут. А во-вторых, катить новую мажорную версию ради одной фичи — ну такое.




.theme_color_whitepaper-brand {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

theme_control




.theme_color_whitepapers-default {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

theme_control
почему whitepaperS?




.theme_color_whitepaper-inverse {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

тут тоже

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.

2 participants