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

[Button] [Input] [Token] Цвет текста на заблокированных контролах #2207

Closed
dzekh opened this issue Nov 9, 2020 · 1 comment · Fixed by #2304
Labels
bug good first issue Easy issue that you can do in several hours. Good for externalev delopers to start contribute.
Milestone

Comments

@dzekh
Copy link

dzekh commented Nov 9, 2020

Нужно изменить цвет текста на заблокированных кнопках, инпутах и токенах с disabled – #A0A0A0 на secondary – #808080, из-за того, что у цвета disabled с серым фоном маленький контраст. Приняли решение после обсуждения в стаффе. В библиотеках в Фигме цвет уже исправлен.

@dzekh dzekh added the bug label Nov 9, 2020
@dzekh dzekh added the good first issue Easy issue that you can do in several hours. Good for externalev delopers to start contribute. label Jan 22, 2021
@dzekh dzekh added this to the 3.0.0 milestone Jan 22, 2021
@draugi draugi linked a pull request Mar 4, 2021 that will close this issue
@dzekh
Copy link
Author

dzekh commented Mar 5, 2021

Если уточнять про контролы в которых этот цвет нужно поменять, это все контролы про которые можно сказать:
Внутри заблокированного контрола есть текст цвета #A0A0A0 и фон под этим текстом становится серым — #F2F2F2 (или rgb(242, 242, 242)). Если это так, то нужно поменять цвет текста с #A0A0A0 на #808080.
Если текст на белом фоне — не нужно трогать, оставить #A0A0A0.

Если это иконки — их цвет не нужно менять.

По хорошему, в идеальном мире этот цвет должен жить где-то в теме в одном месте. Но у нас скорее всего не так.

@zhzz zhzz closed this as completed in 09e27e9 May 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug good first issue Easy issue that you can do in several hours. Good for externalev delopers to start contribute.
Development

Successfully merging a pull request may close this issue.

1 participant