adjustable annotation panel height and vertical scrollbar JAL-516,JAL-338,JAL-306
[jalview.git] / src / jalview / appletgui / AlignFrame.java
index 1b2088e..bad7848 100755 (executable)
@@ -1,24 +1,22 @@
 /*
- * Jalview - A Sequence Alignment Editor and Viewer (Development Version 2.4.1)
- * Copyright (C) 2009 AM Waterhouse, J Procter, G Barton, M Clamp, S Searle
+ * Jalview - A Sequence Alignment Editor and Viewer (Version 2.6)
+ * Copyright (C) 2010 J Procter, AM Waterhouse, G Barton, M Clamp, S Searle
  * 
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License
- * as published by the Free Software Foundation; either version 2
- * of the License, or (at your option) any later version.
+ * This file is part of Jalview.
  * 
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
+ * Jalview is free software: you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License 
+ * as published by the Free Software Foundation, either version 3 of the License, or (at your option) any later version.
  * 
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA
+ * Jalview is distributed in the hope that it will be useful, but 
+ * WITHOUT ANY WARRANTY; without even the implied warranty 
+ * of MERCHANTABILITY or FITNESS FOR A PARTICULAR 
+ * PURPOSE.  See the GNU General Public License for more details.
+ * 
+ * You should have received a copy of the GNU General Public License along with Jalview.  If not, see <http://www.gnu.org/licenses/>.
  */
 package jalview.appletgui;
 
-import java.io.*;
 import java.net.*;
 import java.util.*;
 
@@ -26,6 +24,7 @@ import java.awt.*;
 import java.awt.event.*;
 
 import jalview.analysis.*;
+import jalview.api.SequenceStructureBinding;
 import jalview.bin.JalviewLite;
 import jalview.commands.*;
 import jalview.datamodel.*;
@@ -70,6 +69,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
 
     annotationPanelMenuItem.setState(viewport.showAnnotation);
     displayNonconservedMenuItem.setState(viewport.getShowunconserved());
+    followMouseOverFlag.setState(viewport.getFollowHighlight());
 
     seqLimits.setState(viewport.showJVSuffix);
 
@@ -86,6 +86,10 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
         {
           sortPairwiseMenuItem_actionPerformed();
         }
+        else if (param.equalsIgnoreCase("Length"))
+        {
+          sortLengthMenuItem_actionPerformed();
+        }
       }
 
       param = applet.getParameter("wrap");
@@ -150,7 +154,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
    * DOCUMENT ME!
    * 
    * @param String
-   *                DOCUMENT ME!
+   *          DOCUMENT ME!
    */
 
   public void parseFeaturesFile(String file, String type)
@@ -159,10 +163,10 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
     boolean featuresFile = false;
     try
     {
-      featuresFile = new jalview.io.FeaturesFile(file, type).parse(
-              viewport.alignment, alignPanel.seqPanel.seqCanvas
-                      .getFeatureRenderer().featureColours, featureLinks,
-              true);
+      featuresFile = new jalview.io.FeaturesFile(file, type)
+              .parse(viewport.alignment,
+                      alignPanel.seqPanel.seqCanvas.getFeatureRenderer().featureColours,
+                      featureLinks, true, viewport.applet.getDefaultParameter("relaxedidmatch", false));
     } catch (Exception ex)
     {
       ex.printStackTrace();
@@ -176,7 +180,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
       }
       viewport.showSequenceFeatures = true;
       sequenceFeatures.setState(true);
-      if (viewport.featureSettings!=null)
+      if (viewport.featureSettings != null)
       {
         viewport.featureSettings.refreshTable();
       }
@@ -441,13 +445,14 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
     }
     alignPanel.paintAlignment(true);
   }
+
   /**
    * called by key handler and the hide all/show all menu items
+   * 
    * @param toggleSeqs
    * @param toggleCols
    */
-  private void toggleHiddenRegions(boolean toggleSeqs,
-          boolean toggleCols)
+  private void toggleHiddenRegions(boolean toggleSeqs, boolean toggleCols)
   {
     boolean hide = false;
     SequenceGroup sg = viewport.getSelectionGroup();
@@ -456,8 +461,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
       // Hide everything by the current selection - this is a hack - we do the
       // invert and then hide
       // first check that there will be visible columns after the invert.
-      if ((viewport.colSel != null
-              && viewport.colSel.getSelected() != null && viewport.colSel
+      if ((viewport.colSel != null && viewport.colSel.getSelected() != null && viewport.colSel
               .getSelected().size() > 0)
               || (sg != null && sg.getSize() > 0 && sg.getStartRes() <= sg
                       .getEndRes()))
@@ -508,6 +512,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
       }
     }
   }
+
   public void keyReleased(KeyEvent evt)
   {
   }
