right align ids
[jalview.git] / src / jalview / gui / AlignFrame.java
index 79f1e8b..18ebf0e 100755 (executable)
@@ -33,10 +33,11 @@ import jalview.datamodel.*;
 import jalview.io.*;
 import jalview.jbgui.*;
 import jalview.schemes.*;
-import jalview.util.ShiftList;
+import jalview.commands.*;
 import jalview.ws.*;
 import java.awt.dnd.*;
-import org.biojava.dasobert.eventmodel.*;
+import javax.swing.event.ChangeListener;
+import javax.swing.event.ChangeEvent;
 
 /**
  * DOCUMENT ME!
@@ -44,8 +45,7 @@ import org.biojava.dasobert.eventmodel.*;
  * @author $author$
  * @version $Revision$
  */
-public class AlignFrame
-    extends GAlignFrame implements DropTargetListener, FeatureListener
+public class AlignFrame extends GAlignFrame implements DropTargetListener
 {
   /** DOCUMENT ME!! */
   public static final int DEFAULT_WIDTH = 700;
@@ -53,16 +53,17 @@ public class AlignFrame
   /** DOCUMENT ME!! */
   public static final int DEFAULT_HEIGHT = 500;
   public AlignmentPanel alignPanel;
-  AlignViewport viewport;
 
+  AlignViewport viewport;
 
   Vector alignPanels = new Vector();
 
 
   /** DOCUMENT ME!! */
-  public String currentFileFormat = null;
-  Stack historyList = new Stack();
-  Stack redoList = new Stack();
+  String currentFileFormat = null;
+
+  String fileName = null;
+
   private int treeCount = 0;
 
   /**
@@ -94,7 +95,7 @@ public class AlignFrame
       al.setDataset(null);
     }
 
-    addAlignmentPanel(alignPanel);
+    addAlignmentPanel(alignPanel, true);
     init();
   }
 
@@ -107,7 +108,7 @@ public class AlignFrame
   {
     viewport = ap.av;
     alignPanel = ap;
-    addAlignmentPanel(ap);
+    addAlignmentPanel(ap, false);
     init();
   }
 
@@ -115,15 +116,14 @@ public class AlignFrame
   {
     this.setDropTarget(new java.awt.dnd.DropTarget(this, this));
 
-    if (viewport.vconsensus == null)
+    if (viewport.conservation == null)
     {
-      //Out of memory calculating consensus.
       BLOSUM62Colour.setEnabled(false);
-      PIDColour.setEnabled(false);
       conservationMenuItem.setEnabled(false);
       modifyConservation.setEnabled(false);
-      abovePIDThreshold.setEnabled(false);
-      modifyPID.setEnabled(false);
+    //  PIDColour.setEnabled(false);
+    //  abovePIDThreshold.setEnabled(false);
+    //  modifyPID.setEnabled(false);
     }
 
     String sortby = jalview.bin.Cache.getDefault("SORT_ALIGNMENT", "No sort");
@@ -139,13 +139,22 @@ public class AlignFrame
      setGUINucleotide(viewport.alignment.isNucleotide());
    }
 
-   if (jalview.bin.Cache.getDefault("WRAP_ALIGNMENT", false))
+   setMenusFromViewport(viewport);
+
+   if (viewport.wrapAlignment)
    {
-     wrapMenuItem.setSelected(true);
      wrapMenuItem_actionPerformed(null);
    }
 
    addKeyListener();
+
+  }
+
+  public void setFileName(String file, String format)
+  {
+     fileName = file;
+     currentFileFormat = format;
+     reload.setEnabled(true);
   }
 
   void addKeyListener()
@@ -164,36 +173,6 @@ public class AlignFrame
 
           switch (evt.getKeyCode())
           {
-            case KeyEvent.VK_V:
-              if (!evt.isControlDown())
-              {
-                AlignmentI al;
-                if(viewport.hasHiddenRows)
-                {
-                  al = viewport.alignment.getHiddenSequences().getFullAlignment();
-                }
-                else
-                  al = new Alignment(viewport.alignment.getSequencesArray());
-
-                AlignViewport newview = new AlignViewport(al);
-
-                AlignmentPanel copy = new AlignmentPanel(af, newview);
-
-                if(viewport.viewName==null)
-                  viewport.viewName="View 1";
-
-                copy.av.sequenceSetID = viewport.getSequenceSetId();
-                copy.av.viewName = "View " +
-                    (Desktop.getViewCount(viewport.getSequenceSetId())+1);
-
-
-                addAlignmentPanel(copy);
-                PaintRefresher.Register(copy, viewport.getSequenceSetId());
-                PaintRefresher.Register(alignPanel,
-                                        viewport.getSequenceSetId());
-
-              }
-              break;
             case KeyEvent.VK_G:
               Desktop.instance.gatherViews(af);
               break;
@@ -382,21 +361,24 @@ public class AlignFrame
                                            viewport.startSeq
                                            + viewport.endSeq - viewport.startSeq);
               break;
-
-
           }
         }
       });
   }
 
 
-  public void addAlignmentPanel(final AlignmentPanel ap)
+  public void addAlignmentPanel(final AlignmentPanel ap,
+                                boolean newPanel)
   {
+    ap.alignFrame = this;
+
     alignPanels.addElement(ap);
 
+    PaintRefresher.Register(ap, ap.av.getSequenceSetId());
+
     int aSize = alignPanels.size();
 
-   tabbedPane.setVisible(aSize>1 || ap.av.viewName!=null);
+    tabbedPane.setVisible(aSize>1 || ap.av.viewName!=null);
 
     if (aSize == 1 && ap.av.viewName==null)
     {
@@ -413,26 +395,20 @@ public class AlignFrame
       }
 
       tabbedPane.addTab(ap.av.viewName==null?"Original":ap.av.viewName, ap);
-    }
 
+      ap.setVisible(false);
+    }
 
-    ap.av.addPropertyChangeListener(new PropertyChangeListener()
+    if(newPanel)
     {
-     public void propertyChange(PropertyChangeEvent evt)
-     {
-       if (evt.getPropertyName().equals("alignment"))
-       {
-         PaintRefresher.Refresh(ap, ap.av.getSequenceSetId());
-         alignmentChanged();
-       }
-     }
-   });
-
+      if (ap.av.padGaps)
+        ap.av.alignment.padGaps();
+      ap.av.updateConservation(ap);
+      ap.av.updateConsensus(ap);
+    }
   }
 
 
-
-
   public AlignViewport getViewport()
   {
     return viewport;
@@ -466,7 +442,7 @@ public class AlignFrame
       {
         // System.out.println("deregistering discoverer listener");
         Desktop.discoverer.removePropertyChangeListener(thisListener);
-        closeMenuItem_actionPerformed(null);
+        closeMenuItem_actionPerformed(true);
       }
       ;
     });
@@ -475,10 +451,8 @@ public class AlignFrame
   public void setGUINucleotide(boolean nucleotide)
   {
     showTranslation.setVisible( nucleotide );
-    //sequenceFeatures.setVisible(!nucleotide );
-    //featureSettings.setVisible( !nucleotide );
-    conservationMenuItem.setVisible( !nucleotide );
-    modifyConservation.setVisible(   !nucleotide );
+    conservationMenuItem.setEnabled( !nucleotide );
+    modifyConservation.setEnabled(   !nucleotide );
 
     //Remember AlignFrame always starts as protein
     if(!nucleotide)
@@ -487,18 +461,36 @@ public class AlignFrame
     }
   }
 
-  public void comeBackLater(FeatureEvent evt)
-  {}
-
-  public void newFeatures(FeatureEvent evt)
+  /**
+   * Need to call this method when tabs are selected for multiple views,
+   * or when loading from Jalview2XML.java
+   * @param av AlignViewport
+   */
+  void setMenusFromViewport(AlignViewport av)
   {
-    if (evt.getFeatures().length > 0)
-    {
-      alignPanel.seqPanel.seqCanvas.fr.featuresAdded();
-      alignPanel.repaint();
-    }
+    padGapsMenuitem.setSelected(av.padGaps);
+    colourTextMenuItem.setSelected(av.showColourText);
+    abovePIDThreshold.setSelected(av.getAbovePIDThreshold());
+    conservationMenuItem.setSelected(av.getConservationSelected());
+    seqLimits.setSelected(av.getShowJVSuffix());
+    idRightAlign.setSelected(av.idsAlignRight);
+    renderGapsMenuItem.setSelected(av.renderGaps);
+    wrapMenuItem.setSelected(av.wrapAlignment);
+    annotationPanelMenuItem.setState(av.showAnnotation);
+    viewBoxesMenuItem.setSelected(av.showBoxes);
+    viewTextMenuItem.setSelected(av.showText);
+
+    setColourSelected(ColourSchemeProperty.
+                      getColourName(av.getGlobalColourScheme()));
+
+    showSeqFeatures.setSelected(av.showSequenceFeatures);
+    hiddenMarkers.setState(av.showHiddenMarkers);
+    applyToAllGroups.setState(av.colourAppliesToAllGroups);
+
+    updateEditMenuBar();
   }
 
+
   Hashtable progressBars;
   public void setProgressBar(String message, long id)
   {
@@ -538,6 +530,8 @@ public class AlignFrame
   }
 
 
+
+
   /*
    Added so Castor Mapping file can obtain Jalview Version
   */
@@ -562,6 +556,44 @@ public class AlignFrame
     Desktop.instance.inputLocalFileMenuItem_actionPerformed(viewport);
   }
 
