-
-
Notifications
You must be signed in to change notification settings - Fork 4.5k
Turn Mesh::set_indices into Mesh::insert_indices #11740
Copy link
Copy link
Closed
Labels
A-RenderingDrawing game state to the screenDrawing game state to the screenC-UsabilityA targeted quality-of-life change that makes Bevy easier to useA targeted quality-of-life change that makes Bevy easier to use
Description
What problem does this solve or what need does it fill?
We have Mesh::insert_attribute and Mesh::remove_attribute.
But for indices we have Mesh::remove_indices (added in #11733) and Mesh::set_indices which accepts Option. It's inconsistent with the attributes API.
What solution would you like?
I would suggest to make Mesh::set_indices similar to Mesh::insert_attribute, i.e. pass the value instead of Option.
Ping @solis-lumine-vorago since he may be interested in it.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
A-RenderingDrawing game state to the screenDrawing game state to the screenC-UsabilityA targeted quality-of-life change that makes Bevy easier to useA targeted quality-of-life change that makes Bevy easier to use