Skip to content

refactor(flame_3d): Redone the example to be more extendable#3915

Open
wolfenrain wants to merge 1 commit into
mainfrom
Refactor(flame_3d)-redone-the-example-to-be-more-extendable
Open

refactor(flame_3d): Redone the example to be more extendable#3915
wolfenrain wants to merge 1 commit into
mainfrom
Refactor(flame_3d)-redone-the-example-to-be-more-extendable

Conversation

@wolfenrain
Copy link
Copy Markdown
Contributor

Description

Cleaned up the example to be more extendable for the future.

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

@spydon spydon changed the title Refactor(flame_3d): Redone the example to be more extendable refactor(flame_3d): Redone the example to be more extendable May 8, 2026
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@wolfenrain why remove the commands? I was envisioning it would be really awesome to be able to spawn, destroy objects, to test different things (I hadn't fully implemented many but it already had some functionality). could also have camera controls, config changes, all in a single unified interface

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Because it should be examples to showcase, not a full playground to play around :P

Our non-3D examples dont have such either, so I just wanted to make it more inline with what people expect from examples without some extra stuff

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