johnsoncodehk/unity-ui-controller

Toggle function

Closed this issue · 5 comments

I often add this method to your UIController class to toggle between show and hide. Not sure if it fit's to the hole concept of the asset, but for simple show & hide animations it works very well for me. I would love to see it in the project, so I don't need to add it each time I import your asset and also I could imagine a lot of other users would use it as well.
public virtual void Toggle() { if(this.isShow){ Hide(); } else { Show(); } }

Do you know you can inherit to expand your own function, and you still want to include it in UIController.cs?

public class MyUIController : UIController
{
  public virtual void Toggle() { if(this.isShow){ Hide(); } else { Show(); } }
}

For code design, this is a redundant function for UIController.cs, but yes, this is good for DX. and I am considering.

btw I don’t know this package still has user 😅

I have send a invite to you to be collaborator, actually my work leaved Unity long time, I think it is more reasonable let current users to maintain.

Okay alright, thanks for the fast response. Unfortunately I don't have much time and ideas for improvements. I could have an eye on the project and see if there are any issues coming up here in Github.

@marvpaul you can decide add the Toggle() feature or other change, and commit now. :)

btw when I free I will update to remove unity-animator-override-controller-extras depend, because this is not a good design.