Author: [EMAIL PROTECTED]
Date: Thu Oct  9 11:45:15 2008
New Revision: 3730

Modified:
    changes/jat/oophm-branch/dev/core/src/com/google/gwt/dev/ModulePanel.java
     
changes/jat/oophm-branch/dev/core/src/com/google/gwt/dev/util/log/SwingLoggerPanel.java

Log:
Changes from review.

Review by: bobv


Modified:  
changes/jat/oophm-branch/dev/core/src/com/google/gwt/dev/ModulePanel.java
==============================================================================
---  
changes/jat/oophm-branch/dev/core/src/com/google/gwt/dev/ModulePanel.java       
 
(original)
+++  
changes/jat/oophm-branch/dev/core/src/com/google/gwt/dev/ModulePanel.java       
 
Thu Oct  9 11:45:15 2008
@@ -84,7 +84,8 @@
      this.tabs = tabs;
      JPanel topPanel = new JPanel();
      topPanel.add(new JLabel(moduleName));
-    JButton compileButton = new JButton("Compile");
+    JButton compileButton = new JButton("Compile (not yet implemented)");
+    compileButton.setEnabled(false);
      compileButton.addActionListener(new ActionListener() {
        public void actionPerformed(ActionEvent e) {
          JOptionPane.showMessageDialog(null, "Compiling not implemented  
yet",

Modified:  
changes/jat/oophm-branch/dev/core/src/com/google/gwt/dev/util/log/SwingLoggerPanel.java
==============================================================================
---  
changes/jat/oophm-branch/dev/core/src/com/google/gwt/dev/util/log/SwingLoggerPanel.java
  
(original)
+++  
changes/jat/oophm-branch/dev/core/src/com/google/gwt/dev/util/log/SwingLoggerPanel.java
  
Thu Oct  9 11:45:15 2008
@@ -203,8 +203,6 @@
            hasFocus);
        DefaultMutableTreeNode node = (DefaultMutableTreeNode) value;
        Object userObject = node.getUserObject();
-      // TODO(jat): doesn't seem to work once disconnected
-      setBackground(tree.getBackground());
        if (userObject instanceof LogEvent) {
          LogEvent event = (LogEvent) userObject;
          event.setDisplayProperties(this);
@@ -383,8 +381,6 @@
    }

    public void disconnected() {
-    // TODO(jat): better way to do this, as this doesn't change all the
-    // background colors.
      tree.setBackground(DISCONNECTED_COLOR);
      tree.repaint();
    }

--~--~---------~--~----~------------~-------~--~----~
http://groups.google.com/group/Google-Web-Toolkit-Contributors
-~----------~----~----~----~------~----~------~--~---

Reply via email to