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

The save dialog should not allow to save under the .godot and .import folders #37638

Closed
Zylann opened this issue Apr 6, 2020 · 0 comments · Fixed by #46638
Closed

The save dialog should not allow to save under the .godot and .import folders #37638

Zylann opened this issue Apr 6, 2020 · 0 comments · Fixed by #46638

Comments

@Zylann
Copy link
Contributor

Zylann commented Apr 6, 2020

Godot 3.2.1

If you try to save a scene, a resource or a script, Godot opens a save dialog which allows you to save it under .import/. However this is a bad idea, and should not be proposed IMO, at least by default.
Saving under .import causes a number of issues, such as:

  • Assets in that folder cannot be accessed from the File Explorer. If your main scene is in there, it will still load when the project opens, but if you close it you can't reopen it again.
  • Searching in files from the script editor does not include this folder, because again, it's not supposed to contain assets.
  • Quick-open also doesn't list these assets (Ctrl+Shift+O and Ctrl+Alt+O) for the same reason
  • Accidental loss of assets if you want to force reimport by deleting the folder, or version your files with a VCS while ignoring that folder (and bloat in the opposite case)
  • Maybe more
@Zylann Zylann changed the title The save dialog should not allow to save under the .import folder by default The save dialog should not allow to save under the .import folder Apr 6, 2020
@Calinou Calinou added this to the 4.0 milestone Apr 6, 2020
@akien-mga akien-mga changed the title The save dialog should not allow to save under the .import folder The save dialog should not allow to save under the .godot and .import folders Mar 26, 2022
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.

3 participants