+  public void reload_actionPerformed(ActionEvent e)
+  {
+    if(fileName!=null)
+    {
+      if(currentFileFormat.equals("Jalview"))
+      {
+        this.closeMenuItem_actionPerformed(true);
+      }
+      else
+      {
+        viewport.alignment.deleteAllGroups();
+        viewport.sequenceColours=null;
+        while (viewport.alignment.getHeight() > 0)
+        {
+          viewport.alignment.deleteSequence(0);
+        }
+
+        viewport.historyList.clear();
+        viewport.redoList.clear();
+        Alignment dset = viewport.alignment.getDataset();
+        while (dset.getHeight() > 0)
+        {
+          dset.deleteSequence(0);
+        }
+
+        firePropertyChange("Alignment", null, null);
+
+        updateEditMenuBar();
+      }
+
+      FileLoader loader = new FileLoader();
+      String protocol = fileName.startsWith("http:")? "URL":"File";
+      loader.LoadFile(viewport, fileName, protocol, currentFileFormat);
+
+    }
+  }
+
+
   public void addFromText_actionPerformed(ActionEvent e)
   {
     Desktop.instance.inputTextboxMenuItem_actionPerformed(viewport);
@@ -572,12 +604,21 @@ public class AlignFrame
     Desktop.instance.inputURLMenuItem_actionPerformed(viewport);
   }
 
+
+  public void save_actionPerformed(ActionEvent e)
+  {
+    if(fileName==null || currentFileFormat==null)
+      saveAs_actionPerformed(null);
+    else
+      saveAlignment(fileName, currentFileFormat);
+  }
+
   /**
    * DOCUMENT ME!
    *
    * @param e DOCUMENT ME!
    */
