Updated Button UI Logic to block items when non interactable #256
Conversation
…pt to be more efficient
|
Shit, ik had deze eerst moeten reviewen voordat ik het verplaatsen van de Sun Assets mergde.. ik kan de merge conflicts resolven zodra ik weer achter mijn IDE zit |
|
Merge conflict resolved; ik wacht op de feature branch build |
|
Er zit iets in deze feature branch wat leidt tot errors gerelateerd aam, wat lijkt, jslib referenties? Dit moet eerst uitgezocht worden want dat betekent dat de huidige branch niet gemerged kan worden |
|
Closed, is door #284 vervangen |
| if (Icon != null) | ||
| { | ||
| Icon.color = DisabledTextColor; | ||
| } | ||
| } |
There was a problem hiding this comment.
dit stuk lijk je 2 keer te gebruiken ook in pointerup, kan je misschien gewoon een method maken zoals,
private void EnableButton()
{
bool interactable = button.interactable;
Color color = interactable ? BaseTextColor : DisabledTextColor;
if (ButtonText != null)
{
ButtonText.overrideColorTags = true;
ButtonText.color = color;
}
if (Icon != null)
{
Icon.color = color;
}
}
| if (Icon != null) | ||
| { | ||
| Icon.color = BaseTextColor; | ||
| } |
There was a problem hiding this comment.
ook hier even method maken EnableButton() en in start en pointerup uitvoeren
| button.GetComponent<UIButtonLogic>().Disable_Button(); | ||
| } | ||
| else { | ||
| button.GetComponent<UIButtonLogic>().Enable_Button(); |
There was a problem hiding this comment.
kan je UIButtonLogic class scope cachen. GetComponent is zwaar
Changes zitten in: UIButtonLogic.cs, SunInspector.prefab en SunSettingsPanel.prefab