X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fgui%2FAlignFrame.java;h=b730c0f18517eccfbe9e9dd4b17697bf8008dc5e;hb=04651a0a92ea1b6d9ce590a77ade2b90f446d949;hp=2a323efbbcc95dc1f75b9a8f878a587c1cbb0cee;hpb=f4d412bcbf7c540cb26024e4ec5dab6443ae59e2;p=jalview.git

diff --git a/src/jalview/gui/AlignFrame.java b/src/jalview/gui/AlignFrame.java
index 2a323ef..b730c0f 100755
--- a/src/jalview/gui/AlignFrame.java
+++ b/src/jalview/gui/AlignFrame.java
@@ -13,7 +13,7 @@
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
+ * along with this program; if not, write to the Free Softwarechang
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA
  */
 package jalview.gui;
@@ -68,6 +68,17 @@ public class AlignFrame
   {
     viewport = new AlignViewport(al);
 
+    if(viewport.vconsensus==null)
+    {
+      //Out of memory caluclating consensus.
+      BLOSUM62Colour.setEnabled(false);
+      PIDColour.setEnabled(false);
+      conservationMenuItem.setEnabled(false);
+      modifyConservation.setEnabled(false);
+      abovePIDThreshold.setEnabled(false);
+      modifyPID.setEnabled(false);
+    }
+
     alignPanel = new AlignmentPanel(this, viewport);
     alignPanel.annotationPanel.adjustPanelHeight();
     alignPanel.annotationSpaceFillerHolder.setPreferredSize(alignPanel.
@@ -76,32 +87,36 @@ public class AlignFrame
         getPreferredSize());
     alignPanel.setAnnotationVisible(viewport.getShowAnnotation());
 
+    String sortby = jalview.bin.Cache.getDefault("SORT_ALIGNMENT", "No sort");
+
+    if(sortby.equals("Id"))
+      sortIDMenuItem_actionPerformed(null);
+    else if(sortby.equals("Pairwise Identity"))
+      sortPairwiseMenuItem_actionPerformed(null);
+
     getContentPane().add(alignPanel, java.awt.BorderLayout.CENTER);
 
-    addInternalFrameListener(new InternalFrameAdapter()
+    viewport.addPropertyChangeListener(new PropertyChangeListener()
     {
-      public void internalFrameActivated(InternalFrameEvent evt)
-      {
-        javax.swing.SwingUtilities.invokeLater(new Runnable()
-        {
-          public void run()
-          {
-            alignPanel.requestFocus();
-          }
-        });
-      }
-    });
+     public void propertyChange(PropertyChangeEvent evt)
+     {
+       if (evt.getPropertyName().equals("alignment"))
+       {
+         alignmentChanged();
+       }
+     }
+   });
+
+
     addServiceListeners();
   }
 
   /* Set up intrinsic listeners for dynamically generated GUI bits. */
   private void addServiceListeners()
   {
-
-    final AlignFrame thisFrame = this;
     final java.beans.PropertyChangeListener thisListener;
     // Do this once to get current state
-    thisFrame.BuildWebServiceMenu();
+    BuildWebServiceMenu();
     Desktop.discoverer.addPropertyChangeListener(
         thisListener = new java.beans.PropertyChangeListener()
     {
@@ -111,7 +126,7 @@ public class AlignFrame
         if (evt.getPropertyName().equals("services"))
         {
           // System.out.println("Rebuilding web service menu");
-          thisFrame.BuildWebServiceMenu();
+          BuildWebServiceMenu();
         }
       }
     });
@@ -123,6 +138,7 @@ public class AlignFrame
       {
         // System.out.println("deregistering discoverer listener");
         Desktop.discoverer.removePropertyChangeListener(thisListener);
+        closeMenuItem_actionPerformed(null);
       }
       ;
     });
@@ -264,7 +280,8 @@ public class AlignFrame
       {
         java.io.PrintWriter out = new java.io.PrintWriter(
             new java.io.FileWriter(file));
-        out.println(output);
+
+        out.print(output);
         out.close();
         return true;
       }
