Skip to content

Conversation

@kfc35
Copy link
Contributor

@kfc35 kfc35 commented Dec 30, 2025

Objective

Solution

During automatic directional navigation, additionally query for the current input focus’s ComputedUiTargetCamera. Use that camera to filter for navigable nodes with the same camera.

Testing

To ensure no regressions, I checked that cargo run --example auto_directional_navigation works as intended, which it does

@kfc35 kfc35 changed the title Fix: Filter for currently focused camera target during directional navigation Fix: Filter for currently focused item’s target camera during directional navigation Dec 30, 2025
@alice-i-cecile alice-i-cecile added C-Bug An unexpected or incorrect behavior A-Input Player input via keyboard, mouse, gamepad, and more A-UI Graphical user interfaces, styles, layouts, and widgets D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Dec 30, 2025
Copy link
Contributor

@jbuehler23 jbuehler23 left a comment

Choose a reason for hiding this comment

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

looks good to me :). Thanks for picking this up!

@alice-i-cecile alice-i-cecile added S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it and removed S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Dec 30, 2025
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Dec 30, 2025
Merged via the queue into bevyengine:main with commit c533fad Dec 30, 2025
51 checks passed
@kfc35
Copy link
Contributor Author

kfc35 commented Dec 31, 2025

@alice-i-cecile Just want to make sure this gets added to the 0.18 milestone just because the linked issue it closed is also in the milestone! But if you intend not to add this to the milestone, please disregard

@alice-i-cecile alice-i-cecile added this to the 0.18 milestone Jan 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Input Player input via keyboard, mouse, gamepad, and more A-UI Graphical user interfaces, styles, layouts, and widgets C-Bug An unexpected or incorrect behavior D-Straightforward Simple bug fixes and API improvements, docs, test and examples 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.

bevy_input_focus: Auto navigation ignores camera targets

3 participants