-  public void saveAlignmentMenu_actionPerformed(ActionEvent e)
+  public void saveAs_actionPerformed(ActionEvent e)
   {
     JalviewFileChooser chooser = new JalviewFileChooser(jalview.bin.Cache.
         getProperty( "LAST_DIRECTORY"),
@@ -598,7 +639,6 @@ public class AlignFrame
     if (value == JalviewFileChooser.APPROVE_OPTION)
     {
         currentFileFormat = chooser.getSelectedFormat();
-
         if (currentFileFormat == null)
         {
           JOptionPane.showInternalMessageDialog(Desktop.desktop,
@@ -609,13 +649,14 @@ public class AlignFrame
           return;
         }
 
+        fileName = chooser.getSelectedFile().getPath();
+
       jalview.bin.Cache.setProperty("DEFAULT_FILE_FORMAT",
                                     currentFileFormat);
 
-      String choice = chooser.getSelectedFile().getPath();
-      jalview.bin.Cache.setProperty("LAST_DIRECTORY", choice);
+      jalview.bin.Cache.setProperty("LAST_DIRECTORY", fileName);
 
-      saveAlignment(choice, currentFileFormat);
+      saveAlignment(fileName, currentFileFormat);
     }
   }
 
@@ -633,6 +674,11 @@ public class AlignFrame
 
       new Jalview2XML().SaveAlignment(this, file, shortName);
 
+      statusBar.setText("Successfully saved to file: "
+                          +fileName+" in "
+                          +format +" format.");
+
+
       // USE Jalview2XML to save this file
       return true;
     }
@@ -671,6 +717,9 @@ public class AlignFrame
         out.print(output);
         out.close();
         this.setTitle(file);
+        statusBar.setText("Successfully saved to file: "
+                          +fileName+" in "
+                          +format +" format.");
         return true;
       }
       catch (Exception ex)
@@ -705,6 +754,7 @@ public class AlignFrame
     }
 
     CutAndPasteTransfer cap = new CutAndPasteTransfer();
+    cap.setForInput(null);
     Desktop.addInternalFrame(cap,
                              "Alignment output - " + e.getActionCommand(), 600,
                              500);
@@ -807,22 +857,49 @@ public class AlignFrame
    *
    * @param e DOCUMENT ME!
    */
-  public void closeMenuItem_actionPerformed(ActionEvent e)
+  public void closeMenuItem_actionPerformed(boolean closeAllTabs)
   {
+    if(alignPanels!=null && alignPanels.size()<2)
+      closeAllTabs = true;
+
     try
     {
-      for(int i=0; i<alignPanels.size(); i++)
+      if(alignPanels!=null)
       {
-        AlignmentPanel ap = (AlignmentPanel)alignPanels.elementAt(i);
-        PaintRefresher.RemoveComponent(ap.seqPanel.seqCanvas);
-        PaintRefresher.RemoveComponent(ap.idPanel.idCanvas);
-        PaintRefresher.RemoveComponent(ap);
+        if (closeAllTabs)
+          for (int i = 0; i < alignPanels.size(); i++)
+          {
+            AlignmentPanel ap = (AlignmentPanel) alignPanels.elementAt(i);
+            PaintRefresher.RemoveComponent(ap.seqPanel.seqCanvas);
+            PaintRefresher.RemoveComponent(ap.idPanel.idCanvas);
+            PaintRefresher.RemoveComponent(ap);
+          }
+        else
+        {
+          int index = tabbedPane.getSelectedIndex();
+
+          tabbedPane.removeTabAt(index);
+          alignPanels.removeElement(alignPanel);
+          PaintRefresher.RemoveComponent(alignPanel.seqPanel.seqCanvas);
+          PaintRefresher.RemoveComponent(alignPanel.idPanel.idCanvas);
+          PaintRefresher.RemoveComponent(alignPanel);
+          alignPanel = null;
+          viewport = null;
+          tabbedPane.validate();
+
+          if(index==tabbedPane.getTabCount())
+            index --;
+
+          this.tabSelectionChanged(index);
+        }
       }
 
-      this.setClosed(true);
+      if (closeAllTabs)
+        this.setClosed(true);
     }
     catch (Exception ex)
     {
+      ex.printStackTrace();
     }
   }
 
@@ -832,12 +909,12 @@ public class AlignFrame
    */
   void updateEditMenuBar()
   {
-    if (historyList.size() > 0)
+
+    if (viewport.historyList.size() > 0)
     {
       undoMenuItem.setEnabled(true);
-
-      HistoryItem hi = (HistoryItem) historyList.peek();
-      undoMenuItem.setText("Undo " + hi.getDescription());
+      CommandI command = (CommandI) viewport.historyList.peek();
+      undoMenuItem.setText("Undo " + command.getDescription());
     }
     else
     {
@@ -845,12 +922,12 @@ public class AlignFrame
       undoMenuItem.setText("Undo");
     }
 
-    if (redoList.size() > 0)
+    if (viewport.redoList.size() > 0)
     {
       redoMenuItem.setEnabled(true);
 
-      HistoryItem hi = (HistoryItem) redoList.peek();
-      redoMenuItem.setText("Redo " + hi.getDescription());
+      CommandI command = (CommandI) viewport.redoList.peek();
+      redoMenuItem.setText("Redo " + command.getDescription());
     }
     else
     {
@@ -859,18 +936,20 @@ public class AlignFrame
     }
   }
 
-  /**
-   * DOCUMENT ME!
-   *
-   * @param hi DOCUMENT ME!
-   */
-  public void addHistoryItem(HistoryItem hi)
+
+  public void addHistoryItem(CommandI command)
   {
-    historyList.push(hi);
-    redoList.clear();
-    updateEditMenuBar();
+    if(command.getSize()>0)
+    {
+      viewport.historyList.push(command);
+      viewport.redoList.clear();
+      updateEditMenuBar();
+      viewport.hasHiddenColumns = viewport.colSel.getHiddenColumns() != null;
+    }
   }
 
