Skip to content

Conversation

@tychedelia
Copy link
Member

@tychedelia tychedelia commented Jan 20, 2026

Ideally this should be fully defaultable, but is problematic until we complete the 2d->3d rework where more items can be exported from bevy_material. The temp fix is just to initialize a few more fields. There are other hacky things we could do to keep the default initialization working but I'd rather just fix it correctly when we complete the refactor.

@tychedelia tychedelia requested a review from atlv24 January 20, 2026 23:03
@tychedelia tychedelia added A-Rendering Drawing game state to the screen P-Regression Functionality that used to work but no longer does. Add a test for this! D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Jan 20, 2026
@alice-i-cecile alice-i-cecile added this to the 0.19 milestone Jan 20, 2026
@alice-i-cecile alice-i-cecile added S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it and removed S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Jan 20, 2026
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Jan 20, 2026
Merged via the queue into bevyengine:main with commit bb10502 Jan 20, 2026
49 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Rendering Drawing game state to the screen D-Straightforward Simple bug fixes and API improvements, docs, test and examples P-Regression Functionality that used to work but no longer does. Add a test for this! S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants