Alternative fix for mesh2d_manual example#15883
Alternative fix for mesh2d_manual example#15883alice-i-cecile merged 1 commit intobevyengine:mainfrom
mesh2d_manual example#15883Conversation
mesh2d_manual examplemesh2d_manual example
MiniaczQ
left a comment
There was a problem hiding this comment.
I like this more than the alternative,
can confirm the example works
tychedelia
left a comment
There was a problem hiding this comment.
I think it makes sense to demonstrate something actually custom. Thanks!
|
Why does the current code not work? AFAIK attributes haven't changed at all for a long time? |
@IceSentry It was mistake during the migration in #15862. During the process of figuring out why So the question right now is just "do we want to use a custom vertex attribute in |
IceSentry
left a comment
There was a problem hiding this comment.
Okay, thank you for the context, in that case, yeah I'm happy to use a custom attribute here.
Objective
Fixes #15847
Alternative to #15862. Would appreciate a rendering person signaling preference for one or the other.
Solution
Partially revert the changes made to this example in #15524.
Add comment explaining that the non-usage of the built-in color vertex attribute is intentional.
Testing
cargo run --example mesh2d_manual