improve UX with Clear by enabling it only when it'd perform an action

Signed-off-by: Christian López Barrón <[email protected]>
thegrill · Nov 6, 2024 · 77ff7d2 · 77ff7d2
1 parent b8e27c0
commit 77ff7d2
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/grill/views/usdview.py b/grill/views/usdview.py
@@ -258,7 +258,11 @@ def RunCommand(self):
                 assert attribute.Clear()
 
     def IsEnabled(self):
-        return super().IsEnabled() and any(attr.HasAuthoredValue() for attr in self._attributes)
+        # Usd.Attribute.Clear operates only on specs with an existing authored default value at the current edit target
+        return super().IsEnabled() and any(
+            (spec := attr.GetStage().GetEditTarget().GetAttributeSpecForScenePath(attr.GetPath())) and spec.default
+            for attr in self._attributes
+        )
 
 
 class GrillAttributeBlockMenuItem(_GrillAttributeViewContextMenuItem):