Skip to content

Add advanced entity outline calculations #222

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Jul 17, 2025

Conversation

okhaimie-dev
Copy link
Contributor

The changes add sophisticated AABB (Axis-Aligned Bounding Box) calculations for accurately drawing outlines around selected entities, handling various entity types like meshes and sprites, and properly accounting for hierarchies and transformations.

#221

The changes add sophisticated AABB (Axis-Aligned Bounding Box)
calculations for accurately drawing outlines around selected entities,
handling various entity types like meshes and sprites, and properly
accounting for hierarchies and transformations.
The change extracts multiple queries used in the outline gizmo system
into a dedicated `OutlineGizmoQueries` struct that implements
`SystemParam`
@jbuehler23
Copy link
Contributor

@alice-i-cecile happy to get this merged in, thought it might be better to call this a SelectionTool or SelectionBox instead of OutlineGizmo

@alice-i-cecile
Copy link
Member

I like SelectionBox. @okhaimie-dev, make that change and then I'll merge this? :)

@okhaimie-dev
Copy link
Contributor Author

I'll get back to this soon. Seems I might have to change the file name and remove some implementations from the outline_gizmos selection_box file.

The changes are focused on renaming the outline gizmo functionality to
selection box, which includes renaming the module, component, functions
and UI text for better clarity and consistency.
@tim-blackbird tim-blackbird added this pull request to the merge queue Jul 17, 2025
Merged via the queue into bevyengine:main with commit 2aeb9f0 Jul 17, 2025
6 checks passed
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.

4 participants