svn commit: r944556 - /logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogPanel.java

[email protected]
Newsgroups gmane.comp.apache.logging
Message-ID <[email protected]>
Author: sdeboy
Date: Sat May 15 04:22:18 2010
New Revision: 944556

URL: http://svn.apache.org/viewvc?rev=944556&view=rev
Log:
Fix UI glitch - in multi-line event mode, refine focus with a long line would leave the row the incorrect size when refine focus was removed..resetting the table row height to default height when refine focus changes resolves the issue

Modified:
    logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogPanel.java

Modified: logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogPanel.java
URL: http://svn.apache.org/viewvc/logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogPanel.java?rev=944556&r1=944555&r2=944556&view=diff
==============================================================================
--- logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogPanel.java (original)
+++ logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogPanel.java Sat May 15 04:22:18 2010
@@ -708,6 +708,16 @@ public class LogPanel extends DockablePa
     });
       
     tableModel.addLoggerNameListener(logTreeModel);
+    ruleMediator.addPropertyChangeListener(new PropertyChangeListener()
+    {
+        public void propertyChange(PropertyChangeEvent evt)
+        {
+            //reset row height to default - logger rule or refinement rule changed the displayed rows - when in multiline row mode,
+            //the table rows may not be the correct height (the row's height may be larger than the preferred height)..
+            //updating all rows to the default height resolves this issue
+            table.setRowHeight(ChainsawConstants.DEFAULT_ROW_HEIGHT);
+        }
+    });
 
     /**
      * Set the LoggerRule to be the LoggerTreePanel, as this visual component
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.