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

Adding new points to Path2D and undoing it breaks scene history #89781

Closed
KoBeWi opened this issue Mar 22, 2024 · 3 comments · Fixed by #89874
Closed

Adding new points to Path2D and undoing it breaks scene history #89781

KoBeWi opened this issue Mar 22, 2024 · 3 comments · Fixed by #89874

Comments

@KoBeWi
Copy link
Member

KoBeWi commented Mar 22, 2024

Tested versions

Regression from #86542

System information

Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce GTX 1060 (NVIDIA; 31.0.15.4633) - Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz (8 Threads)

Issue description

When you add a new Path2D and add points to it and then undo it, the scene history breaks and you can't do anything until you reload the scene.

Steps to reproduce

  1. Create new scene
  2. Add Path2D
  3. Ctrl + Click somewhere to add points
  4. Undo
  5. The scene is unusable

Minimal reproduction project (MRP)

N/A

@naturally-intelligent
Copy link

naturally-intelligent commented Apr 9, 2024

I'm getting basically unusable Path2Ds in 4.3dev5, once I add a point to the path they go dead, ex can't add child nodes to Path2D

Existing untouched path2D nodes made in prior versions still work

@djrain
Copy link

djrain commented Aug 24, 2024

Not sure if I should make a new issue, but same bug is present in 4.3 stable if you add a point by clicking on the curve with the "select points" tool.

@Calinou
Copy link
Member

Calinou commented Aug 25, 2024

Not sure if I should make a new issue, but same bug is present in 4.3 stable if you add a point by clicking on the curve with the "select points" tool.

Please open a new issue for this, as the cause may be different.

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.

5 participants