Skip to content

feat: Adding size to the sprite width and sprite animation widget#3870

Open
erickzanardo wants to merge 5 commits intomainfrom
feat/adding-size-to-sprite-widgets
Open

feat: Adding size to the sprite width and sprite animation widget#3870
erickzanardo wants to merge 5 commits intomainfrom
feat/adding-size-to-sprite-widgets

Conversation

@erickzanardo
Copy link
Member

@erickzanardo erickzanardo commented Mar 22, 2026

Description

The sprite and animation widgets by default will use the available space in their parent to render, so the way to limit that, or to render the widgets in a specific dimension the user would need to wrap the widget in a SizedBox (or any other widget that would define dimensions).

this could could be confusing and easy to forget as we already had a couple of people in our discord with trouble using the widgets.

This PR will keep the current behaviour for backwards compatibility and adds a size attribute, which should make the use of the widget more straight forward.

Checklist

  • I have followed the Contributor Guide when preparing my PR.
  • I have updated/added tests for ALL new/updated/fixed functionality.
  • I have updated/added relevant documentation in docs and added dartdoc comments with ///.
  • I have updated/added relevant examples in examples or docs.

Breaking Change?

  • Yes, this PR is a breaking change.
  • No, this PR is not a breaking change.

Related Issues

@erickzanardo erickzanardo marked this pull request as draft March 22, 2026 21:20
Co-authored-by: Erick <erickzanardoo@gmail.com>
@erickzanardo erickzanardo marked this pull request as ready for review March 23, 2026 00:08
@erickzanardo erickzanardo requested a review from a team March 23, 2026 00:08
erickzanardo and others added 2 commits March 23, 2026 13:54
Co-authored-by: Lukas Klingsbo <lukas.klingsbo@gmail.com>
Co-authored-by: Erick <erickzanardoo@gmail.com>
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.

3 participants