Skip to content

Comments

hocs/withClassNames#65

Draft
sadcitizen wants to merge 7 commits intomasterfrom
hocs/with-class-names
Draft

hocs/withClassNames#65
sadcitizen wants to merge 7 commits intomasterfrom
hocs/with-class-names

Conversation

@sadcitizen
Copy link
Member

No description provided.

@tolms
Copy link
Contributor

tolms commented Jan 25, 2024

Хочу обратить внимание, что мы прокидываем в компонент стили из css-файла напрямую, без какой-либо фильтрации на нужные нам классы. Мы доверяем содержимому файла стилей. Т.е. если разработчик опишет в css-файле классы, которые не используются в компоненте, то эти классы также будут прокинуты в компонент, но не будут использованы. Ранее делал метод фильтрации описанных в css-файле классов, но понял, что это не особо нужно. Почему? Потому что в любом случае все классы из css-файла попадают в итоговую сборку вне зависимости от того, использованы/прокинуты в компонент. Не хочется усложнять код...

Co-authored-by: Dmitrii Maletskii <51035551+mltsk@users.noreply.github.com>
@pixel-fixer
Copy link
Contributor

pixel-fixer commented Apr 3, 2025

Может быть готовое использовать вместо mergeClassNames? Есть https://www.npmjs.com/package/clsx, который быстрый и мало весит

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants