viniciusgerevini / godot-aseprite-wizard

Godot Editor plugin to help import Aseprite animations to AnimationPlayers, AnimatedSprites and SpriteFrames.
MIT License
879 stars 42 forks source link

improvement suggestion: print on start and on import about aseprite not found on path #153

Closed Gnumaru closed 3 months ago

Gnumaru commented 4 months ago

I have just reinstalled my OS and forgot to put the aseprite directory in the executable path. When I noticed that my imports were not working I just remembered that I forgot to put the aseprite directory in the executable path.

Some people may have a similar problem, or maybe some people may just install the extension for the first time without reading the readme and later complaing about the extension not working. So it would be usefull if, uppon extension load and uppon each import, if the aseprite executable cannot be find in the executable path, then a warning (or even an error) is printed stating something like "the aseprite executable could not be found in the executable path. Nothing will be imported."

viniciusgerevini commented 4 months ago

Thank you. Will do. It sound like a good improvement. Cheers

viniciusgerevini commented 4 months ago

Hello. I just confirmed a warning is already given in these cases:

Screenshot from 2024-06-28 09-06-23

I also implemented OS based default command selection on #154 , which might make it more likely the right command will be used from the start.