+
+
   /**
    * DOCUMENT ME!
    *
@@ -878,13 +957,14 @@ public class AlignFrame
    */
   protected void undoMenuItem_actionPerformed(ActionEvent e)
   {
-    HistoryItem nh,hi = (HistoryItem) historyList.pop();
-    redoList.push(nh=new HistoryItem(hi.getDescription(), viewport.alignment,
-                                  HistoryItem.HIDE));
-    if (hi.alColumnChanges!=null)
-      nh.alColumnChanges = hi.alColumnChanges.getInverse();
-    restoreHistoryItem(hi);
-    viewport.firePropertyChange("alignment", null, viewport.getAlignment().getSequences());
+    CommandI command = (CommandI)viewport.historyList.pop();
+    viewport.redoList.push(command);
+    command.undoCommand();
+
+
+    viewport.hasHiddenColumns = viewport.colSel.getHiddenColumns() != null;
+    updateEditMenuBar();
+    viewport.firePropertyChange("alignment", null,null);
   }
 
   /**
@@ -894,28 +974,16 @@ public class AlignFrame
    */
   protected void redoMenuItem_actionPerformed(ActionEvent e)
   {
-    HistoryItem nh,hi = (HistoryItem) redoList.pop();
-    historyList.push(nh=new HistoryItem(hi.getDescription(), viewport.alignment,
-        HistoryItem.HIDE));
-    if (hi.alColumnChanges!=null)
-      nh.alColumnChanges=hi.alColumnChanges.getInverse();
-    restoreHistoryItem(hi);
-    updateEditMenuBar();
-    viewport.firePropertyChange("alignment", null, viewport.getAlignment().getSequences());
-  }
-
-  // used by undo and redo
-  void restoreHistoryItem(HistoryItem hi)
-  {
-
-    hi.restore(viewport.getColumnSelection());
+    CommandI command = (CommandI) viewport.redoList.pop();
+    viewport.historyList.push(command);
+    command.doCommand();
+    viewport.hasHiddenColumns = viewport.colSel.getHiddenColumns()!=null;
 
     updateEditMenuBar();
-
-    viewport.firePropertyChange("alignment", null,
-                                viewport.getAlignment().getSequences());
+    viewport.firePropertyChange("alignment", null, null);
   }
 
+
   /**
    * DOCUMENT ME!
    *
@@ -1077,8 +1145,6 @@ public class AlignFrame
    */
   protected void pasteThis_actionPerformed(ActionEvent e)
   {
-    addHistoryItem(new HistoryItem("Paste Sequences", viewport.alignment,
-                                   HistoryItem.PASTE));
     paste(false);
   }
 
@@ -1158,18 +1224,28 @@ public class AlignFrame
         alignment = viewport.getAlignment();
 
         //!newAlignment
