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.

Sign up for GitHub

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

Jump to bottom

fix(SvgIcon): move default display to svg props #317

Merged
merged 1 commit into from
Dec 9, 2022
Merged

fix(SvgIcon): move default display to svg props #317

merged 1 commit into from
Dec 9, 2022

Conversation

eugpoloz
Copy link
Contributor

@eugpoloz eugpoloz commented Dec 9, 2022 •
edited by inomdzhon
Loading

Была проблема, что через CSS не перебёшь style="display: block"

@github-actions
Copy link

github-actions bot commented Dec 9, 2022

👀 Docs deployed

See the docs for this PR at https://vkcom.github.io/icons/pull/317/

@vkcom-publisher
Copy link
Contributor

size-limit report 📦

Path Size
JS 2.7 MB (0%)
JS (gzip) 712.79 KB (+0.01% 🔺)
JS (brotli) 518.61 KB (-0.01% 🔽)
JS ES6 with Icon16Add only import (tree shaking) 4.36 KB (+0.05% 🔺)
SVG 3.06 MB (0%)

@eugpoloz eugpoloz merged commit c422979 into master Dec 9, 2022
@eugpoloz eugpoloz deleted the eugpoloz/svg-display-block branch December 9, 2022 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@inomdzhon inomdzhon inomdzhon approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@eugpoloz @vkcom-publisher @inomdzhon