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] Рассмотреть возможность импортировать компонент SvgIcon для оборачивания своих локальных иконок #1011

Open
inomdzhon opened this issue Oct 23, 2024 · 0 comments
Labels

Comments

@inomdzhon
Copy link
Contributor

см. #998

Проблема

Сейчас в иконках задаются CSS классы, на которые завязывается @vkontakte/vkui и местами компенсирует отсутпы. Если пользователь захочет использовать свою локальную иконку, то ему придётся прописывать эти классы у себя, чтобы в @vkontakte/vkui его иконка также нормально вставала.

Решение

Для упрощения жизни пользователя нужно рассмотреть возможность импортировать SvgIcon.

Примечание

Нужно переделать вот этик строки:

const child = React.Children.toArray(children)[0];
const hasIconChildren =
React.isValidElement(child) && typeof child.type === 'function' && 'mountIcon' in child.type;

Например, перенести в makeIcon().

@inomdzhon inomdzhon added this to VKUI Oct 23, 2024
@github-project-automation github-project-automation bot moved this to 🗃 Backlog in VKUI Oct 23, 2024
@inomdzhon inomdzhon changed the title [Feature] Рассмотреть возможность импортировать компонент SvgIcon для возможности оборачивать свои иконки [Feature] Рассмотреть возможность импортировать компонент SvgIcon для оборачивания своих локальных иконок Oct 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: 🗃 Backlog
Development

No branches or pull requests

1 participant