fix: Auto-resize SpriteComponent
on sprite change
#2430
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR adds an
autoResize
parameter toSpriteComponent
using which users will be able to control if the component should auto-resize on sprite change to match the sprite'ssrcSize
. Some important design decisions aboutautoResize
:It is mutually exclusive with
size
while construction: This means, ifsize
is non-null while construction ofSpriteComponent
,autoResize
cannot be true. This allows use-cases where one might want to render a sprite at a different size than its original size. If an user is setting the size explicitly, we can leave it up to them to handle resizing in such cases.It is optional: If an user does not set this parameter while construction, it will be auto-populated depending upon if a
size
is provided or not.Checklist
docs
and added dartdoc comments with///
.examples
ordocs
.Breaking Change?
Related Issues
Closes #2419