@@ -348,6 +365,7 @@ public class AlignFrame
   {
     try
     {
+      PaintRefresher.components.remove(viewport.alignment);
       this.setClosed(true);
     }
     catch (Exception ex)
@@ -409,6 +427,7 @@ public class AlignFrame
     redoList.push(new HistoryItem(hi.getDescription(), viewport.alignment,
                                   HistoryItem.HIDE));
     restoreHistoryItem(hi);
+    viewport.firePropertyChange("alignment", null, viewport.getAlignment().getSequences());
   }
 
   /**
@@ -421,9 +440,7 @@ public class AlignFrame
     HistoryItem hi = (HistoryItem) redoList.pop();
     restoreHistoryItem(hi);
     updateEditMenuBar();
-    viewport.updateConsensus();
-    alignPanel.repaint();
-    alignPanel.repaint();
+    viewport.firePropertyChange("alignment", null, viewport.getAlignment().getSequences());
   }
 
   // used by undo and redo
@@ -468,9 +485,6 @@ public class AlignFrame
 
     updateEditMenuBar();
 
-    viewport.updateConsensus();
-    viewport.updateConservation();
-    alignPanel.repaint();
     viewport.firePropertyChange("alignment", null,
                                 viewport.getAlignment().getSequences());
   }
@@ -546,15 +560,16 @@ public class AlignFrame
   {
     if (viewport.getSelectionGroup() == null)
     {
+      System.out.println("null here");
       return;
     }
 
     SequenceGroup sg = viewport.getSelectionGroup();
 
     Clipboard c = Toolkit.getDefaultToolkit().getSystemClipboard();
-    StringBuffer buffer = new StringBuffer();
 
     Hashtable orderedSeqs = new Hashtable();
+    SequenceI [] seqs = new SequenceI[sg.getSize()];
 
     for (int i = 0; i < sg.getSize(); i++)
     {
@@ -584,6 +599,8 @@ public class AlignFrame
           index++;
         }
       }
+
+
       //FIND START RES
       //Returns residue following index if gap
       startRes = seq.findPosition(sg.getStartRes());
@@ -606,14 +623,13 @@ public class AlignFrame
         endRes += seq.getStart() - 1;
       }
 
-      buffer.append(seq.getName() + "\t" +
-                    startRes + "\t" +
-                    endRes + "\t" +
-                    seq.getSequence(sg.getStartRes(),
-                                    sg.getEndRes() + 1) + "\n");
+      seqs[i] = new Sequence(seq.getName(),
+                             seq.getSequence(sg.getStartRes(), sg.getEndRes()+1),
+                             startRes,
+                             endRes);
     }
 
-    c.setContents(new StringSelection(buffer.toString()), null);
+    c.setContents(new StringSelection(FastaFile.print(seqs)), null);
   }
 
   /**
@@ -656,33 +672,14 @@ public class AlignFrame
       }
 
       String str = (String) contents.getTransferData(DataFlavor.stringFlavor);
-      StringTokenizer st = new StringTokenizer(str);
-      ArrayList seqs = new ArrayList();
 
-      while (st.hasMoreElements())
-      {
-        String name = st.nextToken();
-        int start = Integer.parseInt(st.nextToken());
-        int end = Integer.parseInt(st.nextToken());
-        Sequence sequence = new Sequence(name, st.nextToken(), start,
-                                         end);
+      String format = IdentifyFile.Identify(str, "Paste");
+      SequenceI[] sequences = FormatAdapter.readFile(str, "Paste", format);
 
-        if (!newAlignment)
-        {
-          viewport.alignment.addSequence(sequence);
-        }
-        else
-        {
-          seqs.add(sequence);
-        }
-      }
 
       if (newAlignment)
       {
-        SequenceI[] newSeqs = new SequenceI[seqs.size()];
-        seqs.toArray(newSeqs);
-
-        AlignFrame af = new AlignFrame(new Alignment(newSeqs));
+        AlignFrame af = new AlignFrame(new Alignment(sequences));
         String newtitle = new String("Copied sequences");
 
         if (title.startsWith("Copied sequences"))
@@ -699,20 +696,21 @@ public class AlignFrame
       }
       else
       {
-        viewport.firePropertyChange("alignment", null,
-                                    viewport.getAlignment().getSequences());
+        //!newAlignment
+        for (int i = 0; i < sequences.length; i++)
+          viewport.alignment.addSequence(sequences[i]);
+
         viewport.setEndSeq(viewport.alignment.getHeight());
         viewport.alignment.getWidth();
-        viewport.updateConservation();
-        viewport.updateConsensus();
-        alignPanel.repaint();
+        viewport.firePropertyChange("alignment", null, viewport.getAlignment().getSequences());
       }
     }
     catch (Exception ex)
     {
+        // could be anything being pasted in here
     }
 
-    // could be anything being pasted in here
+
   }
 
   /**
@@ -733,7 +731,6 @@ public class AlignFrame
    */
   protected void delete_actionPerformed(ActionEvent e)
   {
-    boolean seqsdeleted = false;
 
     if (viewport.getSelectionGroup() == null)
     {
@@ -765,7 +762,6 @@ public class AlignFrame
 
       if (seq.getSequence().length() < 1)
       {
-        seqsdeleted = true;
         viewport.getAlignment().deleteSequence(seq);
       }
       else
@@ -777,13 +773,10 @@ public class AlignFrame
     viewport.setSelectionGroup(null);
     viewport.alignment.deleteGroup(sg);
 
-    if (seqsdeleted)
-    {
-      viewport.firePropertyChange("alignment", null,
+    viewport.firePropertyChange("alignment", null,
                                   viewport.getAlignment().getSequences());
-    }
 
-    viewport.resetSeqLimits(alignPanel.seqPanel.seqCanvas.getHeight());
+
 
     if (viewport.getAlignment().getHeight() < 1)
     {
@@ -795,10 +788,6 @@ public class AlignFrame
       {
       }
     }
-
-    viewport.updateConservation();
-    viewport.updateConsensus();
-    alignPanel.repaint();
   }
 
   /**
@@ -907,7 +896,7 @@ public class AlignFrame
         }
       }
 
-      alignPanel.repaint();
+      viewport.firePropertyChange("alignment", null, viewport.getAlignment().getSequences());
     }
   }
 
@@ -945,7 +934,7 @@ public class AlignFrame
         }
       }
 
-      alignPanel.repaint();
+      viewport.firePropertyChange("alignment", null, viewport.getAlignment().getSequences());
     }
   }
 
@@ -959,10 +948,16 @@ public class AlignFrame
     addHistoryItem(new HistoryItem("Remove Gapped Columns",
                                    viewport.alignment, HistoryItem.HIDE));
 
+    //This is to maintain viewport position on first residue
+    //of first sequence
+    SequenceI seq = viewport.alignment.getSequenceAt(0);
+    int startRes = seq.findPosition(viewport.startRes);
+
     viewport.getAlignment().removeGaps();
-    viewport.updateConservation();
-    viewport.updateConsensus();
-    alignPanel.repaint();
+
+    viewport.setStartRes(seq.findIndex(startRes)-1);
+
+   viewport.firePropertyChange("alignment", null, viewport.getAlignment().getSequences());
   }
 
   /**
@@ -975,6 +970,12 @@ public class AlignFrame
     addHistoryItem(new HistoryItem("Remove Gaps", viewport.alignment,
                                    HistoryItem.HIDE));
 
+    //This is to maintain viewport position on first residue
+    //of first sequence
+    SequenceI seq = viewport.alignment.getSequenceAt(0);
+    int startRes = seq.findPosition(viewport.startRes);
+
+
     SequenceI current;
     int jSize;
 
@@ -989,7 +990,7 @@ public class AlignFrame
     {
       seqs = viewport.getSelectionGroup().sequences;
       start = viewport.getSelectionGroup().getStartRes();
-      end = viewport.getSelectionGroup().getEndRes();
+      end = viewport.getSelectionGroup().getEndRes()+1;
     }
     else
     {
@@ -1001,24 +1002,103 @@ public class AlignFrame
       current = (SequenceI) seqs.elementAt(i);
       jSize = current.getLength();
 
+      // Removing a range is much quicker than removing gaps
+      // one by one for long sequences
       int j = start;
+      int rangeStart=-1, rangeEnd=-1;
 
       do
       {
         if (jalview.util.Comparison.isGap(current.getCharAt(j)))
         {
-          current.deleteCharAt(j);
-          j--;
-          jSize--;
+          if(rangeStart==-1)
+           {
+             rangeStart = j;
+             rangeEnd = j+1;
+           }
+           else
+           {
+             rangeEnd++;
+           }
+           j++;
+        }
+        else
+        {
+          if(rangeStart>-1)
+          {
+            current.deleteChars(rangeStart, rangeEnd);
+            j-=rangeEnd-rangeStart;
+            jSize-=rangeEnd-rangeStart;
+            rangeStart = -1;
+            rangeEnd = -1;
+          }
+          else
+            j++;
         }
-        j++;
       }
       while (j < end && j < jSize);
+      if(rangeStart>-1)
+      {
+       current.deleteChars(rangeStart, rangeEnd);
+      }
     }
 
-    viewport.updateConservation();
-    viewport.updateConsensus();
-    alignPanel.repaint();
+    viewport.setStartRes(seq.findIndex(startRes)-1);
+
+    viewport.firePropertyChange("alignment", null, viewport.getAlignment().getSequences());
+  }
+
+ public void alignmentChanged()
+ {
+   if(viewport.vconsensus!=null)
+   {
+     viewport.updateConsensus();
+     viewport.updateConservation();
+   }
+   resetAllColourSchemes();
+   if(alignPanel.overviewPanel!=null)
+     alignPanel.overviewPanel.updateOverviewImage();
+
+   alignPanel.repaint();
+ }
+
+  void resetAllColourSchemes()
+  {
+    ColourSchemeI cs = viewport.globalColourScheme;
+    if(cs!=null)
+    {
+      if (cs instanceof ClustalxColourScheme)
+      {
+        ( (ClustalxColourScheme) viewport.getGlobalColourScheme()).
+            resetClustalX(viewport.alignment.getSequences(),
+                          viewport.alignment.getWidth());
+      }
+
+      cs.setConsensus(viewport.vconsensus);
+      if (cs.conservationApplied())
+      {
+        Alignment al = (Alignment) viewport.alignment;
+        Conservation c = new Conservation("All",
+                                          ResidueProperties.propHash, 3,
+                                          al.getSequences(), 0,
+                                          al.getWidth() - 1);
+        c.calculate();
+        c.verdict(false, viewport.ConsPercGaps);
+
+        cs.setConservation(c);
+      }
+    }
+
+    int s, sSize = viewport.alignment.getGroups().size();
+    for(s=0; s<sSize; s++)
+    {
+      SequenceGroup sg = (SequenceGroup)viewport.alignment.getGroups().elementAt(s);
+      if(sg.cs!=null && sg.cs instanceof ClustalxColourScheme)
+      {
+        ((ClustalxColourScheme)sg.cs).resetClustalX(sg.sequences, sg.getWidth());
+      }
+      sg.recalcConservation();
+    }
   }
 
   /**
@@ -1045,9 +1125,7 @@ public class AlignFrame
       }
     }
 
-    viewport.updateConservation();
-    viewport.updateConsensus();
-    alignPanel.repaint();
+    alignmentChanged();
   }
 
   /**
@@ -1371,65 +1449,50 @@ public class AlignFrame
   {
     int threshold = 0;
 
-    if (viewport.getAbovePIDThreshold())
+    if(cs!=null)
     {
-      threshold = SliderPanel.setPIDSliderSource(alignPanel, cs,
-                                                 "Background");
-
-      if (cs instanceof ResidueColourScheme)
+      if (viewport.getAbovePIDThreshold())
       {
-        ( (ResidueColourScheme) cs).setThreshold(threshold,
-                                                 viewport.
-                                                 getIgnoreGapsConsensus());
+        threshold = SliderPanel.setPIDSliderSource(alignPanel, cs,
+                                                   "Background");
+
+        cs.setThreshold(threshold,
+                        viewport.getIgnoreGapsConsensus());
+
+        viewport.setGlobalColourScheme(cs);
       }
-      else if (cs instanceof ScoreColourScheme)
+      else
       {
-        ( (ScoreColourScheme) cs).setThreshold(threshold,
-                                               viewport.getIgnoreGapsConsensus());
+        cs.setThreshold(0, viewport.getIgnoreGapsConsensus());
       }
 
-      viewport.setGlobalColourScheme(cs);
-    }
-    else if (cs instanceof ResidueColourScheme)
-    {
-      ( (ResidueColourScheme) cs).setThreshold(0,
-                                               viewport.getIgnoreGapsConsensus());
-    }
-
-
-    if (viewport.getConservationSelected())
-    {
-      ConservationColourScheme ccs = null;
-
-      Alignment al = (Alignment) viewport.alignment;
-      Conservation c = new Conservation("All",
-                                        ResidueProperties.propHash, 3,
-                                        al.getSequences(), 0,
-                                        al.getWidth() - 1);
-
-      c.calculate();
-      c.verdict(false, viewport.ConsPercGaps);
+      if (viewport.getConservationSelected())
+      {
 
-      ccs = new ConservationColourScheme(c, cs);
+        Alignment al = (Alignment) viewport.alignment;
+        Conservation c = new Conservation("All",
+                                          ResidueProperties.propHash, 3,
+                                          al.getSequences(), 0,
+                                          al.getWidth() - 1);
 
-      ccs.setConsensus(viewport.vconsensus);
+        c.calculate();
+        c.verdict(false, viewport.ConsPercGaps);
 
-      viewport.setGlobalColourScheme(ccs);
+        cs.setConservation(c);
 
-      ccs.inc = SliderPanel.setConservationSlider(alignPanel, ccs,
-                                                  "Background");
-    }
-    else
-    {
-      // MUST NOTIFY THE COLOURSCHEME OF CONSENSUS!
-      if (cs != null)
+        cs.setConservationInc(SliderPanel.setConservationSlider(alignPanel, cs,
+            "Background"));
+      }
+      else
       {
-        cs.setConsensus(viewport.vconsensus);
+        cs.setConservation(null);
       }
 
-      viewport.setGlobalColourScheme(cs);
+      cs.setConsensus(viewport.vconsensus);
     }
 
+    viewport.setGlobalColourScheme(cs);
+
     if (viewport.getColourAppliesToAllGroups())
     {
       Vector groups = viewport.alignment.getGroups();
@@ -1441,8 +1504,10 @@ public class AlignFrame
         if (cs == null)
         {
           sg.cs = null;
+          continue;
         }
-        else if (cs instanceof ClustalxColourScheme)
+
+        if (cs instanceof ClustalxColourScheme)
         {
           sg.cs = new ClustalxColourScheme(sg.sequences, sg.getWidth());
         }
@@ -1461,17 +1526,19 @@ public class AlignFrame
           }
         }
 
-        if (viewport.getAbovePIDThreshold())
+        if (viewport.getAbovePIDThreshold()
+            || cs instanceof PIDColourScheme
+            || cs instanceof Blosum62ColourScheme)
         {
-          if (sg.cs instanceof ResidueColourScheme)
-          {
-            ( (ResidueColourScheme) sg.cs).setThreshold(threshold,
+         sg.cs.setThreshold(threshold,
                 viewport.getIgnoreGapsConsensus());
-          }
 
           sg.cs.setConsensus(AAFrequency.calculate(sg.sequences, 0,
               sg.getWidth()));
         }
+        else
+          sg.cs.setThreshold(0, viewport.getIgnoreGapsConsensus());
+
 
         if (viewport.getConservationSelected())
         {
@@ -1481,21 +1548,10 @@ public class AlignFrame
                                             viewport.alignment.getWidth() - 1);
           c.calculate();
           c.verdict(false, viewport.ConsPercGaps);
-
-          ConservationColourScheme ccs = new ConservationColourScheme(c,
-              sg.cs);
-
-          // MUST NOTIFY THE COLOURSCHEME OF CONSENSUS!
-          ccs.setConsensus(AAFrequency.calculate(sg.sequences, 0,
-                                                 sg.getWidth()));
-          sg.cs = ccs;
-        }
-        else if (cs != null)
-        {
-          // MUST NOTIFY THE COLOURSCHEME OF CONSENSUS!
-          sg.cs.setConsensus(AAFrequency.calculate(sg.sequences, 0,
-              sg.getWidth()));
+          sg.cs.setConservation(c);
         }
+        else
+          sg.cs.setConservation(null);
       }
     }
 
@@ -1514,7 +1570,7 @@ public class AlignFrame
    */
   protected void modifyPID_actionPerformed(ActionEvent e)
   {
-    if (viewport.getAbovePIDThreshold())
+    if (viewport.getAbovePIDThreshold() && viewport.globalColourScheme!=null)
     {
       SliderPanel.setPIDSliderSource(alignPanel,
                                      viewport.getGlobalColourScheme(),
@@ -1530,7 +1586,7 @@ public class AlignFrame
    */
   protected void modifyConservation_actionPerformed(ActionEvent e)
   {
-    if (viewport.getConservationSelected())
+    if (viewport.getConservationSelected() && viewport.globalColourScheme!=null)
     {
       SliderPanel.setConservationSlider(alignPanel,
                                         viewport.globalColourScheme,
@@ -1551,16 +1607,7 @@ public class AlignFrame
     viewport.setAbovePIDThreshold(false);
     abovePIDThreshold.setSelected(false);
 
-    ColourSchemeI cs = viewport.getGlobalColourScheme();
-
-    if (cs instanceof ConservationColourScheme)
-    {
-      changeColour( ( (ConservationColourScheme) cs).cs);
-    }
-    else
-    {
-      changeColour(cs);
-    }
+    changeColour(viewport.getGlobalColourScheme());
 
     modifyConservation_actionPerformed(null);
   }
@@ -1579,14 +1626,7 @@ public class AlignFrame
 
     ColourSchemeI cs = viewport.getGlobalColourScheme();
 
-    if (cs instanceof ConservationColourScheme)
-    {
-      changeColour( ( (ConservationColourScheme) cs).cs);
-    }
-    else
-    {
-      changeColour(cs);
-    }
+    changeColour(viewport.getGlobalColourScheme());
 
     modifyPID_actionPerformed(null);
   }
@@ -1632,8 +1672,37 @@ public class AlignFrame
 
       while (userColours.hasMoreElements())
       {
-        JRadioButtonMenuItem radioItem = new JRadioButtonMenuItem(userColours.
+        final JRadioButtonMenuItem radioItem = new JRadioButtonMenuItem(userColours.
             nextElement().toString());
+        radioItem.setName("USER_DEFINED");
+        radioItem.addMouseListener(new MouseAdapter()
+            {
+              public void mousePressed(MouseEvent evt)
+              {
+                if(evt.isControlDown() || SwingUtilities.isRightMouseButton(evt))
+                {
+                  radioItem.removeActionListener(radioItem.getActionListeners()[0]);
+
+                  int option = JOptionPane.showInternalConfirmDialog(jalview.gui.Desktop.desktop,
+                      "Remove from default list?",
+                      "Remove user defined colour",
+                      JOptionPane.YES_NO_OPTION);
+                  if(option == JOptionPane.YES_OPTION)
+                  {
+                    jalview.gui.UserDefinedColours.removeColourFromDefaults(radioItem.getText());
+                    colourMenu.remove(radioItem);
+                  }
+                  else
+                    radioItem.addActionListener(new ActionListener()
+                    {
+                      public void actionPerformed(ActionEvent evt)
+                      {
+                        userDefinedColour_actionPerformed(evt);
+                      }
+                    });
+                }
+              }
+            });
         radioItem.addActionListener(new ActionListener()
         {
           public void actionPerformed(ActionEvent evt)
@@ -1641,6 +1710,7 @@ public class AlignFrame
             userDefinedColour_actionPerformed(evt);
           }
         });
+
         colourMenu.insert(radioItem, 15);
         colours.add(radioItem);
       }
@@ -1766,21 +1836,11 @@ public class AlignFrame
       return;
     }
 
-    try
-    {
       PCAPanel pcaPanel = new PCAPanel(viewport, null);
       JInternalFrame frame = new JInternalFrame();
       frame.setContentPane(pcaPanel);
       Desktop.addInternalFrame(frame, "Principal component analysis",
                                400, 400);
-    }
-    catch (java.lang.OutOfMemoryError ex)
-    {
-      JOptionPane.showInternalMessageDialog(this,
-                                            "Too many sequences selected\nfor Principal Component Analysis!!",
-                                            "Out of memory",
-                                            JOptionPane.WARNING_MESSAGE);
-    }
   }
 
   /**
@@ -1962,17 +2022,6 @@ public class AlignFrame
       }
       ;
     });
-    viewport.addPropertyChangeListener(new java.beans.PropertyChangeListener()
-    {
-      public void propertyChange(PropertyChangeEvent evt)
-      {
-        if (evt.getPropertyName().equals("alignment"))
-        {
-          treePanel.getTree().UpdatePlaceHolders( (Vector) evt.getNewValue());
-          treePanel.repaint();
-        }
-      }
-    });
   }
 
   /**
@@ -2119,6 +2168,11 @@ public class AlignFrame
     }
   }
 
+
+  public TreePanel ShowNewickTree(NewickFile nf, String title)
+  {
+    return ShowNewickTree(nf,title,600,500,4,5);
+  }
   /**
    * DOCUMENT ME!
    *
@@ -2127,7 +2181,7 @@ public class AlignFrame
    *
    * @return DOCUMENT ME!
    */
-  public TreePanel ShowNewickTree(NewickFile nf, String title)
+  public TreePanel ShowNewickTree(NewickFile nf, String title, int w,int h,int x, int y)
   {
     TreePanel tp = null;
 
@@ -2141,7 +2195,14 @@ public class AlignFrame
                            viewport.getAlignment().getSequences(), nf,
                            "FromFile",
                            title);
-        Desktop.addInternalFrame(tp, title, 600, 500);
+
+        tp.setSize(w,h);
+
+        if(x>0 && y>0)
+          tp.setLocation(x,y);
+
+
+        Desktop.addInternalFrame(tp, title, w, h);
         addTreeMenuItem(tp, title);
       }
     }