Skip to content

Conversation

@pablo-lua
Copy link
Contributor

@pablo-lua pablo-lua commented Feb 16, 2024

Objective

Solution

  • Creation of new field and method to the ArrowBuilder

Changelog

Added

  • New field ArrowBuilder::double_ended
  • New method ArrowBuilder::with_double_end to redefine the double_ended field

Additional

I added this in the 3d_gizmos example, that's the result:

image

I added this arrow in the 2d_gizmos example too:

image

Copy link
Contributor

@afonsolage afonsolage left a comment

Choose a reason for hiding this comment

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

The code seems fine, only small changes and I think the 3d example doesn't showcase well the double ended gizmos.

@Kanabenki Kanabenki added C-Feature A new feature, making something new possible A-Gizmos Visual editor and debug gizmos labels Feb 16, 2024
@pablo-lua
Copy link
Contributor Author

I changed the arrow position in the example, it is like the image now.

@nicopap nicopap mentioned this pull request Feb 20, 2024
57 tasks
@pablo-lua pablo-lua added the S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it label Apr 9, 2024
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Apr 16, 2024
Merged via the queue into bevyengine:main with commit 6b0e3fa Apr 16, 2024
@pablo-lua pablo-lua deleted the double-end-arrow branch April 16, 2024 04:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Gizmos Visual editor and debug gizmos C-Feature A new feature, making something new possible S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants