From e105db6b823677906fc35b5c42ef04eb2d769e8a Mon Sep 17 00:00:00 2001
From: Konstantin Bulenkov <kb@jetbrains.com>
Date: Mon, 19 Sep 2011 15:56:05 +0200
Subject: [PATCH] better look when table hasn't focus

---
 .../JavaChangeSignatureDialog.java                |  7 +++++--
 .../src/com/intellij/ui/table/JBTable.java        | 15 +++++++++------
 2 files changed, 14 insertions(+), 8 deletions(-)

diff --git a/java/java-impl/src/com/intellij/refactoring/changeSignature/JavaChangeSignatureDialog.java b/java/java-impl/src/com/intellij/refactoring/changeSignature/JavaChangeSignatureDialog.java
index d272c9baa1eb..2398fbe66b1d 100644
--- a/java/java-impl/src/com/intellij/refactoring/changeSignature/JavaChangeSignatureDialog.java
+++ b/java/java-impl/src/com/intellij/refactoring/changeSignature/JavaChangeSignatureDialog.java
@@ -194,7 +194,7 @@ public class JavaChangeSignatureDialog extends ChangeSignatureDialogBase<Paramet
   }
 
   @Override
-  protected JComponent getRowPresentation(ParameterTableModelItemBase<ParameterInfoImpl> item, boolean selected, boolean focused) {
+  protected JComponent getRowPresentation(ParameterTableModelItemBase<ParameterInfoImpl> item, boolean selected, final boolean focused) {
     final JPanel panel = new JPanel(new BorderLayout());
     final String typeText = item.typeCodeFragment.getText();
     final String separator = " ";StringUtil.repeatSymbol(' ', getTypesMaxLength() - typeText.length() + 1);
@@ -219,11 +219,14 @@ public class JavaChangeSignatureDialog extends ChangeSignatureDialogBase<Paramet
     //final Font font = EditorColorsManager.getInstance().getGlobalScheme().getFont(EditorFontType.PLAIN);
     //field.setFont(font);
 
-    if (selected) {
+    if (selected && focused) {
       panel.setBackground(UIUtil.getTableSelectionBackground());
       field.setAsRendererWithSelection(UIUtil.getTableSelectionBackground(), UIUtil.getTableSelectionForeground());
     } else {
       panel.setBackground(UIUtil.getTableBackground());
+      if (selected && !focused) {
+        panel.setBorder(new DottedBorder(UIUtil.getTableForeground()));
+      }
     }
     panel.add(field, BorderLayout.CENTER);
     return panel;
diff --git a/platform/platform-api/src/com/intellij/ui/table/JBTable.java b/platform/platform-api/src/com/intellij/ui/table/JBTable.java
index 34dada0f02c8..627b42d19e59 100644
--- a/platform/platform-api/src/com/intellij/ui/table/JBTable.java
+++ b/platform/platform-api/src/com/intellij/ui/table/JBTable.java
@@ -387,14 +387,17 @@ public class JBTable extends JTable implements ComponentWithEmptyText, Component
       UIUtil.changeBackGround(this, UIUtil.getTreeTextBackground());
     }
 
-    if (!isCellSelected(row, column) && isTableDecorationSupported() && isStriped() && result instanceof JComponent) {
+    if (isTableDecorationSupported() && isStriped() && result instanceof JComponent) {
       final Color bg =  row % 2 == 1 ? getBackground() : DECORATED_ROW_BG_COLOR;
       final JComponent c = (JComponent)result;
-      c.setOpaque(true);
-      c.setBackground(bg);
-      for (Component child : c.getComponents()) {
-        child.setBackground(bg);
-      }
+      final boolean cellSelected = isCellSelected(row, column);
+      if (!cellSelected || (!hasFocus() && !getSelectionBackground().equals(c.getBackground()))) {
+        c.setOpaque(true);
+        c.setBackground(bg);
+        for (Component child : c.getComponents()) {
+          child.setBackground(bg);
+        }
+      } 
     }
 
     if (!selected) return result;