Add support for flutter shaders#596
Conversation
0f49f09 to
1a866f2
Compare
1a866f2 to
472c384
Compare
AlexV525
left a comment
There was a problem hiding this comment.
Just a quick look on my phone without reviewing generators implementation.
- reverted manual version bump - shaders placed after fonts
Fixed @AlexV525 |
|
Just had a complete review of the request.
Indeed. Shaders are mostly a sub-implementation to the assets so far. We may have some abstractions so assets and shaders can use both. However, we can also use the assets generator for shaders by tweaking some parameters. These APIs are just internally used so that we can refactor them anytime. |
What does this change?
Flutter has had shader support for quite some time now. However, this package does not implement working with them. This PR aims to solve this problem.
I had to make this addition quickly, perhaps it is too rough, but still there was an attempt to do the maximum. I apologize for not creating an issue.
I added support even for what Flutter itself most likely has not implemented yet - flavor for shaders, but nevertheless. According to the settings, I implemented an almost complete copy of regular assets. It would be possible to simply pass shaders to the asset generator, but I decided to make a copy - perhaps this is excessive.
Type of change
Please delete options that are not relevant.
Checklist:
Before submitting your PR, there are a few things you can do to make sure it goes smoothly:
melos run test)melos run formatto automatically apply formatting)