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

Add Default Settings for 2D Snap Step #1348

Closed
DleanJeans opened this issue Aug 11, 2020 · 8 comments · Fixed by godotengine/godot#74682
Closed

Add Default Settings for 2D Snap Step #1348

DleanJeans opened this issue Aug 11, 2020 · 8 comments · Fixed by godotengine/godot#74682

Comments

@DleanJeans
Copy link

Describe the project you are working on: a 2D Top Down Shooter

Describe the problem or limitation you are having in your project:
I want to snap to 5 or 10 px Grid Step instead of 8 px but couldn't find the default settings for them so I have to change them every scene.

image

Describe the feature / enhancement and how it helps to overcome the problem or limitation:
Add default settings for the configurations in the picture above (especially Grid Step which is what I need)

Describe how your proposal will work, with code, pseudocode, mockups, and/or diagrams:
Add them to Editor Settings: Editors > 2d along with the color settings.

image

If this enhancement will not be used often, can it be worked around with a few lines of script?: No.

Is there a reason why this should be core and not an add-on in the asset library?: This is an editor patch.

@KoBeWi
Copy link
Member

KoBeWi commented Aug 11, 2020

What version are you using? In 3.2 when you create a new scene, it automatically keeps the snap settings from the last scene that was opened.

@DleanJeans
Copy link
Author

For some reason, I remember having to change it half a dozen times. I'll keep it closed until it happens again.

@nihiluis
Copy link

nihiluis commented Jun 20, 2022

this always gets reset. I have to do it again and again. can someone reopen please. @KoBeWi approach doesn't work. snap mode also gets reset. I'm on v3.4.2.

@Calinou
Copy link
Member

Calinou commented Jun 20, 2022

2D editor snap settings are set on a per-scene basis, not a per-project or editor-wide basis. This is why you have to set it several times in the same project.

There's probably a case for making this setting per-project instead of per-scene, but this should be discussed in a separate proposal. A per-project setting would also allow for saving this in version control, as per-scene metadata isn't saved to version control.

@KoBeWi
Copy link
Member

KoBeWi commented Jun 20, 2022

this always gets reset. I have to do it again and again.

Yeah I had this problem in a new project. Basically if you already opened some scenes, they will have the old settings stored. So whatever settings will a new scene have depends on what scene you have opened when creating a new one. So this solution isn't very reliable unfortuantely.

@PrinceMerluza
Copy link

There's probably a case for making this setting per-project instead of per-scene,

Yes, at least one. An entire genre called 2d top-down rpg.

Or at least it should be possible to set a default for new scenes and can override it for specific scenes.

@kregi123
Copy link

2D editor snap settings are set on a per-scene basis, not a per-project or editor-wide basis

This is not about making it per-project setting but adding possibility to change the default value which is currently 8xp.

It's annoying that it needs to be changed in every scene. The feature that keeps the snap settings from the last opened scene works only on new scenes if I open already existing scene, in which I never used grid before, it defaults to 8px.

There is already editor-wide default setting 8px why not allow to change it?

@Calinou
Copy link
Member

Calinou commented Jan 18, 2023

There is already editor-wide default setting 8px why not allow to change it?

A hardcoded value isn't an editor setting 🙂

That said, feel free to open a pull request for this. For comparison, we do have a default camera FOV editor setting, even though it can be overridden on a per-scene basis.

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.

7 participants