+        SequenceI [] newseqs = new SequenceI[sequences.length];
         for (int i = 0; i < sequences.length; i++)
         {
-          Sequence newseq = new Sequence(sequences[i].getName(),
+          newseqs[i] = new Sequence(sequences[i].getName(),
               sequences[i].getSequence(), sequences[i].getStart(),
               sequences[i].getEnd());
 
-          alignment.addSequence(newseq);
-          PaintRefresher.Refresh(alignPanel,
-                                 alignPanel.av.getSequenceSetId(),
-                                 null,newseq);
+          alignment.addSequence(newseqs[i]);
         }
 
+        /*
+         //ADD HISTORY ITEM
+         */
+        addHistoryItem(new EditCommand(
+            "Add sequences",
+            EditCommand.PASTE,
+            newseqs,
+            0,
+            alignment.getWidth(),
+            alignment)
+            );
+
 
         viewport.setEndSeq(alignment.getHeight());
         alignment.getWidth();
@@ -1270,46 +1346,47 @@ public class AlignFrame
    *
    * @param e DOCUMENT ME!
    */
-  protected void delete_actionPerformed(ActionEvent e)
+  protected void delete_actionPerformed(ActionEvent evt)
   {
 
-    if (viewport.getSelectionGroup() == null)
+    SequenceGroup sg = viewport.getSelectionGroup();
+    if (sg == null)
     {
       return;
     }
 
-
-    SequenceGroup sg = viewport.getSelectionGroup();
+    Vector seqs = new Vector();
+    SequenceI seq;
+    for (int i = 0; i < sg.getSize(false); i++)
+    {
+      seq = sg.getSequenceAt(i);
+      seqs.addElement(seq);
+    }
 
 
-    addHistoryItem(new HistoryItem("Delete Sequences", viewport.alignment,
-                                   HistoryItem.HIDE));
+   // If the cut affects all sequences, remove highlighted columns
+   if (sg.getSize(false) == viewport.alignment.getHeight())
+   {
+     viewport.getColumnSelection().removeElements(sg.getStartRes(),
+         sg.getEndRes() + 1);
+   }
 
 
-    for (int i = 0; i < sg.getSize(false); i++)
-    {
-      SequenceI seq = sg.getSequenceAt(i);
-      int index = viewport.getAlignment().findIndex(seq);
+    SequenceI [] cut = new SequenceI[seqs.size()];
+    for(int i=0; i<seqs.size(); i++)
+      cut[i] = (SequenceI)seqs.elementAt(i);
 
-      seq.deleteChars(sg.getStartRes(), sg.getEndRes() + 1);
 
-      // If the cut affects all sequences, remove highlighted columns
-      if (sg.getSize(false) == viewport.alignment.getHeight())
-      {
-        viewport.getColumnSelection().removeElements(sg.getStartRes(),
-            sg.getEndRes() + 1);
-      }
+    /*
+    //ADD HISTORY ITEM
+    */
+    addHistoryItem(new EditCommand("Cut Sequences",
+                                      EditCommand.CUT,
+                                      cut,
+                                      sg.getStartRes(),
+                                      sg.getEndRes()-sg.getStartRes()+1,
+                                      viewport.alignment));
 
-      if (seq.getSequence().length() < 1)
-      {
-        viewport.getAlignment().deleteSequence(seq);
-        PaintRefresher.Refresh(alignPanel,alignPanel.av.getSequenceSetId(),seq,null);
-      }
-      else
-      {
-        viewport.getAlignment().getSequences().setElementAt(seq, index);
-      }
-    }
 
     viewport.setSelectionGroup(null);
     viewport.alignment.deleteGroup(sg);
@@ -1317,8 +1394,6 @@ public class AlignFrame
     viewport.firePropertyChange("alignment", null,
                                   viewport.getAlignment().getSequences());
 
-
-
     if (viewport.getAlignment().getHeight() < 1)
     {
       try
@@ -1339,6 +1414,7 @@ public class AlignFrame
   protected void deleteGroups_actionPerformed(ActionEvent e)
   {
     viewport.alignment.deleteAllGroups();
+    viewport.sequenceColours.clear();
     viewport.setSelectionGroup(null);
     PaintRefresher.Refresh(this, viewport.getSequenceSetId());
     alignPanel.repaint();
@@ -1383,7 +1459,7 @@ public class AlignFrame
     alignPanel.seqPanel.seqCanvas.highlightSearchResults(null);
     alignPanel.idPanel.idCanvas.searchResults = null;
     alignPanel.repaint();
-    PaintRefresher.Refresh(this, viewport.getSequenceSetId());
+    PaintRefresher.Refresh(alignPanel, viewport.getSequenceSetId());
   }
 
   /**
@@ -1408,7 +1484,7 @@ public class AlignFrame
       sg.addOrRemove(viewport.getAlignment().getSequenceAt(i), false);
     }
 
-    PaintRefresher.Refresh(this, viewport.getSequenceSetId());
+    PaintRefresher.Refresh(alignPanel, viewport.getSequenceSetId());
   }
 
   public void invertColSel_actionPerformed(ActionEvent e)
@@ -1425,37 +1501,7 @@ public class AlignFrame
    */
   public void remove2LeftMenuItem_actionPerformed(ActionEvent e)
   {
-    ColumnSelection colSel = viewport.getColumnSelection();
-
-    if (colSel.size() > 0)
-    {
-      HistoryItem edit;
-      addHistoryItem(edit=new HistoryItem("Remove Left", viewport.alignment,
-                                     HistoryItem.HIDE));
-
-      int min = colSel.getMin();
-      viewport.getAlignment().trimLeft(min);
-      colSel.compensateForEdit(0, min);
-      edit.addShift(0,min);
-      if (viewport.getSelectionGroup() != null)
-      {
-        viewport.getSelectionGroup().adjustForRemoveLeft(min);
-      }
-
-      Vector groups = viewport.alignment.getGroups();
-
-      for (int i = 0; i < groups.size(); i++)
-      {
-        SequenceGroup sg = (SequenceGroup) groups.get(i);
-
-        if (!sg.adjustForRemoveLeft(min))
-        {
-          viewport.alignment.deleteGroup(sg);
-        }
-      }
-
-      viewport.firePropertyChange("alignment", null, viewport.getAlignment().getSequences());
-    }
+    trimAlignment(true);
   }
 
   /**
@@ -1465,21 +1511,55 @@ public class AlignFrame
    */
   public void remove2RightMenuItem_actionPerformed(ActionEvent e)
   {
+      trimAlignment(false);
+  }
+
+  void trimAlignment(boolean trimLeft)
+  {
     ColumnSelection colSel = viewport.getColumnSelection();
+    int column;
 
     if (colSel.size() > 0)
     {
-      addHistoryItem(new HistoryItem("Remove Right", viewport.alignment,
-                                     HistoryItem.HIDE));
+      if(trimLeft)
+        column = colSel.getMin();
+      else
+        column = colSel.getMax();
+
+      SequenceI [] seqs;
+      if(viewport.getSelectionGroup()!=null)
+        seqs = viewport.getSelectionGroup().getSequencesAsArray(true);
+      else
+        seqs = viewport.alignment.getSequencesArray();
+
 
-      int max = colSel.getMax();
-      viewport.getAlignment().trimRight(max);
-      // TODO: delete hidden column entries in colSel to right of max
-      // TODO: record hidden columns in history for undo.
-      if (viewport.getSelectionGroup() != null)
+      TrimRegionCommand trimRegion;
+      if(trimLeft)
       {
-        viewport.getSelectionGroup().adjustForRemoveRight(max);
+        trimRegion = new TrimRegionCommand("Remove Left",
+                                    TrimRegionCommand.TRIM_LEFT,
+                                    seqs,
+                                    column,
+                                    viewport.alignment,
+                                    viewport.colSel,
+                                    viewport.selectionGroup);
+        viewport.setStartRes(0);
       }
+     else
+     {
+       trimRegion = new TrimRegionCommand("Remove Right",
+                                   TrimRegionCommand.TRIM_RIGHT,
+                                   seqs,
+                                   column,
+                                   viewport.alignment,
+                                   viewport.colSel,
+                                   viewport.selectionGroup);
+     }
+
+     statusBar.setText("Removed "+trimRegion.getSize()+" columns.");
+
+
+      addHistoryItem(trimRegion);
 
       Vector groups = viewport.alignment.getGroups();
 
@@ -1487,13 +1567,15 @@ public class AlignFrame
       {
         SequenceGroup sg = (SequenceGroup) groups.get(i);
 
-        if (!sg.adjustForRemoveRight(max))
+        if ( (trimLeft && !sg.adjustForRemoveLeft(column))
+            || (!trimLeft && !sg.adjustForRemoveRight(column)))
         {
           viewport.alignment.deleteGroup(sg);
         }
       }
 
-      viewport.firePropertyChange("alignment", null, viewport.getAlignment().getSequences());
+      viewport.firePropertyChange("alignment", null,
+                                  viewport.getAlignment().getSequences());
     }
   }
 
@@ -1504,20 +1586,39 @@ public class AlignFrame
    */
   public void removeGappedColumnMenuItem_actionPerformed(ActionEvent e)
   {
-    HistoryItem edit;
-    addHistoryItem(edit=new HistoryItem("Remove Gapped Columns",
-                                   viewport.alignment, HistoryItem.HIDE));
+    int start = 0, end = viewport.alignment.getWidth()-1;
+
+    SequenceI[] seqs;
+    if (viewport.getSelectionGroup() != null)
+    {
+      seqs = viewport.getSelectionGroup().getSequencesAsArray(true);
+      start = viewport.getSelectionGroup().getStartRes();
+      end = viewport.getSelectionGroup().getEndRes();
+    }
+    else
+      seqs = viewport.alignment.getSequencesArray();
+
+
+    RemoveGapColCommand removeGapCols =
+        new RemoveGapColCommand("Remove Gapped Columns",
+                                seqs,
+                                start, end,
+                                viewport.getGapCharacter());
+
+    addHistoryItem(removeGapCols);
+
+    statusBar.setText("Removed "+removeGapCols.getSize()+" empty columns.");
 
     //This is to maintain viewport position on first residue
     //of first sequence
     SequenceI seq = viewport.alignment.getSequenceAt(0);
     int startRes = seq.findPosition(viewport.startRes);
-    ShiftList shifts;
-    viewport.getAlignment().removeGaps(shifts=new ShiftList());
-    edit.alColumnChanges=shifts.getInverse();
-    if (viewport.hasHiddenColumns)
-      viewport.getColumnSelection().compensateForEdits(shifts);
-    viewport.setStartRes(seq.findIndex(startRes)-1);
+   // ShiftList shifts;
+   // viewport.getAlignment().removeGaps(shifts=new ShiftList());
+   // edit.alColumnChanges=shifts.getInverse();
+   // if (viewport.hasHiddenColumns)
+   //   viewport.getColumnSelection().compensateForEdits(shifts);
+   viewport.setStartRes(seq.findIndex(startRes)-1);
    viewport.firePropertyChange("alignment", null, viewport.getAlignment().getSequences());
 
   }
@@ -1529,71 +1630,27 @@ public class AlignFrame
    */
   public void removeAllGapsMenuItem_actionPerformed(ActionEvent e)
   {
-    // TODO: hidden regions should not be touched by removeAllGaps - a minimal number of gaps will remain in alignment segments containing uneven length subsequences
-    // TODO: columnSelection.compensateforedits should be called (and passed to history item)
-    addHistoryItem(new HistoryItem("Remove Gaps", viewport.alignment,
-                                   HistoryItem.HIDE));
+    int start = 0, end = viewport.alignment.getWidth()-1;
+
+    SequenceI[] seqs;
+    if (viewport.getSelectionGroup() != null)
+    {
+      seqs = viewport.getSelectionGroup().getSequencesAsArray(true);
+      start = viewport.getSelectionGroup().getStartRes();
+      end = viewport.getSelectionGroup().getEndRes();
+    }
+    else
+      seqs = viewport.alignment.getSequencesArray();
 
     //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;
-
-    Vector seqs = null;
-
-    int start = 0;
-    int end = viewport.alignment.getWidth();
-
-    if (viewport.getSelectionGroup() != null
-        && viewport.getSelectionGroup().getSequences(true) != null
-        && viewport.getSelectionGroup().getSize(true) > 0)
-    {
-      seqs = viewport.getSelectionGroup().getSequences(true);
-      start = viewport.getSelectionGroup().getStartRes();
-      end = viewport.getSelectionGroup().getEndRes()+1;
-    }
-    else
-    {
-      seqs = viewport.alignment.getSequences();
-    }
-    /* Commented out regions below are partial implementation of todo above.
-       * divide start,end into visible chunks, and for each:
-      int diff=end-start+1;
-      int diffmax=0;
-      int dr[] = new int[seqs.size()];
-      */
-     for (int i = 0; i < seqs.size(); i++)
-     {
-       current = (SequenceI) seqs.elementAt(i);
-       //dr[i]=
-       current.removeGaps(start, end);
-       /*if (d<diff) // can only shift
-         diff=d;
-       if (diffmax<d)
-         diffmax=d;
-         */
-     }
-     /* // after the end of each chunk -
-      * if (diff>0) {
-      // record shift for history.
-       editgaps.addShift(start, diff);
-       if (viewport.hasHiddenColumns && diffmax>diff) {
-       // pad sequence
-        StringBuffer gaps=new StringBuffer(diffmax);
-        for (int i=0,j=diffmax-diff; i<j; i++)
-        gaps.append(viewport.getGapCharacter());
-        for (int i=0, j=seqs.size(); i<j; i++) {
-        current = (SequenceI) seqs.elementAt(i);
-        if (dr[i]-diff>0) {
-        String sq = current.getSequence();
-        current.setSequence(sq.substring(0, hcend-dr[i])+gaps.substring(0, dr[i]-diff)+sq.substring());
-        }
-        }
-       }
-       }*/
+    addHistoryItem(new RemoveGapsCommand("Remove Gaps",
+                                           seqs,
+                                           start, end,
+                     viewport.getGapCharacter()));
 
     viewport.setStartRes(seq.findIndex(startRes)-1);
 
@@ -1601,76 +1658,6 @@ public class AlignFrame
 
   }
 
- public void alignmentChanged()
- {
-   AlignViewport av;
-   AlignmentPanel ap;
-
-   for(int i=0; i<alignPanels.size(); i++)
-   {
-     ap = (AlignmentPanel) alignPanels.elementAt(i);
-     av = ap.av;
-
-   if (av.padGaps)
-       av.getAlignment().padGaps();
-
-     if (av.vconsensus != null && av.autoCalculateConsensus)
-     {
-       av.updateConsensus();
-       av.updateConservation();
-     }
-
-     resetAllColourSchemes();
-
-     av.alignment.adjustSequenceAnnotations();
-
-     if (ap.overviewPanel != null)
-       ap.overviewPanel.updateOverviewImage();
-
-     ap.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.getSequences(true), sg.getWidth());
-      }
-      sg.recalcConservation();
-    }
-  }
-
   /**
    * DOCUMENT ME!
    *
@@ -1678,9 +1665,6 @@ public class AlignFrame
    */
   public void padGapsMenuitem_actionPerformed(ActionEvent e)
   {
-    addHistoryItem(new HistoryItem("Pad Gaps", viewport.alignment,
-                                   HistoryItem.HIDE));
-
     viewport.padGaps = padGapsMenuitem.isSelected();
 
     viewport.firePropertyChange("alignment",
@@ -1702,6 +1686,29 @@ public class AlignFrame
     Desktop.addInternalFrame(frame, "Find", 340, 110);
   }
 
+  public void newView_actionPerformed(ActionEvent e)
+  {
+    AlignmentPanel newap =
+        new Jalview2XML().copyAlignPanel(alignPanel, true);
+
+    if (viewport.viewName == null)
+      viewport.viewName = "View 1";
+
+    newap.av.historyList = viewport.historyList;
+    newap.av.redoList = viewport.redoList;
+    newap.av.viewName = "View " +
+        (Desktop.getViewCount(viewport.getSequenceSetId()) + 1);
+
+    addAlignmentPanel(newap, false);
+
+    if(alignPanels.size()==2)
+    {
+      viewport.gatherViewsHere = true;
+    }
+    tabbedPane.setSelectedIndex(tabbedPane.getTabCount() - 1);
+  }
+
+
   /**
    * DOCUMENT ME!
    *
@@ -1712,13 +1719,6 @@ public class AlignFrame
     new FontChooser(alignPanel);
   }
 
-  public void smoothFont_actionPerformed(ActionEvent e)
-  {
-    viewport.antiAlias = smoothFont.isSelected();
-    alignPanel.annotationPanel.image = null;
-    alignPanel.repaint();
-  }
-
 
   /**
    * DOCUMENT ME!
@@ -1733,6 +1733,13 @@ public class AlignFrame
     alignPanel.repaint();
   }
 
+  public void idRightAlign_actionPerformed(ActionEvent e)
+  {
+    viewport.idsAlignRight = idRightAlign.isSelected();
+    alignPanel.repaint();
+  }
+
+
 
   /**
    * DOCUMENT ME!
@@ -1924,6 +1931,114 @@ public class AlignFrame
     alignPanel.setOverviewPanel(overview);
   }
 
+  public void textColour_actionPerformed(ActionEvent e)
+  {
+    int original1 = viewport.textColour.getRGB();
+    int original2 = viewport.textColour2.getRGB();
+    int thresh = viewport.thresholdTextColour;
+    final JSlider slider = new JSlider(0, 750, viewport.thresholdTextColour);
+    final JButton col1 = new JButton("DARK");
+    col1.setBackground(viewport.textColour);
+    col1.setForeground(viewport.textColour);
+    final JButton col2 = new JButton("LIGHT");
+    col2.setBackground(viewport.textColour2);
+    col2.setForeground(viewport.textColour2);
+    final JPanel bigpanel = new JPanel(new BorderLayout());
+    JPanel panel = new JPanel();
+    bigpanel.add(panel, BorderLayout.CENTER);
+    bigpanel.add(new JLabel(
+      "<html><i>Select a dark and light text colour, then set the threshold to"
+     +"<br>switch between colours, based on background colour</i></html>"),
+                 BorderLayout.NORTH);
+    panel.add(col1);
+    panel.add(slider);
+    panel.add(col2);
+
+    col1.addActionListener(new ActionListener()
+        {
+          public void actionPerformed(ActionEvent e)
+          {
+            Color col = JColorChooser.showDialog(bigpanel,
+                                                 "Select Colour for Text",
+                                                 viewport.textColour);
+            if (col != null)
+            {
+              viewport.textColour = col;
+              col1.setBackground(col);
+              col1.setForeground(col);
+              if(viewport.colourAppliesToAllGroups)
+                setGroupTextColour();
+            }
+            alignPanel.repaint();
+          }
+        });
+
+    col2.addActionListener(new ActionListener()
+    {
+      public void actionPerformed(ActionEvent e)
+      {
+        Color col = JColorChooser.showDialog(bigpanel,
+                                             "Select Colour for Text",
+                                             viewport.textColour);
+        if (col != null)
+        {
+          viewport.textColour2 = col;
+          col2.setBackground(col);
+          col2.setForeground(col);
+          if(viewport.colourAppliesToAllGroups)
+                setGroupTextColour();
+        }
+        alignPanel.repaint();
+      }
+    });
+
+
+    slider.addChangeListener(new ChangeListener()
+    {
+      public void stateChanged(ChangeEvent evt)
+      {
+        viewport.thresholdTextColour = slider.getValue();
+        if(viewport.colourAppliesToAllGroups)
+                setGroupTextColour();
+        alignPanel.repaint();
+      }
+    });
+
+    int reply = JOptionPane.showInternalOptionDialog(this,
+                                                 bigpanel,
+                                                 "Adjust Foreground Text Colour Threshold",
+                                                 JOptionPane.OK_CANCEL_OPTION,
+                                                 JOptionPane.QUESTION_MESSAGE,
+                                                 null,
+                                                 null, null);
+
+    if(reply==JOptionPane.CANCEL_OPTION)
+    {
+      viewport.textColour = new Color(original1);
+      viewport.textColour2 = new Color(original2);
+      viewport.thresholdTextColour = thresh;
+    }
+  }
+
+  void setGroupTextColour()
+  {
+    if(viewport.alignment.getGroups()==null)
+      return;
+
+    Vector groups = viewport.alignment.getGroups();
+
+    for(int i=0; i<groups.size(); i++)
+    {
+      SequenceGroup sg = (SequenceGroup)groups.elementAt(i);
+      sg.textColour = viewport.textColour;
+      sg.textColour2 = viewport.textColour2;
+      sg.thresholdTextColour = viewport.thresholdTextColour;
+    }
+
+
+  }
+
+
   /**
    * DOCUMENT ME!
    *
@@ -2089,7 +2204,7 @@ public class AlignFrame
         cs.setConservation(null);
       }
 
-      cs.setConsensus(viewport.vconsensus);
+      cs.setConsensus(viewport.hconsensus);
     }
 
     viewport.setGlobalColourScheme(cs);
@@ -2136,8 +2251,8 @@ public class AlignFrame
                 viewport.getIgnoreGapsConsensus());
 
          sg.cs.setConsensus(AAFrequency.calculate(
-             sg.getSequences(true), 0,
-             sg.getWidth()));
+             sg.getSequences(true), sg.getStartRes(),
+             sg.getEndRes()+1));
        }
         else
           sg.cs.setThreshold(0, viewport.getIgnoreGapsConsensus());
@@ -2147,8 +2262,9 @@ public class AlignFrame
         {
           Conservation c = new Conservation("Group",
                                             ResidueProperties.propHash, 3,
-                                            sg.getSequences(true), 0,
-                                            viewport.alignment.getWidth() - 1);
+                                            sg.getSequences(true),
+                                            sg.getStartRes(),
+                                            sg.getEndRes()+1);
           c.calculate();
           c.verdict(false, viewport.ConsPercGaps);
           sg.cs.setConservation(c);
@@ -2345,10 +2461,10 @@ public class AlignFrame
    */
   public void sortPairwiseMenuItem_actionPerformed(ActionEvent e)
   {
-    addHistoryItem(new HistoryItem("Pairwise Sort", viewport.alignment,
-                                   HistoryItem.SORT));
+    SequenceI [] oldOrder = viewport.getAlignment().getSequencesArray();
     AlignmentSorter.sortByPID(viewport.getAlignment(),
                               viewport.getAlignment().getSequenceAt(0));
+    addHistoryItem(new OrderCommand("Pairwise Sort", oldOrder, viewport.alignment));
     alignPanel.repaint();
   }
 
@@ -2359,9 +2475,9 @@ public class AlignFrame
    */
   public void sortIDMenuItem_actionPerformed(ActionEvent e)
   {
-    addHistoryItem(new HistoryItem("ID Sort", viewport.alignment,
-                                   HistoryItem.SORT));
+    SequenceI [] oldOrder = viewport.getAlignment().getSequencesArray();
     AlignmentSorter.sortByID(viewport.getAlignment());
+    addHistoryItem(new OrderCommand("ID Sort", oldOrder, viewport.alignment));
     alignPanel.repaint();
   }
 
@@ -2372,10 +2488,10 @@ public class AlignFrame
    */
   public void sortGroupMenuItem_actionPerformed(ActionEvent e)
   {
-    addHistoryItem(new HistoryItem("Group Sort", viewport.alignment,
-                                   HistoryItem.SORT));
-
+    SequenceI [] oldOrder = viewport.getAlignment().getSequencesArray();
     AlignmentSorter.sortByGroup(viewport.getAlignment());
+    addHistoryItem(new OrderCommand("Group Sort", oldOrder, viewport.alignment));
+
     alignPanel.repaint();
   }
 
@@ -2573,11 +2689,13 @@ public class AlignFrame
     {
       public void actionPerformed(ActionEvent e)
       {
-        addHistoryItem(new HistoryItem("Sort", viewport.alignment,
-                                       HistoryItem.SORT));
+        SequenceI [] oldOrder = viewport.getAlignment().getSequencesArray();
 
         // TODO: JBPNote - have to map order entries to curent SequenceI pointers
         AlignmentSorter.sortBy(viewport.getAlignment(), order);
+
+        addHistoryItem(new OrderCommand(order.getName(), oldOrder, viewport.alignment));
+
         alignPanel.repaint();
       }
     });
@@ -2608,10 +2726,15 @@ public class AlignFrame
     {
       public void actionPerformed(ActionEvent e)
       {
-        addHistoryItem(new HistoryItem("Tree Sort",
-                                       viewport.alignment, HistoryItem.SORT));
+        SequenceI[] oldOrder = viewport.getAlignment().getSequencesArray();
         AlignmentSorter.sortByTree(viewport.getAlignment(),
                                    treePanel.getTree());
+
+        addHistoryItem(new OrderCommand("Tree Sort",
+                                        oldOrder,
+                                        viewport.alignment));
+
+
         alignPanel.repaint();
       }
     });
