talecrafter / AnimationImporter

Aseprite Animation Importer for Unity
613 stars 72 forks source link

Text fields for custom paths to Components #43

Closed johnW-ret closed 3 years ago

johnW-ret commented 4 years ago

It's currently assumed that your SpriteRenderer and/or Image component is on the same GameObject as your Animator component. This pull request adds two text fields which allow you to specify a path from the parent GameObject with the Animator component, to the child GameObject(s) with the animated components. If the text fields are left blank, the importer continues as normal and assumes all components are on the same GameObject. If we are updating a previous import, we don't use the path from the editor window but restore the path from the previous animation.

GieziJo commented 3 years ago

Nice work! I have one question though, is there a point of having two fields for that? Since Target Object will be either a sprite or image, one field should be enough, right?

I think we can either only show the used field depending on the Target Object or merge both fields. I am going for the first option as an easy fix.

Edit: Obviously there is the option to have both, my bad. I thus went for the second option and only display the appropriate fields