Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Consider changing the default picking behavior of sprites to be non-pickable by default #16903

Open
coreh opened this issue Dec 19, 2024 · 0 comments
Open

Consider changing the default picking behavior of sprites to be non-pickable by default #16903

coreh opened this issue Dec 19, 2024 · 0 comments
Labels
A-Picking Pointing at and selecting objects of all sorts C-Usability A targeted quality-of-life change that makes Bevy easier to use D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Ready-For-Implementation This issue is ready for an implementation PR. Go for it!

Comments

@coreh
Copy link
Contributor

coreh commented Dec 19, 2024 •
edited
Loading

From this discord conversation:

While I think having all UI nodes pickable by default is a very good choice, making all sprites pickable and blocking lower entities by default is not ideal, especially as sprites tend to be used for misc non-interactive things (like overlays, or nested cameras for rendering at lower resolutions) more often than not. You end up having to patch lots of spawning code adding PickingBehavior to suppress that behavior.

Also, some precedent from mesh picking, which has the RayCastPickable component.

(Also, if #16886 is merged, this should probably also apply to that.)
@alice-i-cecile alice-i-cecile added S-Ready-For-Implementation This issue is ready for an implementation PR. Go for it! D-Straightforward Simple bug fixes and API improvements, docs, test and examples A-Picking Pointing at and selecting objects of all sorts C-Usability A targeted quality-of-life change that makes Bevy easier to use labels Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Picking Pointing at and selecting objects of all sorts C-Usability A targeted quality-of-life change that makes Bevy easier to use D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Ready-For-Implementation This issue is ready for an implementation PR. Go for it!
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@coreh @alice-i-cecile