-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
feat: support filters on web #2346
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we wait for #2334 with merging this one?
@WoLewicki |
And it works the same in the browser? And at the same time is not part of the standard? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added one comment. Please answer it and then I think we are good to go 🚀
Summary
Filter
andFeColorMatrix
components onweb
FilterImage
*FilterImage
example on web, since it's crashing the whole site (see feat: add new resolve web asset function #2334)* ID on web has to be unique, otherwise it'll use the first element with that ID, even if they are in the separate SVG elements
Test Plan
run
web-example
appCompatibility