viniciusgerevini / godot-aseprite-wizard

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

Wrong Tooltip #28

Closed imsamuka closed 3 years ago

imsamuka commented 3 years ago

In the "Aseprite Wizard" window, the Do not create resource file checkbox, have the following tooltip: Removes borders from sprites/layers before saving them. which is the same as the checkbox above.

Also, in the same window, both Configuration and Close buttons are both uncapitalized.

I thought about doing a pull request solving both problems, but actually I don't know exactly the function of that checkbox. The name leads me to think it's a pointless feature, so I guess it's not so obvious what it really does.

viniciusgerevini commented 3 years ago

Thanks @imsamuka . copypasta issue.

This option was a feature request. Some people use the plugin to get just the .json and .png files. In those cases they don't need the resource file to be created. The README has the right tooltip:

Does not create SpriteFrames resource. Useful if you are only interested in the .json and .png output from Aseprite.

It is a very specific use case.

imsamuka commented 3 years ago

Messing up with the plugins got me to use this feature in the end. I'm very glad that it exists hahah.

viniciusgerevini commented 3 years ago

lol feel free to open the PR with the changes you suggested, otherwise I'll fix it over the weekend. Thanks.

imsamuka commented 3 years ago

I did the PR and with another fix. I don't know how to link this issue to the PR and if it does something, so I will close the issue now, thanks!