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

View Grid setting is not remembered between sessions #95818

Closed
tetrapod00 opened this issue Aug 19, 2024 · 2 comments · Fixed by #95874
Closed

View Grid setting is not remembered between sessions #95818

tetrapod00 opened this issue Aug 19, 2024 · 2 comments · Fixed by #95874

Comments

@tetrapod00
Copy link
Contributor

tetrapod00 commented Aug 19, 2024

Tested versions

  • Reproducible in 4.3stable (self-contained); 4.3.stable.mono (normal)
  • 4.2 does not have View Grid in this menu

System information

Godot v4.3.stable (using self-contained mode) - Windows 10.0.19045 - GLES3 (Compatibility) - NVIDIA GeForce GTX 1660 Ti with Max-Q Design (NVIDIA; 32.0.15.5612) - AMD Ryzen 7 3750H with Radeon Vega Mobile Gfx (8 Threads)

Issue description

The checkbox settings to see views in the 3d workspace are all remembered between editor sessions, except for View Grid, which defaults to ON.

It looks like these settings are stored per-scene, and View Grid was added in 4.3. So the mismatched behavior is likely because storing the value was either done differently or not done at all.

Steps to reproduce

Make a new Node3D scene. Save it.

Click on Perspective:
Godot_v4 3-stable_win64_hErGzIcxgB

Set all views OFF:
Godot_v4 3-stable_win64_Zl7CPhYk5A
Save the scene and restart the editor. After editor restart. Note that View Grid is still ON:
Godot_v4 3-stable_win64_H4oYj1laPb

Set all views ON:
Godot_v4 3-stable_win64_Jhqz7RbROv
Save the scene and restart the editor. After editor restart:
Godot_v4 3-stable_win64_a9w18Iu7a8

Minimal reproduction project (MRP)

N/A

@tetrapod00
Copy link
Contributor Author

Unless this is specifically intended behavior, I'm going to try to make a PR for this.

@Calinou
Copy link
Member

Calinou commented Aug 20, 2024

For reference, View Grid was implemented in #88584.

cc @ryevdokimov

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