-
-
Notifications
You must be signed in to change notification settings - Fork 4.3k
Open
Labels
A-UIGraphical user interfaces, styles, layouts, and widgetsGraphical user interfaces, styles, layouts, and widgetsC-BugAn unexpected or incorrect behaviorAn unexpected or incorrect behaviorD-ModestA "normal" level of difficulty; suitable for simple features or challenging fixesA "normal" level of difficulty; suitable for simple features or challenging fixesS-Needs-DesignThis issue requires design work to think about how it would best be accomplishedThis issue requires design work to think about how it would best be accomplished
Milestone
Description
Bevy version and features
0.18-dev, post #22124
Problem
#22124 FYI this PR means that the docs stating "respects manual edges (they always take precedence)" is stale and this block of code in the rebuilding graph logic
is meaningless since the entire graph is cleared IIUC.bevy/crates/bevy_input_focus/src/directional_navigation.rs
Lines 670 to 677 in 3ab5c2c
// Skip if manual edge already exists (check inline to avoid borrow issues) if nav_map .get_neighbors(origin.entity) .and_then(|neighbors| neighbors.get(octant)) .is_some() { continue; // Respect manual override }
From Discord by ZymartuGames.
Proposed solution
We can probably mark manual edges and do a partial rebuild? Not sure yet.
We need to avoid regressing #21949 though, so a simple revert is sub-optimal.
Metadata
Metadata
Assignees
Labels
A-UIGraphical user interfaces, styles, layouts, and widgetsGraphical user interfaces, styles, layouts, and widgetsC-BugAn unexpected or incorrect behaviorAn unexpected or incorrect behaviorD-ModestA "normal" level of difficulty; suitable for simple features or challenging fixesA "normal" level of difficulty; suitable for simple features or challenging fixesS-Needs-DesignThis issue requires design work to think about how it would best be accomplishedThis issue requires design work to think about how it would best be accomplished