@@ -587,7 +592,8 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
     else if (evt.getSource() == this.centreColumnLabelFlag)
     {
       centreColumnLabelFlag_stateChanged();
-    } else if (evt.getSource() == this.followMouseOverFlag)
+    }
+    else if (evt.getSource() == this.followMouseOverFlag)
     {
       mouseOverFlag_stateChanged();
     }
@@ -598,7 +604,8 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
   private void mouseOverFlag_stateChanged()
   {
     viewport.followHighlight = followMouseOverFlag.getState();
-    // TODO: could kick the scrollTo mechanism to reset view for current searchresults.
+    // TODO: could kick the scrollTo mechanism to reset view for current
+    // searchresults.
   }
 
   private void centreColumnLabelFlag_stateChanged()
@@ -761,18 +768,21 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
       viewport.hideAllSelectedSeqs();
       alignPanel.paintAlignment(true);
     }
-    else if (source == hideAllButSelection) {
-      toggleHiddenRegions(false,false);
+    else if (source == hideAllButSelection)
+    {
+      toggleHiddenRegions(false, false);
       alignPanel.paintAlignment(true);
     }
-    else if (source == hideAllSelection) {
+    else if (source == hideAllSelection)
+    {
       SequenceGroup sg = viewport.getSelectionGroup();
       viewport.expandColSelection(sg, false);
       viewport.hideAllSelectedSeqs();
       viewport.hideSelectedColumns();
       alignPanel.paintAlignment(true);
     }
-    else if (source == showAllHidden) {
+    else if (source == showAllHidden)
+    {
       viewport.showAllHiddenColumns();
       viewport.showAllHiddenSeqs();
       alignPanel.paintAlignment(true);
@@ -783,38 +793,8 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
     }
     else if (source == alProperties)
     {
-      StringBuffer contents = new StringBuffer();
-
-      float avg = 0;
-      int min = Integer.MAX_VALUE, max = 0;
-      for (int i = 0; i < viewport.alignment.getHeight(); i++)
-      {
-        int size = viewport.alignment.getSequenceAt(i).getEnd()
-                - viewport.alignment.getSequenceAt(i).getStart();
-        avg += size;
-        if (size > max)
-          max = size;
-        if (size < min)
-          min = size;
-      }
-      avg = avg / (float) viewport.alignment.getHeight();
-
-      contents.append("\nSequences: " + viewport.alignment.getHeight());
-      contents.append("\nMinimum Sequence Length: " + min);
-      contents.append("\nMaximum Sequence Length: " + max);
-      contents.append("\nAverage Length: " + (int) avg);
-
-      if (((Alignment) viewport.alignment).alignmentProperties != null)
-      {
-        Hashtable props = ((Alignment) viewport.alignment).alignmentProperties;
-        Enumeration en = props.keys();
-        while (en.hasMoreElements())
-        {
-          String key = en.nextElement().toString();
-          contents.append("\n" + key + "\t" + props.get(key));
-        }
-      }
-
+      StringBuffer contents = new jalview.io.AlignmentProperties(
+              viewport.alignment).formatAsString();
       CutAndPasteTransfer cap = new CutAndPasteTransfer(false, this);
       cap.setText(contents.toString());
       Frame frame = new Frame();
@@ -833,8 +813,9 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
     else if (source == clustalColour)
     {
       abovePIDThreshold.setState(false);
-      changeColour(new ClustalxColourScheme(viewport.alignment
-              .getSequences(), viewport.alignment.getWidth()));
+      changeColour(new ClustalxColourScheme(
+              viewport.alignment.getSequences(),
+              viewport.alignment.getWidth()));
     }
     else if (source == zappoColour)
     {
@@ -900,6 +881,10 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
     {
       sortIDMenuItem_actionPerformed();
     }
+    else if (source == sortLengthMenuItem)
+    {
+      sortLengthMenuItem_actionPerformed();
+    }
     else if (source == sortGroupMenuItem)
     {
       sortGroupMenuItem_actionPerformed();
@@ -956,10 +941,10 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
     CutAndPasteTransfer cap = new CutAndPasteTransfer(true, this);
     Frame frame = new Frame();
     frame.add(cap);
-    jalview.bin.JalviewLite.addFrame(frame, "Alignment output - "
-            + e.getActionCommand(), 600, 500);
-    cap.setText(new AppletFormatAdapter().formatSequences(e
-            .getActionCommand(), viewport.getAlignment(),
+    jalview.bin.JalviewLite.addFrame(frame,
+            "Alignment output - " + e.getActionCommand(), 600, 500);
+    cap.setText(new AppletFormatAdapter().formatSequences(
+            e.getActionCommand(), viewport.getAlignment(),
             viewport.showJVSuffix));
   }
 
@@ -996,14 +981,15 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
 
   private Hashtable getDisplayedFeatureCols()
   {
-    if (alignPanel.getFeatureRenderer()!=null) {
+    if (alignPanel.getFeatureRenderer() != null)
+    {
       FeatureRenderer fr = alignPanel.getFeatureRenderer();
       Hashtable fcols = new Hashtable();
       Enumeration en = viewport.featuresDisplayed.keys();
       while (en.hasMoreElements())
       {
         Object col = en.nextElement();
-        fcols.put(col,fr.featureColours.get(col));
+        fcols.put(col, fr.featureColours.get(col));
       }
       return fcols;
     }
@@ -1015,13 +1001,15 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
     String features;
     if (format.equalsIgnoreCase("Jalview"))
     {
-      features = new FeaturesFile().printJalviewFormat(viewport.alignment
-              .getSequencesArray(), getDisplayedFeatureCols());
+      features = new FeaturesFile().printJalviewFormat(
+              viewport.alignment.getSequencesArray(),
+              getDisplayedFeatureCols());
     }
     else
     {
-      features = new FeaturesFile().printGFFFormat(viewport.alignment
-              .getSequencesArray(), getDisplayedFeatureCols());
+      features = new FeaturesFile().printGFFFormat(
+              viewport.alignment.getSequencesArray(),
+              getDisplayedFeatureCols());
     }
 
     if (displayTextbox)
@@ -1053,9 +1041,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
     if (viewport.applet.getParameter("annotations") != null)
     {
       url.append("&annotations=");
-      url
-              .append(appendProtocol(viewport.applet
-                      .getParameter("annotations")));
+      url.append(appendProtocol(viewport.applet.getParameter("annotations")));
     }
 
     if (viewport.applet.getParameter("jnetfile") != null)
@@ -1191,7 +1177,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
    * DOCUMENT ME!
    * 
    * @param e
-   *                DOCUMENT ME!
+   *          DOCUMENT ME!
    */
   protected void undoMenuItem_actionPerformed()
   {
@@ -1216,7 +1202,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
    * DOCUMENT ME!
    * 
    * @param e
-   *                DOCUMENT ME!
+   *          DOCUMENT ME!
    */
   protected void redoMenuItem_actionPerformed()
   {
@@ -1594,8 +1580,8 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
 
     viewport.setEndSeq(viewport.alignment.getHeight());
     viewport.alignment.getWidth();
-    viewport.firePropertyChange("alignment", null, viewport.alignment
-            .getSequences());
+    viewport.firePropertyChange("alignment", null,
+            viewport.alignment.getSequences());
 
   }
 
@@ -1652,14 +1638,17 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
     {
       this.setVisible(false);
     }
+    viewport.sendSelection();
   }
+
   protected void makeGrpsFromSelection_actionPerformed()
   {
     if (viewport.getSelectionGroup() != null)
     {
       SequenceGroup[] gps = jalview.analysis.Grouping.makeGroupsFrom(
-              viewport.getSequenceSelection(), viewport.getAlignmentView(
-                      true).getSequenceStrings(viewport.getGapCharacter()),
+              viewport.getSequenceSelection(),
+              viewport.getAlignmentView(true).getSequenceStrings(
+                      viewport.getGapCharacter()),
               viewport.alignment.getGroups());
       viewport.alignment.deleteAllGroups();
       viewport.sequenceColours = null;
@@ -1668,10 +1657,10 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
       for (int g = 0; g < gps.length; g++)
       {
         // gps[g].setShowunconserved(viewport.getShowUnconserved());
-        gps[g].setIncludeAllConsSymbols(viewport.isIncludeAllConsensusSymbols());
+        gps[g].setshowSequenceLogo(viewport.isShowSequenceLogo());
         viewport.alignment.addGroup(gps[g]);
-        Color col = new Color((int) (Math.random() * 255), (int) (Math
-                .random() * 255), (int) (Math.random() * 255));
+        Color col = new Color((int) (Math.random() * 255),
+                (int) (Math.random() * 255), (int) (Math.random() * 255));
         col = col.brighter();
         for (Enumeration sq = gps[g].getSequences(null).elements(); sq
                 .hasMoreElements(); viewport.setSequenceColour(
@@ -1683,6 +1672,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
       alignPanel.paintAlignment(true);
     }
   }
+
   protected void deleteGroups_actionPerformed()
   {
     viewport.alignment.deleteAllGroups();
@@ -1703,6 +1693,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
     viewport.setSelectionGroup(sg);
     alignPanel.paintAlignment(true);
     PaintRefresher.Refresh(alignPanel, viewport.getSequenceSetId());
+    viewport.sendSelection();
   }
 
   public void deselectAllSequenceMenuItem_actionPerformed()
@@ -1719,6 +1710,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
     alignPanel.seqPanel.seqCanvas.highlightSearchResults(null);
     alignPanel.paintAlignment(true);
     PaintRefresher.Refresh(alignPanel, viewport.getSequenceSetId());
+    viewport.sendSelection();
   }
 
   public void invertSequenceMenuItem_actionPerformed()
@@ -1730,12 +1722,15 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
     }
 
     PaintRefresher.Refresh(alignPanel, viewport.getSequenceSetId());
+    viewport.sendSelection();
   }
+
   public void invertColSel_actionPerformed()
   {
     viewport.invertColumnSelection();
     alignPanel.paintAlignment(true);
     PaintRefresher.Refresh(alignPanel, viewport.getSequenceSetId());
+    viewport.sendSelection();
   }
 
   void trimAlignment(boolean trimLeft)
@@ -1906,9 +1901,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
       {
         if (!viewport.alignment.getAlignmentAnnotation()[i].autoCalculated)
         {
-          newal
-                  .addAnnotation(viewport.alignment
-                          .getAlignmentAnnotation()[i]);
+          newal.addAnnotation(viewport.alignment.getAlignmentAnnotation()[i]);
         }
       }
     }
@@ -1917,8 +1910,8 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
 
     newaf.viewport.sequenceSetID = alignPanel.av.getSequenceSetId();
     PaintRefresher.Register(alignPanel, alignPanel.av.getSequenceSetId());
-    PaintRefresher.Register(newaf.alignPanel, newaf.alignPanel.av
-            .getSequenceSetId());
+    PaintRefresher.Register(newaf.alignPanel,
+            newaf.alignPanel.av.getSequenceSetId());
 
     PaintRefresher.Register(newaf.alignPanel.idPanel.idCanvas,
             newaf.alignPanel.av.getSequenceSetId());
@@ -1976,7 +1969,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
    * get sequence feature groups that are hidden or shown
    * 
    * @param visible
-   *                true is visible
+   *          true is visible
    * @return list
    */
   public String[] getFeatureGroupsOfState(boolean visible)
@@ -1994,9 +1987,9 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
    * Change the display state for the given feature groups
    * 
    * @param groups
-   *                list of group strings
+   *          list of group strings
    * @param state
-   *                visible or invisible
+   *          visible or invisible
    */
   public void setFeatureGroupState(String[] groups, boolean state)
   {
@@ -2097,8 +2090,8 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
 
         Alignment al = (Alignment) viewport.alignment;
         Conservation c = new Conservation("All",
-                ResidueProperties.propHash, 3, al.getSequences(), 0, al
-                        .getWidth() - 1);
+                ResidueProperties.propHash, 3, al.getSequences(), 0,
+                al.getWidth() - 1);
 
         c.calculate();
         c.verdict(false, viewport.ConsPercGaps);
@@ -2133,8 +2126,9 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
         }
         if (cs instanceof ClustalxColourScheme)
         {
-          sg.cs = new ClustalxColourScheme(sg
-                  .getSequences(viewport.hiddenRepSequences), sg.getWidth());
+          sg.cs = new ClustalxColourScheme(
+                  sg.getSequences(viewport.hiddenRepSequences),
+                  sg.getWidth());
         }
         else
         {
@@ -2153,9 +2147,9 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
                 || cs instanceof Blosum62ColourScheme)
         {
           sg.cs.setThreshold(threshold, viewport.getIgnoreGapsConsensus());
-          sg.cs.setConsensus(AAFrequency.calculate(sg
-                  .getSequences(viewport.hiddenRepSequences), 0, sg
-                  .getWidth()));
+          sg.cs.setConsensus(AAFrequency.calculate(
+                  sg.getSequences(viewport.hiddenRepSequences), 0,
+                  sg.getWidth()));
         }
         else
         {
@@ -2165,8 +2159,8 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
         if (viewport.getConservationSelected())
         {
           Conservation c = new Conservation("Group",
-                  ResidueProperties.propHash, 3, sg
-                          .getSequences(viewport.hiddenRepSequences), 0,
+                  ResidueProperties.propHash, 3,
+                  sg.getSequences(viewport.hiddenRepSequences), 0,
                   viewport.alignment.getWidth() - 1);
           c.calculate();
           c.verdict(false, viewport.ConsPercGaps);
@@ -2198,8 +2192,8 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
     if (viewport.getAbovePIDThreshold()
             && viewport.globalColourScheme != null)
     {
-      SliderPanel.setPIDSliderSource(alignPanel, viewport
-              .getGlobalColourScheme(), "Background");
+      SliderPanel.setPIDSliderSource(alignPanel,
+              viewport.getGlobalColourScheme(), "Background");
       SliderPanel.showPIDSlider();
     }
   }
@@ -2238,6 +2232,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
 
     modifyPID_actionPerformed();
   }
+
   public void sortPairwiseMenuItem_actionPerformed()
   {
     SequenceI[] oldOrder = viewport.getAlignment().getSequencesArray();
@@ -2253,7 +2248,15 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
   {
     SequenceI[] oldOrder = viewport.getAlignment().getSequencesArray();
     AlignmentSorter.sortByID(viewport.getAlignment());
-    addHistoryItem(new OrderCommand("ID Sort", oldOrder,
+    addHistoryItem(new OrderCommand("ID Sort", oldOrder, viewport.alignment));
+    alignPanel.paintAlignment(true);
+  }
+
+  public void sortLengthMenuItem_actionPerformed()
+  {
+    SequenceI[] oldOrder = viewport.getAlignment().getSequencesArray();
+    AlignmentSorter.sortByLength(viewport.getAlignment());
+    addHistoryItem(new OrderCommand("Length Sort", oldOrder,
             viewport.alignment));
     alignPanel.paintAlignment(true);
   }
@@ -2392,28 +2395,36 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
     jalview.bin.JalviewLite.addFrame(tp, treeFile, 600, 500);
     addTreeMenuItem(tp, treeFile);
   }
+
   /**
    * sort the alignment using the given treePanel
-   * @param treePanel tree used to sort view
-   * @param title string used for undo event name
+   * 
+   * @param treePanel
+   *          tree used to sort view
+   * @param title
+   *          string used for undo event name
    */
   public void sortByTree(TreePanel treePanel, String title)
   {
     SequenceI[] oldOrder = viewport.getAlignment().getSequencesArray();
-    AlignmentSorter.sortByTree(viewport.getAlignment(), treePanel
-            .getTree());
+    AlignmentSorter
+            .sortByTree(viewport.getAlignment(), treePanel.getTree());
     // addHistoryItem(new HistoryItem("Sort", viewport.alignment,
     // HistoryItem.SORT));
-    addHistoryItem(new OrderCommand("Order by "+title, oldOrder,
+    addHistoryItem(new OrderCommand("Order by " + title, oldOrder,
             viewport.alignment));
     alignPanel.paintAlignment(true);
   }
+
   /**
-   * Do any automatic reordering of the alignment and add the necessary bits to the menu structure for the new tree
+   * Do any automatic reordering of the alignment and add the necessary bits to
+   * the menu structure for the new tree
+   * 
    * @param treePanel
    * @param title
    */
-  protected void addTreeMenuItem(final TreePanel treePanel, final String title)
+  protected void addTreeMenuItem(final TreePanel treePanel,
+          final String title)
   {
     final MenuItem item = new MenuItem(title);
     sortByTreeMenu.add(item);
@@ -2442,7 +2453,22 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
       };
     });
   }
-
+  public boolean sortBy(AlignmentOrder alorder, String undoname)
+  {
+    SequenceI[] oldOrder = viewport.getAlignment()
+    .getSequencesArray();
+    if (viewport.applet.debug)
+    {
+      System.err.println("Sorting "+alorder.getOrder().size()+" in alignment '"+getTitle()+"'");
+    }
+    AlignmentSorter.sortBy(viewport.getAlignment(), alorder);
+    if (undoname!=null)
+    {
+      addHistoryItem(new OrderCommand(undoname, oldOrder, viewport.alignment));
+    }
+    alignPanel.paintAlignment(true);
+    return true;
+  }
   protected void documentation_actionPerformed()
   {
     showURL("http://www.jalview.org/help.html", "HELP");
@@ -2454,7 +2480,9 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
     class AboutPanel extends Canvas
     {
       String version;
+
       String builddate;
+
       public AboutPanel(String version, String builddate)
       {
         this.version = version;
@@ -2475,31 +2503,33 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
         g.setFont(new Font("Helvetica", Font.BOLD, 14));
         g.drawString("JalviewLite - Release " + version, x, y += fh);
         g.setFont(new Font("Helvetica", Font.BOLD, 12));
-        g.drawString("Build date: "+builddate, x, y += fh);
+        g.drawString("Build date: " + builddate, x, y += fh);
         g.setFont(new Font("Helvetica", Font.PLAIN, 12));
-        g
-                .drawString(
-                        "Authors:  Andrew Waterhouse, Jim Procter, Michele Clamp, James Cuff, Steve Searle,",
-                        x, y += fh * 1.5);
+        g.drawString(
+                "Authors:  Jim Procter, Andrew Waterhouse, Michele Clamp, James Cuff, Steve Searle,",
+                x, y += fh * 1.5);
         g.drawString("David Martin & Geoff Barton.", x + 50, y += fh);
-        g
-                .drawString(
-                        "Development managed by The Barton Group, University of Dundee, Scotland, UK.",
-                        x, y += fh);
-        g
-                .drawString(
-                        "For help, see the FAQ at www.jalview.org and/or join the jalview-discuss@jalview.org mailing list",
-                        x, y += fh);
+        g.drawString(
+                "Development managed by The Barton Group, University of Dundee, Scotland, UK.",
+                x, y += fh);
+        g.drawString(
+                "For help, see the FAQ at www.jalview.org and/or join the jalview-discuss@jalview.org mailing list",
+                x, y += fh);
         g.drawString("If  you use Jalview, please cite:", x, y += fh + 8);
         g.drawString(
-                        "Waterhouse, A.M., Procter, J.B., Martin, D.M.A, Clamp, M. and Barton, G. J. (2009)",x,y+=fh);
-        g.drawString("Jalview Version 2 - a multiple sequence alignment editor and analysis workbench",x,y+=fh);
-        g.drawString("Bioinformatics doi: 10.1093/bioinformatics/btp033",x,y+=fh);
+                "Waterhouse, A.M., Procter, J.B., Martin, D.M.A, Clamp, M. and Barton, G. J. (2009)",
+                x, y += fh);
+        g.drawString(
+                "Jalview Version 2 - a multiple sequence alignment editor and analysis workbench",
+                x, y += fh);
+        g.drawString("Bioinformatics doi: 10.1093/bioinformatics/btp033",
+                x, y += fh);
       }
     }
 
     Frame frame = new Frame();
-    frame.add(new AboutPanel(JalviewLite.getVersion(), JalviewLite.getBuildDate()));
+    frame.add(new AboutPanel(JalviewLite.getVersion(), JalviewLite
+            .getBuildDate()));
     jalview.bin.JalviewLite.addFrame(frame, "Jalview", 580, 220);
 
   }
@@ -2514,22 +2544,38 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
     {
       try
       {
-        if (url.indexOf(":")==-1)
+        if (url.indexOf(":") == -1)
         {
-          // TODO: verify (Bas Vroling bug) prepend codebase or server URL to form valid URL
-          if (url.indexOf("/")==0)
+          // TODO: verify (Bas Vroling bug) prepend codebase or server URL to
+          // form valid URL
+          if (url.indexOf("/") == 0)
           {
             String codebase = viewport.applet.getCodeBase().toString();
-            url = codebase.substring(0,codebase.length()-viewport.applet.getCodeBase().getFile().length())+url;
-          } else {
-            url = viewport.applet.getCodeBase()+url;
+            url = codebase.substring(0, codebase.length()
+                    - viewport.applet.getCodeBase().getFile().length())
+                    + url;
+          }
+          else
+          {
+            url = viewport.applet.getCodeBase() + url;
           }
           System.out.println("Show url (prepended codebase): " + url);
-        } else {
+        }
+        else
+        {
           System.out.println("Show url: " + url);
         }
-        viewport.applet.getAppletContext().showDocument(
-                new java.net.URL(url), target);
+        if (url.indexOf("javascript:") == 0)
+        {
+          // no target for the javascript context
+          viewport.applet.getAppletContext().showDocument(
+                  new java.net.URL(url));
+        }
+        else
+        {
+          viewport.applet.getAppletContext().showDocument(
+                  new java.net.URL(url), target);
+        }
       } catch (Exception ex)
       {
         ex.printStackTrace();
@@ -2586,6 +2632,8 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
 
   MenuItem sortIDMenuItem = new MenuItem();
 
+  MenuItem sortLengthMenuItem = new MenuItem();
+
   MenuItem sortGroupMenuItem = new MenuItem();
 
   MenuItem removeRedundancyMenuItem = new MenuItem();
@@ -2660,7 +2708,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
 
   MenuItem deleteGroups = new MenuItem();
 
-  MenuItem grpsFromSelection= new MenuItem();
+  MenuItem grpsFromSelection = new MenuItem();
 
   MenuItem delete = new MenuItem();
 
@@ -2763,6 +2811,8 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
     sortPairwiseMenuItem.addActionListener(this);
     sortIDMenuItem.setLabel("by ID");
     sortIDMenuItem.addActionListener(this);
+    sortLengthMenuItem.setLabel("by Length");
+    sortLengthMenuItem.addActionListener(this);
     sortGroupMenuItem.setLabel("by Group");
     sortGroupMenuItem.addActionListener(this);
     removeRedundancyMenuItem.setLabel("Remove Redundancy...");
@@ -2812,7 +2862,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
     annotationPanelMenuItem.addItemListener(this);
     colourTextMenuItem.setLabel("Colour Text");
     colourTextMenuItem.addItemListener(this);
-    displayNonconservedMenuItem.setLabel("Show non-conserved");
+    displayNonconservedMenuItem.setLabel("Show nonconserved");
     displayNonconservedMenuItem.addItemListener(this);
     alProperties.addActionListener(this);
     overviewMenuItem.setLabel("Overview Window");
@@ -3006,6 +3056,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
     pasteMenu.add(pasteNew);
     pasteMenu.add(pasteThis);
     sort.add(sortIDMenuItem);
+    sort.add(sortLengthMenuItem);
     sort.add(sortByTreeMenu);
     sort.add(sortGroupMenuItem);
     sort.add(sortPairwiseMenuItem);
@@ -3042,7 +3093,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
     selectMenu.add(invertColSel);
     selectMenu.add(grpsFromSelection);
     selectMenu.add(deleteGroups);
-    
+
   }
 
   MenuItem featureSettings = new MenuItem();
@@ -3066,7 +3117,9 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
   MenuItem hideSequences = new MenuItem();
 
   MenuItem hideAllButSelection = new MenuItem();
+
   MenuItem hideAllSelection = new MenuItem();
+
   MenuItem showAllHidden = new MenuItem();
 
   Menu formatMenu = new Menu();
@@ -3081,8 +3134,8 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
    * platform independent manner to avoid OSX/Mac menu appendage daftness.
    * 
    * @param reallyEmbedded
-   *                true to attach the view to the applet area on the page
-   *                rather than in a new window
+   *          true to attach the view to the applet area on the page rather than
+   *          in a new window
    */
   public void createAlignFrameWindow(boolean reallyEmbedded, String title)
   {
@@ -3100,10 +3153,27 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
       viewport.applet.setLayout(new BorderLayout());
       viewport.applet.add(embeddedMenu, BorderLayout.NORTH);
       viewport.applet.add(statusBar, BorderLayout.SOUTH);
-      alignPanel.setSize(viewport.applet.getSize().width, viewport.applet
-              .getSize().height
-              - embeddedMenu.HEIGHT - statusBar.HEIGHT);
+      alignPanel.setSize(viewport.applet.getSize().width,
+              viewport.applet.getSize().height - embeddedMenu.HEIGHT
+                      - statusBar.HEIGHT);
       viewport.applet.add(alignPanel, BorderLayout.CENTER);
+      final AlignFrame me = this;
+      viewport.applet.addFocusListener(new FocusListener()
+      {
+        
+        @Override
+        public void focusLost(FocusEvent e)
+        {
+          if (me.viewport.applet.currentAlignFrame==me) {
+                  me.viewport.applet.currentAlignFrame = null;
+        }}
+        
+        @Override
+        public void focusGained(FocusEvent e)
+        {
+          me.viewport.applet.currentAlignFrame = me;
+        }
+      });
       viewport.applet.validate();
     }
     else
@@ -3124,4 +3194,236 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,
               DEFAULT_HEIGHT);
     }
   }
+
+  /**
+   * create a new binding between structures in an existing jmol viewer instance
+   * and an alignpanel with sequences that have existing PDBFile entries. Note,
+   * this does not open a new Jmol window, or modify the display of the
+   * structures in the original jmol window. Note This method doesn't work
+   * without an additional javascript library to exchange messages between the
+   * distinct applets. See http://issues.jalview.org/browse/JAL-621
+   * 
+   * @param viewer
+   *          JmolViewer instance
+   * @param sequenceIds
+   *          - sequence Ids to search for associations
+   */
+  public SequenceStructureBinding addStructureViewInstance(
+          Object jmolviewer, String[] sequenceIds)
+  {
+    org.jmol.api.JmolViewer viewer = null;
+    try
+    {
+      viewer = (org.jmol.api.JmolViewer) jmolviewer;
+    } catch (ClassCastException ex)
+    {
+      System.err.println("Unsupported viewer object :"
+              + jmolviewer.getClass());
+    }
+    if (viewer == null)
+    {
+      System.err.println("Can't use this object as a structure viewer:"
+              + jmolviewer.getClass());
+      return null;
+    }
+    SequenceI[] seqs = null;
+    if (sequenceIds == null || sequenceIds.length == 0)
+    {
+      seqs = viewport.getAlignment().getSequencesArray();
+    }
+    else
+    {
+      Vector sqi = new Vector();
+      AlignmentI al = viewport.getAlignment();
+      for (int sid = 0; sid < sequenceIds.length; sid++)
+      {
+        SequenceI sq = al.findName(sequenceIds[sid]);
+        if (sq != null)
+        {
+          sqi.addElement(sq);
+        }
+      }
+      if (sqi.size() > 0)
+      {
+        seqs = new SequenceI[sqi.size()];
+        for (int sid = 0, sSize = sqi.size(); sid < sSize; sid++)
+        {
+          seqs[sid] = (SequenceI) sqi.elementAt(sid);
+        }
+      }
+      else
+      {
+        return null;
+      }
+    }
+    ExtJmol jmv = null;
+    // TODO: search for a jmv that involves viewer
+    if (jmv == null)
+    { // create a new viewer/jalview binding.
+      jmv = new ExtJmol(viewer, alignPanel, new SequenceI[][] {seqs});
+    }
+    return jmv;
+
+  }
+
+  public boolean addPdbFile(String sequenceId, String pdbEntryString,
+          String pdbFile)
+  {
+    SequenceI toaddpdb = viewport.getAlignment().findName(sequenceId);
+    boolean needtoadd = false;
+    if (toaddpdb != null)
+    {
+      Vector pdbe = toaddpdb.getPDBId();
+      PDBEntry pdbentry = null;
+      if (pdbe != null && pdbe.size() > 0)
+      {
+        for (int pe = 0, peSize = pdbe.size(); pe < peSize; pe++)
+        {
+          pdbentry = (PDBEntry) pdbe.elementAt(pe);
+          if (!pdbentry.getId().equals(pdbEntryString)
+                  && !pdbentry.getFile().equals(pdbFile))
+          {
+            pdbentry = null;
+          }
+          else
+          {
+            continue;
+          }
+        }
+      }
+      if (pdbentry == null)
+      {
+        pdbentry = new PDBEntry();
+        pdbentry.setId(pdbEntryString);
+        pdbentry.setFile(pdbFile);
+        needtoadd = true; // add this new entry to sequence.
+      }
+      // resolve data source
+      // TODO: this code should be a refactored to an io package
+      String protocol = AppletFormatAdapter.resolveProtocol(pdbFile, "PDB");
+      if (protocol == null)
+      {
+        return false;
+      }
+      if (needtoadd)
+      {
+        // make a note of the access mode and add
+        if (pdbentry.getProperty() == null)
+        {
+          pdbentry.setProperty(new Hashtable());
+        }
+        pdbentry.getProperty().put("protocol", protocol);
+        toaddpdb.addPDBId(pdbentry);
+      }
+    }
+    return true;
+  }
+
+  private Object[] cleanSeqChainArrays(SequenceI[] seqs, String[] chains)
+  {
+    if (seqs != null)
+    {
+      Vector sequences = new Vector();
+      for (int i = 0; i < seqs.length; i++)
+      {
+        if (seqs[i] != null)
+        {
+          sequences.addElement(new Object[]
+          { seqs[i], (chains != null) ? chains[i] : null });
+        }
+      }
+      seqs = new SequenceI[sequences.size()];
+      chains = new String[sequences.size()];
+      for (int i = 0, isize = sequences.size(); i < isize; i++)
+      {
+        Object[] oj = (Object[]) sequences.elementAt(i);
+
+        seqs[i] = (SequenceI) oj[0];
+        chains[i] = (String) oj[1];
+      }
+    }
+    return new Object[]
+    { seqs, chains };
+
+  }
+
+  public void newStructureView(JalviewLite applet, PDBEntry pdb,
+          SequenceI[] seqs, String[] chains, String protocol)
+  {
+    // Scrub any null sequences from the array
+    Object[] sqch = cleanSeqChainArrays(seqs, chains);
+    seqs = (SequenceI[]) sqch[0];
+    chains = (String[]) sqch[1];
+    if (seqs == null || seqs.length == 0)
+    {
+      System.err
+              .println("JalviewLite.AlignFrame:newStructureView: No sequence to bind structure to.");
+    }
+    if (protocol == null || protocol.trim().length() == 0
+            || protocol.equals("null"))
+    {
+      protocol = (String) pdb.getProperty().get("protocol");
+      if (protocol == null)
+      {
+        System.err.println("Couldn't work out protocol to open structure: "
+                + pdb.getId());
+        return;
+      }
+    }
+    if (applet.isAlignPdbStructures() && applet.jmolAvailable)
+    {
+      // can only do alignments with Jmol
+      // find the last jmol window assigned to this alignment
+      jalview.appletgui.AppletJmol ajm = null, tajm;
+      Vector jmols = applet
+              .getAppletWindow(jalview.appletgui.AppletJmol.class);
+      for (int i = 0, iSize = jmols.size(); i < iSize; i++)
+      {
+        tajm = (jalview.appletgui.AppletJmol) jmols.elementAt(i);
+        if (tajm.ap.alignFrame == this)
+        {
+          ajm = tajm;
+          break;
+        }
+      }
+      if (ajm != null)
+      {
+        System.err
+                .println("Incremental adding and aligning structure to existing Jmol view not yet implemented.");
+        // try and add the pdb structure
+        // ajm.addS
+        ajm = null;
+      }
+    }
+    // otherwise, create a new window
+    if (applet.jmolAvailable)
+    {
+      new jalview.appletgui.AppletJmol(pdb, seqs, chains, alignPanel,
+              protocol);
+      applet.lastFrameX += 40;
+      applet.lastFrameY += 40;
+    }
+    else
+    {
+      new MCview.AppletPDBViewer(pdb, seqs, chains, alignPanel, protocol);
+    }
+
+  }
+
+  public void alignedStructureView(JalviewLite applet, PDBEntry[] pdb,
+          SequenceI[][] seqs, String[][] chains, String[] protocols)
+  {
+    // TODO Auto-generated method stub
+    System.err.println("Aligned Structure View: Not yet implemented.");
+  }
+
+  /**
+   * modify the current selection, providing the user has not made a selection already.
+   * @param sel - sequences from this alignment 
+   * @param csel - columns to be selected on the alignment
+   */
+  public void select(SequenceGroup sel, ColumnSelection csel)
+  {
+    alignPanel.seqPanel.selection(sel, csel, null);
+  }
 }