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

Иконки в кнопках - разного размера #25

Closed
anmaslov opened this issue Mar 31, 2017 · 4 comments
Closed

Иконки в кнопках - разного размера #25

anmaslov opened this issue Mar 31, 2017 · 4 comments
Labels

Comments

@anmaslov
Copy link

Если добавить svg иконку кнопке, при разных размерах иконки и кнопки, размеры у иконки всегда одинаковые.

@zoxon
Copy link
Owner

zoxon commented Mar 31, 2017

Размеры иконки одинаковые по тому что они явно задаются в шаблоне
Решение которое вы предлагаете больше похоже на костыль. Нужно решать использовать размеры иконок по умолчанию или нет.

@anmaslov
Copy link
Author

А как быть с разными размерами кнопок? Для самой маленькой и самой большой кнопки размер иконки будет одинаковый.

@zoxon
Copy link
Owner

zoxon commented Mar 31, 2017

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

.icon
	&_size_normal
		size 16px
		
	&_size_extra-large
		size 20px
		
	&_size_large
		size 18px
		
	&_size_small
		size 14px
		
	&_size_extra-small
		size 12px

Ну и другие модификаторы тоже можно использовать.

Либо использовать элемент в вместо тега

button-size(padding, font-size, line-height)
	padding padding
	font-size font-size
	if line-height
		line-height line-height
	
	.button__icon
		size font-size

А скорее всего удобнее будет комбинировать эти подходы.
Вообщем надо пробовать и экспериментировать, я пока с такой ситуацией не встречался.

@anmaslov
Copy link
Author

ок, для меня второй подход более предпочтительней. Огромное спасибо за проделанную работу!

@zoxon zoxon added the bug label May 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants