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.

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

NavRegion2d merge indicators are only generated when the scene is loaded #85242

Closed
sandygk opened this issue Nov 22, 2023 · 0 comments · Fixed by #85258
Closed

NavRegion2d merge indicators are only generated when the scene is loaded #85242

sandygk opened this issue Nov 22, 2023 · 0 comments · Fixed by #85258

Comments

@sandygk
Copy link
Contributor

sandygk commented Nov 22, 2023

Godot version

4.2 beta 3

System information

Godot v4.2.beta3 - Windows 10.0.22631 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3060 Laptop GPU (NVIDIA; 31.0.15.3699) - AMD Ryzen 9 6900HS with Radeon Graphics (16 Threads)

Issue description

The connection indicators between two NavRegion2D (the pink circles) are not redrawn when they are moved around, instead each nav region move its indicators with it. When the scene is reloaded the indicators are drawn correctly, but then if you move the regions you have to reload the scene to get the correct indicators.

Here is a gif to illustrate the issue.

GIF 11-22-2023 3-45-46 PM

Steps to reproduce

This happens with any 2d scene that have 2 or more connected nav regions 2d.

But to simplify things, you can go to the attached project, open the Scene3 and move one of the nav regions in the editor to notice the issue

Minimal reproduction project

nav-issues.zip

@akien-mga akien-mga added this to the 4.2 milestone Nov 23, 2023
@YuriSizov YuriSizov modified the milestones: 4.2, 4.3 Nov 26, 2023
@AThousandShips AThousandShips changed the title NavRegion2d merge indicators are only generetad when the scene is loaded NavRegion2d merge indicators are only generated when the scene is loaded Nov 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants