Skip to content

feat(flame_3d)!: Allow for nested Component3D by using world matrix calculations#3867

Merged
wolfenrain merged 3 commits intomainfrom
feat/add-nested-compontent3d-support
Mar 21, 2026
Merged

feat(flame_3d)!: Allow for nested Component3D by using world matrix calculations#3867
wolfenrain merged 3 commits intomainfrom
feat/add-nested-compontent3d-support

Conversation

@wolfenrain
Copy link
Copy Markdown
Contributor

Description

Added a world matrix to use across the tree for proper nested 3D components.

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

@wolfenrain wolfenrain force-pushed the feat/add-nested-compontent3d-support branch from bbf74e5 to 6bcd625 Compare March 20, 2026 09:06
@wolfenrain wolfenrain force-pushed the feat/add-nested-compontent3d-support branch from 6bcd625 to 2544273 Compare March 20, 2026 09:09
Copy link
Copy Markdown
Member

@luanpotter luanpotter left a comment

Choose a reason for hiding this comment

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

LGTM - I assume this works with animations?
also just to be clear, for any moving component, the matrix will be dirty every single tick right? the cache is just to optmize for non-moving things?

@wolfenrain
Copy link
Copy Markdown
Contributor Author

LGTM - I assume this works with animations? also just to be clear, for any moving component, the matrix will be dirty every single tick right? the cache is just to optmize for non-moving things?

Yes sir, every time our matrix gets dirty we re-calc and cache it's own and propagate that to it's children. We can probably optimize this further later on but for now it seems to work fine enough

@wolfenrain wolfenrain merged commit 48d0a84 into main Mar 21, 2026
9 checks passed
@wolfenrain wolfenrain deleted the feat/add-nested-compontent3d-support branch March 21, 2026 12:29
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