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

add text utils #240

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

add text utils #240

wants to merge 1 commit into from

Conversation

svyatsvyat
Copy link
Collaborator

No description provided.

@pacahon
Copy link
Contributor

pacahon commented Jul 18, 2019

Давай такие штуки по необходимости добавлять. Где сейчас они нужны и без них не обойтись?
Я бы может и text-small попытался выпилить. Тока надо проверить, можно ли его безопасно заменить на card__content _small.

@svyatsvyat
Copy link
Collaborator Author

Используются не только в карточках, но и для оформления текста в подписях например

@pacahon
Copy link
Contributor

pacahon commented Jul 22, 2019

А что за подписи?

@svyatsvyat
Copy link
Collaborator Author

Ну какие-то подписи которые не в рамках компонентов
image

@pacahon
Copy link
Contributor

pacahon commented Jul 22, 2019

Может тогда имеет смысл переписать стили для стандартного тега small? Его из 5го стандарта не убрали, в отличие от big. Либо оставить text-small, но не вводить text-lg (по аналогии со стандартом). А может имеет смысл даже оба варианта поддержать (и тег small и .text-small)

@pacahon pacahon added enhancement New feature or request minor labels Oct 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request minor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants