ejlersen/module-inspector
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Activity

Added obsolete text to tooltip (only works, if another property attribute is present)

Browse source
This commit is contained in:
Anders Ejlersen 2021-12-04 14:25:57 +01:00
parent 492b882a51
commit e4eb4dab47
2 changed files with 15 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
Editor/AbstractPropertyDrawer.cs
View file

@ -24,7 +24,6 @@ namespace Module.Inspector.Editor
bool prevEnabled = GUI.enabled; bool prevEnabled = GUI.enabled;
GUI.enabled = accessType == EAccessType.Enabled; GUI.enabled = accessType == EAccessType.Enabled;
// TODO: What to do with [Obsolete]?
if (result.draw != null) if (result.draw != null)
{ {

18
Editor/Utilities/EditorPropertyUtility.cs
View file

@ -126,12 +126,22 @@ namespace Module.Inspector.Editor.Utilities
} }
} }
if (!string.IsNullOrEmpty(obsoleteText)) if (isObsolete)
{ {
if (tooltip != null) if (string.IsNullOrEmpty(obsoleteText))
tooltip += $"\n[Obsolete: {obsoleteText}"; {
if (tooltip != null)
tooltip += $"\n[Obsolete]";
else
tooltip = $"[Obsolete]";
}
else else
tooltip = $"Obsolete: {obsoleteText}"; {
if (tooltip != null)
tooltip += $"\n[Obsolete: {obsoleteText}]";
else
tooltip = $"[Obsolete: {obsoleteText}]";
}
} }
return new Result(drawer, predrawerModifiers, valueModifiers, accessModifiers, tooltip, isObsolete); return new Result(drawer, predrawerModifiers, valueModifiers, accessModifiers, tooltip, isObsolete);