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

Show .ico and .icns files in FileSystem #10662

Closed
AdriaandeJongh opened this issue Sep 5, 2024 · 0 comments · Fixed by godotengine/godot#96603
Closed

Show .ico and .icns files in FileSystem #10662

AdriaandeJongh opened this issue Sep 5, 2024 · 0 comments · Fixed by godotengine/godot#96603
Milestone

Comments

@AdriaandeJongh
Copy link

AdriaandeJongh commented Sep 5, 2024

Describe the project you are working on

A tower defense game called Rift Riff and soon a sequel to my indie hit game Hidden Folks.

Describe the problem or limitation you are having in your project

The Project Settings contains fields for macOS and Windows native icons with the .ico and .icns formats respectively. However, those type of files don't show up in the FileSystem. When my artist wanted to updated the icon, he naturally thought he only had to replace the .png to update the icon everywhere. This is a discrepancy.

Describe the feature / enhancement and how it helps to overcome the problem or limitation

Show .ico and .icns files in FileSystem.

Describe how your proposal will work, with code, pseudo-code, mock-ups, and/or diagrams

if file_ext == "ico" or file_ext == "icns": show()

(j/k. I don't know how this stuff works internally.)

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

The discrepancy can just remain there I guess. You can obviously select the files in the Project Settings anyway, but that's not the point here.

Is there a reason why this should be core and not an add-on in the asset library?

What FileSystem shows and doesn't show seems to be part of core, no?

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