Skip to content

fix: [flame_texturepacker] Refactor asset path resolution and loading in TexturePackerAtlas to c…#3845

Merged
spydon merged 1 commit intoflame-engine:mainfrom
s1r1m1r1:main
Mar 6, 2026
Merged

fix: [flame_texturepacker] Refactor asset path resolution and loading in TexturePackerAtlas to c…#3845
spydon merged 1 commit intoflame-engine:mainfrom
s1r1m1r1:main

Conversation

@s1r1m1r1
Copy link
Contributor

@s1r1m1r1 s1r1m1r1 commented Mar 6, 2026

…orrectly handle prefixes and different asset types. [#3844]

Description

Replace this text.

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

@s1r1m1r1 s1r1m1r1 force-pushed the main branch 2 times, most recently from 002bcd2 to 87daa6e Compare March 6, 2026 15:14
@s1r1m1r1 s1r1m1r1 changed the title Refactor asset path resolution and loading in TexturePackerAtlas to c… [flame_texturepacker]Refactor asset path resolution and loading in TexturePackerAtlas to c… Mar 6, 2026
@s1r1m1r1 s1r1m1r1 changed the title [flame_texturepacker]Refactor asset path resolution and loading in TexturePackerAtlas to c… fix: [flame_texturepacker]Refactor asset path resolution and loading in TexturePackerAtlas to c… Mar 6, 2026
@s1r1m1r1 s1r1m1r1 changed the title fix: [flame_texturepacker]Refactor asset path resolution and loading in TexturePackerAtlas to c… fix: [flame_texturepacker] Refactor asset path resolution and loading in TexturePackerAtlas to c… Mar 6, 2026
Copy link
Member

@spydon spydon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm, but can you add some tests verifying the behavior?

Copy link
Member

@spydon spydon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm!

@spydon spydon merged commit 17fac08 into flame-engine:main Mar 6, 2026
9 checks passed
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.

2 participants