YarnSpinnerTool / YarnSpinner-Unity

The official Unity integration for Yarn Spinner, the friendly dialogue tool.
MIT License
522 stars 91 forks source link

v3, minor suggestion: collision between Unity.Yarn and OdinInspector attributes #279

Open dario-zubovic opened 6 months ago

dario-zubovic commented 6 months ago

What is the current behavior? Both Yarn.Unity and Sirenix.OdinInspector namespaces define attributes named ShowIf and HideIf, causing annoying conflicts in scripts that are using both. Odin Inspector is a hugely popular Unity plugin that's used in a lot of projects, so it might make sense to avoid this collision.

Suggestion Move classes in Attributes.cs to Yarn.Unity.Attributes or Yarn.UnityYarn.Utils namespace.

Environment: