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

[Bug]: IconButton проставляет внутренние отступы не для всех иконок #5130

Open
SevereCloud opened this issue May 25, 2023 · 3 comments

Comments

@SevereCloud
Copy link
Contributor

Описание

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

Версия

v5.4.2

В каких браузерах воспроизводится проблема?

No response

Шаги воспроизведения

No response

Ожидаемое поведение

No response

Скриншоты

image

Пример с воспроизведением

https://vkcom.github.io/VKUI/#/IconButton

@SevereCloud SevereCloud added this to VKUI May 25, 2023
@github-project-automation github-project-automation bot moved this to 🗃 Backlog in VKUI May 25, 2023
@mendrew mendrew self-assigned this May 29, 2023
@mendrew mendrew moved this from 🗃 Backlog to 🔧 In progress in VKUI May 29, 2023
@mendrew
Copy link
Contributor

mendrew commented May 29, 2023

@SevereCloud

У нас в https://github.com/VKCOM/icons довольно много иконок особых размеров.

Кроме уже покрытой паддингами Icon16MoreVertical размер которой 8x16.

.IconButton :global(.vkuiIcon--16.vkuiIcon--w-8) {
padding: 16px 14px;
}
.IconButton--sizeY-compact :global(.vkuiIcon--16),
.IconButton--sizeY-compact :global(.vkuiIcon--16.vkuiIcon--w-8) {
padding: 14px;
}

у нас есть:

  • 16x24 Icon24Chevron
  • 24x48 Icon48SwipeUp
  • 26x24 Icon24Switch
  • совсем экзотика 29x28 Icon28Crown у которого класс vkuiIcon--29 вопреки имени.
  • и др.

Чтобы проставить отступы для всех иконок нам надо проставить отступы и для таких особых случаев.

Стоит ли для всех таких случаев добавлять стили в IconButton или это ошибки в иконках?

@mendrew
Copy link
Contributor

mendrew commented May 29, 2023

Вот полный список уникальных иконок для которых придется добавлять особые условия.

разме имя_файла название_компонента
12x8 chevron_up_12 Icon12ChevronUp
12x8 dropdown_12 Icon12Dropdown
8x12 online_mobile_12 Icon12OnlineMobile
8x12 online_vkmobile_12 Icon12OnlineVkmobile
12x16 arrows_up_down_16 Icon16ArrowsUpDown
17x16 articles_outline_16 Icon16ArticlesOutline
12x16 chevron_16 Icon16Chevron
12x16 chevron_left_16 Icon16ChevronLeft
12x16 chevron_outline_16 Icon16ChevronOutline
16x12 dropdown_16 Icon16Dropdown
16x12 dropdown_flipped_16 Icon16DropdownFlipped
16x12 dropdown_flipped_outline_16 Icon16DropdownFlippedOutline
16x12 dropdown_outline_16 Icon16DropdownOutline
8x16 more_vertical_16 Icon16MoreVertical
12x16 online_mobile_16 Icon16OnlineMobile
21x20 arrow_up_outline_20 Icon20ArrowUpOutline
20x16 chevron_up_20 Icon20ChevronUp
20x16 dropdown_20 Icon20Dropdown
16x24 chevron_24 Icon24Chevron
16x24 chevron_compact_left_24 Icon24ChevronCompactLeft
16x24 chevron_compact_right_24 Icon24ChevronCompactRight
12x24 more_vertical_24 Icon24MoreVertical
26x24 switch_24 Icon24Switch
20x28 chevron_back_28 Icon28ChevronBack
20x28 chevron_left_28 Icon28ChevronLeft
29x28 crown_28 Icon28Crown
48x24 swipe_up_48 Icon48SwipeUp

@mendrew
Copy link
Contributor

mendrew commented May 31, 2023

Ждём дизайна для IconButton чтобы было понятно какие отступы задавать в зависимости от размера иконки.
#5157

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ⛔️ Blocked
Development

No branches or pull requests

3 participants