ActionToolbar: disabled action groups should be painted disabled
authorKirill Kalishev <kirill.kalishev@jetbrains.com>
Mon, 26 Apr 2010 08:42:47 +0000 (12:42 +0400)
committerKirill Kalishev <kirill.kalishev@jetbrains.com>
Mon, 26 Apr 2010 08:42:47 +0000 (12:42 +0400)
platform/platform-impl/src/com/intellij/openapi/actionSystem/impl/Utils.java

index 2d918426be4d3058777bb346f17198589904b633..6dd22e93b87c26149f9a8dc0b457b067631f866c 100644 (file)
@@ -113,9 +113,11 @@ public class Utils{
       if (child instanceof ActionGroup) {
         ActionGroup actionGroup = (ActionGroup)child;
         if (actionGroup.isPopup()) { // popup menu has its own presentation
-          // disable group if it contains no visible actions
-          final boolean enabled = actionGroup.canBePerformed(context) || hasVisibleChildren(actionGroup, presentationFactory, context, place);
-          presentation.setEnabled(enabled);
+          if (presentation.isEnabled()) {
+            // disable group if it contains no visible actions
+            final boolean enabled = actionGroup.canBePerformed(context) || hasVisibleChildren(actionGroup, presentationFactory, context, place);
+            presentation.setEnabled(enabled);
+          }
           list.add(child);
         }
         else {