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

[Feature][Counter]: Обновить пропы Counter #7805

Open
qurle opened this issue Oct 21, 2024 · 8 comments · May be fixed by #7919
Open

[Feature][Counter]: Обновить пропы Counter #7805

qurle opened this issue Oct 21, 2024 · 8 comments · May be fixed by #7919

Comments

@qurle
Copy link
Contributor

qurle commented Oct 21, 2024

Описание

Давайте разобьём проп каунтера mode на mode и appearance. Как обычно, mode будет отвечать за контрастность элемента, а appearance — за цвета.

mode: primary | contrast | tertiary (простите, secondary не завезли)
appearance: accent | neutral | accent-green | accent-red (во многом по аналогии с бейджом)

+ хотелось бы дать возможность задавать цвет вручную (у нас это типа appearance = custom). Вижу два пути: через традиционные стили или через проп color, который будет красить фон в режиме primary (а цвет текста вычислять) и текст в режимах contrast и tertiary.

Текущие моды:
inherit → оставляем, или заменяем на mode = undefined, appearance = undefined
primarymode = "primary" appearance = "accent"
secondarymode = "primary" appearance = "neutral"
prominentmode = "primary" appearance = "accent-red"
contrastmode = "contrast" appearance = "accent"

Макет в фигме ↗️

Скриншоты

image
@qurle
Copy link
Contributor Author

qurle commented Oct 21, 2024

Как думаете, успеем такую штуку или уже поздновато закинул?

@inomdzhon
Copy link
Contributor

inomdzhon commented Oct 22, 2024

Как думаете, успеем такую штуку или уже поздновато закинул?

Сделаем в рамках минора после релиза v7.0.0, старые значения mode задепрекейтим и удалим в v8

@inomdzhon inomdzhon added this to the v7.1.0 milestone Oct 22, 2024
@EldarMuhamethanov EldarMuhamethanov self-assigned this Nov 7, 2024
@EldarMuhamethanov
Copy link
Contributor

@qurle

через проп color, который будет красить фон в режиме primary (а цвет текста вычислять)

Есть примерное представление, по какому принципу вычислять цвет текста? Может формула какая-то?

@qurle
Copy link
Contributor Author

qurle commented Nov 7, 2024

@EldarMuhamethanov
Что-то вроде такого мб?

На самом деле тут есть вопросики к тёмной теме и работе с токенами, поэтому можно вынести в отдельную задачу и отложить до востребованности.

@EldarMuhamethanov
Copy link
Contributor

На самом деле тут есть вопросики к тёмной теме и работе с токенами, поэтому можно вынести в отдельную задачу и отложить до востребованности.

Вынести appearance=custom ?

@qurle
Copy link
Contributor Author

qurle commented Nov 7, 2024

Йес

@EldarMuhamethanov
Copy link
Contributor

@qurle Может пока для mode=primary appearance=custom сделаем так, чтобы color менял только цвет фона?

@qurle
Copy link
Contributor Author

qurle commented Nov 8, 2024

@EldarMuhamethanov, да, можно. Тогда текст будет цветом text contrast.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 👀 In Review
Development

Successfully merging a pull request may close this issue.

3 participants