@@ -2747,16 +2870,26 @@ public class AlignFrame
   {
     return ShowNewickTree(nf,title,600,500,4,5);
   }
+  public TreePanel ShowNewickTree(NewickFile nf, String title, AlignmentView input)
+  {
+    return ShowNewickTree(nf,title, input, 600,500,4,5);
+  }
+  public TreePanel ShowNewickTree(NewickFile nf, String title, int w,int h,int x, int y) {
+    return ShowNewickTree(nf, title, null, w, h, x, y);
+  }
   /**
-   * DOCUMENT ME!
+   * Add a treeviewer for the tree extracted from a newick file object to the current alignment view
    *
-   * @param nf DOCUMENT ME!
-   * @param title DOCUMENT ME!
-   *
-   * @return DOCUMENT ME!
+   * @param nf the tree
+   * @param title tree viewer title
+   * @param input Associated alignment input data (or null)
+   * @param w width
+   * @param h height
+   * @param x position
+   * @param y position
+   * @return TreePanel handle
    */
-  public TreePanel ShowNewickTree(NewickFile nf, String title, int w,int h,int x, int y)
-  {
+  public TreePanel ShowNewickTree(NewickFile nf, String title, AlignmentView input, int w,int h,int x, int y) {
     TreePanel tp = null;
 
     try
@@ -2768,7 +2901,7 @@ public class AlignFrame
         tp = new TreePanel(alignPanel,
                            "FromFile",
                            title,
-                           nf);
+                           nf, input);
 
         tp.setSize(w,h);
 
@@ -2884,7 +3017,7 @@ public class AlignFrame
               if (msa.getSequences().length == 1)
               {
                 // Single Sequence prediction
-                new jalview.ws.JPredClient(sh, title, false, msa, af);
+                new jalview.ws.JPredClient(sh, title, false, msa, af, true);
               }
               else
               {
@@ -2892,7 +3025,7 @@ public class AlignFrame
                 {
                   // Sequence profile based prediction
                   new jalview.ws.JPredClient(sh,
-                      title, true, msa, af);
+                      title, true, msa, af, true);
                 }
               }
             }
@@ -3185,6 +3318,7 @@ public void drop(DropTargetDropEvent evt)
     {
       alignPanel = (AlignmentPanel) alignPanels.elementAt(index);
       viewport = alignPanel.av;
+      setMenusFromViewport(viewport);
     }
   }