JAL-1031 allow empty selections to pass through to tree builder, which will ignore...
[jalview.git] / src / jalview / appletgui / AlignFrame.java
index 276fbda..bcb518f 100644 (file)
@@ -1,18 +1,18 @@
 /*\r
  * Jalview - A Sequence Alignment Editor and Viewer (Version 2.7)\r
  * Copyright (C) 2011 J Procter, AM Waterhouse, J Engelhardt, LM Lui, G Barton, M Clamp, S Searle\r
- * \r
+ *\r
  * This file is part of Jalview.\r
- * \r
+ *\r
  * Jalview is free software: you can redistribute it and/or\r
- * modify it under the terms of the GNU General Public License \r
+ * modify it under the terms of the GNU General Public License\r
  * as published by the Free Software Foundation, either version 3 of the License, or (at your option) any later version.\r
- * \r
- * Jalview is distributed in the hope that it will be useful, but \r
- * WITHOUT ANY WARRANTY; without even the implied warranty \r
- * of MERCHANTABILITY or FITNESS FOR A PARTICULAR \r
+ *\r
+ * Jalview is distributed in the hope that it will be useful, but\r
+ * WITHOUT ANY WARRANTY; without even the implied warranty\r
+ * of MERCHANTABILITY or FITNESS FOR A PARTICULAR\r
  * PURPOSE.  See the GNU General Public License for more details.\r
- * \r
+ *\r
  * You should have received a copy of the GNU General Public License along with Jalview.  If not, see <http://www.gnu.org/licenses/>.\r
  */\r
 package jalview.appletgui;\r
@@ -35,6 +35,7 @@ import jalview.datamodel.AlignmentOrder;
 import jalview.datamodel.ColumnSelection;\r
 import jalview.datamodel.PDBEntry;\r
 import jalview.datamodel.Sequence;\r
+import jalview.datamodel.SequenceCollectionI;\r
 import jalview.datamodel.SequenceGroup;\r
 import jalview.datamodel.SequenceI;\r
 import jalview.io.AnnotationFile;\r
@@ -87,6 +88,7 @@ import java.net.URL;
 import java.net.URLEncoder;\r
 import java.util.Enumeration;\r
 import java.util.Hashtable;\r
+import java.util.List;\r
 import java.util.StringTokenizer;\r
 import java.util.Vector;\r
 \r
@@ -101,7 +103,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
   int DEFAULT_HEIGHT = 500;\r
 \r
   String jalviewServletURL;\r
-  \r
+\r
 \r
   public AlignFrame(AlignmentI al, jalview.bin.JalviewLite applet, String title, boolean embedded)\r
   {\r
@@ -131,6 +133,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
     showGroupConservation.setState(viewport.isShowGroupConservation());\r
     showConsensusHistogram.setState(viewport.isShowConsensusHistogram());\r
     showSequenceLogo.setState(viewport.isShowSequenceLogo());\r
+    normSequenceLogo.setState(viewport.isNormaliseSequenceLogo());\r
 \r
     seqLimits.setState(viewport.showJVSuffix);\r
 \r
@@ -231,25 +234,28 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
 \r
   /**\r
    * Load a features file onto the alignment\r
-   * \r
+   *\r
    * @param file file URL, content, or other resolvable path\r
    * @param type is protocol for accessing data referred to by file\r
    */\r
 \r
-  public void parseFeaturesFile(String file, String type)\r
+  public boolean parseFeaturesFile(String file, String type)\r
   {\r
-    parseFeaturesFile(file, type, true);\r
+    return parseFeaturesFile(file, type, true);\r
   }\r
-  \r
+\r
   /**\r
    * Load a features file onto the alignment\r
-   * \r
+   *\r
    * @param file file URL, content, or other resolvable path\r
    * @param type is protocol for accessing data referred to by file\r
    * @param autoenabledisplay when true, display features flag will be automatically enabled if features are loaded\r
+   * @return true if data parsed as a features file\r
    */\r
-  public void parseFeaturesFile(String file, String type, boolean autoenabledisplay)\r
-  {    \r
+  public boolean parseFeaturesFile(String file, String type, boolean autoenabledisplay)\r
+  {\r
+    // TODO: test if importing a features file onto an alignment which already has features with links overwrites the original links.\r
+\r
     Hashtable featureLinks = new Hashtable();\r
     boolean featuresFile = false;\r
     try\r
@@ -279,10 +285,12 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
         viewport.featureSettings.refreshTable();\r
       }\r
       alignPanel.paintAlignment(true);\r
+      statusBar.setText("Successfully added features to alignment.");\r
     }\r
-\r
+    return featuresFile;\r
   }\r
 \r
+  @Override\r
   public void keyPressed(KeyEvent evt)\r
   {\r
     if (viewport.cursorMode\r
@@ -296,8 +304,8 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
     {\r
     case 27: // escape key\r
       deselectAllSequenceMenuItem_actionPerformed();\r
-      \r
-      alignPanel.alabels.cancelDrag(); \r
+\r
+      alignPanel.alabels.cancelDrag();\r
       break;\r
     case KeyEvent.VK_X:\r
       if (evt.isControlDown() || evt.isMetaDown())\r
@@ -544,7 +552,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
 \r
   /**\r
    * called by key handler and the hide all/show all menu items\r
-   * \r
+   *\r
    * @param toggleSeqs\r
    * @param toggleCols\r
    */\r
@@ -609,14 +617,17 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
     }\r
   }\r
 \r
+  @Override\r
   public void keyReleased(KeyEvent evt)\r
   {\r
   }\r
 \r
+  @Override\r
   public void keyTyped(KeyEvent evt)\r
   {\r
   }\r
 \r
+  @Override\r
   public void itemStateChanged(ItemEvent evt)\r
   {\r
     if (evt.getSource() == displayNonconservedMenuItem)\r
@@ -709,6 +720,10 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
     {\r
       showSequenceLogo_actionPerformed();\r
     }\r
+    else if (evt.getSource() == normSequenceLogo)\r
+    {\r
+      normSequenceLogo_actionPerformed();\r
+    }\r
     else if (evt.getSource() == showConsensusHistogram)\r
     {\r
       showConsensusHistogram_actionPerformed();\r
@@ -733,6 +748,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
     this.alignPanel.annotationPanel.repaint();\r
   }\r
 \r
+  @Override\r
   public void actionPerformed(ActionEvent evt)\r
   {\r
     Object source = evt.getSource();\r
@@ -753,9 +769,6 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
     {\r
       loadAnnotations();\r
     }\r
-    else if( source == loadScores ) {\r
-       loadScores();\r
-    }\r
     else if (source == outputAnnotations)\r
     {\r
       outputAnnotations(true);\r
@@ -921,6 +934,10 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
     {\r
       showSequenceLogo_actionPerformed();\r
     }\r
+    else if (source == normSequenceLogo)\r
+    {\r
+      normSequenceLogo_actionPerformed();\r
+    }\r
     else if (source == showConsensusHistogram)\r
     {\r
       showConsensusHistogram_actionPerformed();\r
@@ -955,9 +972,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
     else if (source == clustalColour)\r
     {\r
       abovePIDThreshold.setState(false);\r
-      changeColour(new ClustalxColourScheme(\r
-              viewport.getAlignment().getSequences(),\r
-              viewport.getAlignment().getWidth()));\r
+      changeColour(new ClustalxColourScheme(viewport.getAlignment(),null));\r
     }\r
     else if (source == zappoColour)\r
     {\r
@@ -1104,18 +1119,13 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
   public void loadAnnotations()\r
   {\r
     CutAndPasteTransfer cap = new CutAndPasteTransfer(true, this);\r
-    cap.setText("Paste your features / annotations file here.");\r
+    cap.setText("Paste your features / annotations / T-coffee score file here.");\r
     cap.setAnnotationImport();\r
     Frame frame = new Frame();\r
     frame.add(cap);\r
     jalview.bin.JalviewLite.addFrame(frame, "Paste Annotations ", 400, 300);\r
 \r
   }\r
-  \r
-  public void loadScores() {\r
-         //TODO\r
-         \r
-  }\r
 \r
   public String outputAnnotations(boolean displayTextbox)\r
   {\r
@@ -1178,7 +1188,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
         features = "# No features visible - paste some and import them here.";\r
         frimport=true;\r
       }\r
-      \r
+\r
       CutAndPasteTransfer cap = new CutAndPasteTransfer(frimport, this);\r
       if (frimport)\r
       {\r
@@ -1290,8 +1300,13 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
   public void closeMenuItem_actionPerformed()\r
   {\r
     PaintRefresher.RemoveComponent(alignPanel);\r
-    PaintRefresher.RemoveComponent(alignPanel.seqPanel.seqCanvas);\r
-    PaintRefresher.RemoveComponent(alignPanel.idPanel.idCanvas);\r
+    if (alignPanel.seqPanel!=null && alignPanel.seqPanel.seqCanvas!=null)\r
+    {\r
+      PaintRefresher.RemoveComponent(alignPanel.seqPanel.seqCanvas);\r
+    }\r
+    if (alignPanel.idPanel!=null && alignPanel.idPanel.idCanvas!=null) {\r
+      PaintRefresher.RemoveComponent(alignPanel.idPanel.idCanvas);\r
+    }\r
 \r
     if (PaintRefresher.components.size() == 0 && viewport.applet == null)\r
     {\r
@@ -1304,7 +1319,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
   }\r
 \r
   /**\r
-   * DOCUMENT ME!\r
+   * TODO: JAL-1104\r
    */\r
   void updateEditMenuBar()\r
   {\r
@@ -1335,6 +1350,9 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
     }\r
   }\r
 \r
+  /**\r
+   * TODO: JAL-1104\r
+   */\r
   public void addHistoryItem(CommandI command)\r
   {\r
     if (command.getSize() > 0)\r
@@ -1347,8 +1365,9 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
   }\r
 \r
   /**\r
+   * TODO: JAL-1104\r
    * DOCUMENT ME!\r
-   * \r
+   *\r
    * @param e\r
    *          DOCUMENT ME!\r
    */\r
@@ -1375,8 +1394,9 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
   }\r
 \r
   /**\r
+   * TODO: JAL-1104\r
    * DOCUMENT ME!\r
-   * \r
+   *\r
    * @param e\r
    *          DOCUMENT ME!\r
    */\r
@@ -1451,57 +1471,16 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
     {\r
       return;\r
     }\r
-\r
-    if (up)\r
-    {\r
-      for (int i = 1; i < viewport.getAlignment().getHeight(); i++)\r
-      {\r
-        SequenceI seq = viewport.getAlignment().getSequenceAt(i);\r
-        if (!sg.getSequences(null).contains(seq))\r
-        {\r
-          continue;\r
-        }\r
-\r
-        SequenceI temp = viewport.getAlignment().getSequenceAt(i - 1);\r
-        if (sg.getSequences(null).contains(temp))\r
-        {\r
-          continue;\r
-        }\r
-\r
-        viewport.getAlignment().getSequences().setElementAt(temp, i);\r
-        viewport.getAlignment().getSequences().setElementAt(seq, i - 1);\r
-      }\r
-    }\r
-    else\r
-    {\r
-      for (int i = viewport.getAlignment().getHeight() - 2; i > -1; i--)\r
-      {\r
-        SequenceI seq = viewport.getAlignment().getSequenceAt(i);\r
-        if (!sg.getSequences(viewport.getHiddenRepSequences()).contains(seq))\r
-        {\r
-          continue;\r
-        }\r
-\r
-        SequenceI temp = viewport.getAlignment().getSequenceAt(i + 1);\r
-        if (sg.getSequences(viewport.getHiddenRepSequences()).contains(temp))\r
-        {\r
-          continue;\r
-        }\r
-\r
-        viewport.getAlignment().getSequences().setElementAt(temp, i);\r
-        viewport.getAlignment().getSequences().setElementAt(seq, i + 1);\r
-      }\r
-    }\r
-\r
+    viewport.getAlignment().moveSelectedSequencesByOne(sg, up ? null : viewport.getHiddenRepSequences(), up);\r
     alignPanel.paintAlignment(true);\r
   }\r
 \r
   synchronized void slideSequences(boolean right, int size)\r
   {\r
-    Vector sg = new Vector();\r
+    List<SequenceI>sg = new Vector<SequenceI>();\r
     if (viewport.cursorMode)\r
     {\r
-      sg.addElement(viewport.getAlignment()\r
+      sg.add(viewport.getAlignment()\r
               .getSequenceAt(alignPanel.seqPanel.seqCanvas.cursorY));\r
     }\r
     else if (viewport.getSelectionGroup() != null\r
@@ -1517,7 +1496,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
       return;\r
     }\r
 \r
-    Vector invertGroup = new Vector();\r
+    Vector<SequenceI> invertGroup = new Vector();\r
 \r
     for (int i = 0; i < viewport.getAlignment().getHeight(); i++)\r
     {\r
@@ -1525,13 +1504,11 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
         invertGroup.addElement(viewport.getAlignment().getSequenceAt(i));\r
     }\r
 \r
-    SequenceI[] seqs1 = new SequenceI[sg.size()];\r
-    for (int i = 0; i < sg.size(); i++)\r
-      seqs1[i] = (SequenceI) sg.elementAt(i);\r
+    SequenceI[] seqs1 = sg.toArray(new SequenceI[sg.size()]);\r
 \r
-    SequenceI[] seqs2 = new SequenceI[invertGroup.size()];\r
+    SequenceI[] seqs2 = invertGroup.toArray(new SequenceI[invertGroup.size()]);\r
     for (int i = 0; i < invertGroup.size(); i++)\r
-      seqs2[i] = (SequenceI) invertGroup.elementAt(i);\r
+      seqs2[i] = invertGroup.elementAt(i);\r
 \r
     SlideSequencesCommand ssc;\r
     if (right)\r
@@ -1823,7 +1800,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
 \r
   /**\r
    * group consensus toggled\r
-   * \r
+   *\r
    */\r
   protected void showGroupConsensus_actionPerformed()\r
   {\r
@@ -1843,7 +1820,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
 \r
   /*\r
    * (non-Javadoc)\r
-   * \r
+   *\r
    * @see\r
    * jalview.jbgui.GAlignFrame#showConsensusHistogram_actionPerformed(java.awt\r
    * .event.ActionEvent)\r
@@ -1855,7 +1832,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
   }\r
   /*\r
    * (non-Javadoc)\r
-   * \r
+   *\r
    * @see\r
    * jalview.jbgui.GAlignFrame#showConsensusProfile_actionPerformed(java.awt\r
    * .event.ActionEvent)\r
@@ -1865,6 +1842,13 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
     viewport.setShowSequenceLogo(showSequenceLogo.getState());\r
     alignPanel.updateAnnotation(applyAutoAnnotationSettings.getState());\r
   }\r
+  protected void normSequenceLogo_actionPerformed()\r
+  {\r
+    showSequenceLogo.setState(true);\r
+    viewport.setShowSequenceLogo(true);\r
+    viewport.setNormaliseSequenceLogo(normSequenceLogo.getState());\r
+    alignPanel.updateAnnotation(applyAutoAnnotationSettings.getState());\r
+  }\r
 \r
   protected void applyAutoAnnotationSettings_actionPerformed()\r
   {\r
@@ -1892,9 +1876,9 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
         Color col = new Color((int) (Math.random() * 255),\r
                 (int) (Math.random() * 255), (int) (Math.random() * 255));\r
         col = col.brighter();\r
-        for (Enumeration sq = gps[g].getSequences(null).elements(); sq\r
-                .hasMoreElements(); viewport.setSequenceColour(\r
-                (SequenceI) sq.nextElement(), col))\r
+        for (SequenceI sq : gps[g].getSequences(null))\r
+          viewport.setSequenceColour(\r
+                sq, col)\r
           ;\r
       }\r
       PaintRefresher.Refresh(this, viewport.getSequenceSetId());\r
@@ -2011,12 +1995,10 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
 \r
       addHistoryItem(trimRegion);\r
 \r
-      Vector groups = viewport.getAlignment().getGroups();\r
 \r
-      for (int i = 0; i < groups.size(); i++)\r
-      {\r
-        SequenceGroup sg = (SequenceGroup) groups.elementAt(i);\r
 \r
+      for (SequenceGroup sg:viewport.getAlignment().getGroups())\r
+      {\r
         if ((trimLeft && !sg.adjustForRemoveLeft(column))\r
                 || (!trimLeft && !sg.adjustForRemoveRight(column)))\r
         {\r
@@ -2108,7 +2090,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
 \r
   /**\r
    * create a new view derived from the current view\r
-   * \r
+   *\r
    * @param viewtitle\r
    * @return frame for the new view\r
    */\r
@@ -2181,7 +2163,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
   }\r
 \r
   /**\r
-   * \r
+   *\r
    * @return list of feature groups on the view\r
    */\r
   public String[] getFeatureGroups()\r
@@ -2197,7 +2179,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
 \r
   /**\r
    * get sequence feature groups that are hidden or shown\r
-   * \r
+   *\r
    * @param visible\r
    *          true is visible\r
    * @return list\r
@@ -2215,7 +2197,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
 \r
   /**\r
    * Change the display state for the given feature groups\r
-   * \r
+   *\r
    * @param groups\r
    *          list of group strings\r
    * @param state\r
@@ -2283,11 +2265,15 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
             overview.getPreferredSize().height + 50);\r
 \r
     frame.pack();\r
+    final AlignmentPanel ap=alignPanel;\r
     frame.addWindowListener(new WindowAdapter()\r
     {\r
+      @Override\r
       public void windowClosing(WindowEvent e)\r
       {\r
-        alignPanel.setOverviewPanel(null);\r
+        if (ap!=null) {\r
+          ap.setOverviewPanel(null);\r
+        }\r
       };\r
     });\r
 \r
@@ -2339,71 +2325,9 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
 \r
       cs.setConsensus(viewport.getSequenceConsensusHash());\r
 \r
-    }             \r
+    }\r
     viewport.setGlobalColourScheme(cs);\r
 \r
-    if (viewport.getColourAppliesToAllGroups())\r
-    {\r
-      Vector groups = viewport.getAlignment().getGroups();\r
-      for (int i = 0; i < groups.size(); i++)\r
-      {\r
-        SequenceGroup sg = (SequenceGroup) groups.elementAt(i);\r
-\r
-        if (cs == null)\r
-        {\r
-          sg.cs = null;\r
-          continue;\r
-        }\r
-        if (cs instanceof ClustalxColourScheme)\r
-        {\r
-          sg.cs = new ClustalxColourScheme(\r
-                  sg.getSequences(viewport.getHiddenRepSequences()),\r
-                  sg.getWidth());\r
-        }\r
-        else\r
-        {\r
-          try\r
-          {\r
-            sg.cs = cs.getClass().newInstance();\r
-          } catch (Exception ex)\r
-          {\r
-            ex.printStackTrace();\r
-            sg.cs = cs;\r
-          }\r
-        }\r
-\r
-        if (viewport.getAbovePIDThreshold()\r
-                || cs instanceof PIDColourScheme\r
-                || cs instanceof Blosum62ColourScheme)\r
-        {\r
-          sg.cs.setThreshold(threshold, viewport.getIgnoreGapsConsensus());\r
-          sg.cs.setConsensus(AAFrequency.calculate(\r
-                  sg.getSequences(viewport.getHiddenRepSequences()), 0,\r
-                  sg.getWidth()));\r
-        }\r
-        else\r
-        {\r
-          sg.cs.setThreshold(0, viewport.getIgnoreGapsConsensus());\r
-        }\r
-\r
-        if (viewport.getConservationSelected())\r
-        {\r
-          Conservation c = new Conservation("Group",\r
-                  ResidueProperties.propHash, 3,\r
-                  sg.getSequences(viewport.getHiddenRepSequences()), 0,\r
-                  viewport.getAlignment().getWidth() - 1);\r
-          c.calculate();\r
-          c.verdict(false, viewport.getConsPercGaps());\r
-          sg.cs.setConservation(c);\r
-        }\r
-        else\r
-        {\r
-          sg.cs.setConservation(null);\r
-          sg.cs.setThreshold(0, viewport.getIgnoreGapsConsensus());\r
-        }\r
-\r
-      }\r
-    }\r
 \r
     if (alignPanel.getOverviewPanel() != null)\r
     {\r
@@ -2598,7 +2522,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
 \r
     if ((viewport.getSelectionGroup() != null && viewport\r
             .getSelectionGroup().getSize() > 1)\r
-            || (viewport.getSelectionGroup() == null && viewport.getAlignment()\r
+            || (viewport.getAlignment()\r
                     .getHeight() > 1))\r
     {\r
       final TreePanel tp = new TreePanel(alignPanel, type, pwType);\r
@@ -2628,7 +2552,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
 \r
   /**\r
    * sort the alignment using the given treePanel\r
-   * \r
+   *\r
    * @param treePanel\r
    *          tree used to sort view\r
    * @param title\r
@@ -2649,7 +2573,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
   /**\r
    * Do any automatic reordering of the alignment and add the necessary bits to\r
    * the menu structure for the new tree\r
-   * \r
+   *\r
    * @param treePanel\r
    * @param title\r
    */\r
@@ -2660,14 +2584,16 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
     sortByTreeMenu.add(item);\r
     item.addActionListener(new java.awt.event.ActionListener()\r
     {\r
+      @Override\r
       public void actionPerformed(ActionEvent evt)\r
       {\r
         sortByTree(treePanel, title); // treePanel.getTitle());\r
       }\r
     });\r
-    \r
+\r
     treePanel.addWindowListener(new WindowAdapter()\r
     {\r
+      @Override\r
       public void windowOpened(WindowEvent e)\r
       {\r
         if (viewport.sortByTree)\r
@@ -2677,6 +2603,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
         super.windowOpened(e);\r
       }\r
 \r
+      @Override\r
       public void windowClosing(WindowEvent e)\r
       {\r
         sortByTreeMenu.remove(item);\r
@@ -2720,6 +2647,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
         this.builddate = builddate;\r
       }\r
 \r
+      @Override\r
       public void paint(Graphics g)\r
       {\r
         g.setColor(Color.white);\r
@@ -2789,8 +2717,6 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
   MenuItem loadTree = new MenuItem("Load Associated Tree ...");\r
 \r
   MenuItem loadAnnotations = new MenuItem("Load Features/Annotations ...");\r
-  \r
-  MenuItem loadScores = new MenuItem("Load Associated T-Coffee scores ...");\r
 \r
   MenuItem outputFeatures = new MenuItem("Export Features ...");\r
 \r
@@ -2866,13 +2792,13 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
 \r
   MenuItem purinePyrimidineColour = new MenuItem();\r
   MenuItem RNAHelixColour = new MenuItem();\r
-  \r
+\r
   MenuItem userDefinedColour = new MenuItem();\r
 \r
   MenuItem PIDColour = new MenuItem();\r
 \r
   MenuItem BLOSUM62Colour = new MenuItem();\r
-  \r
+\r
   MenuItem tcoffeeColour = new MenuItem();\r
 \r
   MenuItem njTreeBlosumMenuItem = new MenuItem();\r
@@ -2960,7 +2886,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
   CheckboxMenuItem seqLimits = new CheckboxMenuItem();\r
 \r
   CheckboxMenuItem centreColumnLabelFlag = new CheckboxMenuItem();\r
-  \r
+\r
   CheckboxMenuItem followMouseOverFlag = new CheckboxMenuItem();\r
   Menu autoAnnMenu=new Menu();\r
   CheckboxMenuItem showSequenceLogo= new CheckboxMenuItem();\r
@@ -2968,6 +2894,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
   CheckboxMenuItem showConsensusHistogram = new CheckboxMenuItem();\r
   CheckboxMenuItem showGroupConsensus = new CheckboxMenuItem();\r
   CheckboxMenuItem showGroupConservation = new CheckboxMenuItem();\r
+  CheckboxMenuItem normSequenceLogo = new CheckboxMenuItem();\r
 \r
   private void jbInit() throws Exception\r
   {\r
@@ -2985,6 +2912,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
 \r
       item.addActionListener(new java.awt.event.ActionListener()\r
       {\r
+        @Override\r
         public void actionPerformed(ActionEvent e)\r
         {\r
           outputText_actionPerformed(e);\r
@@ -2998,7 +2926,6 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
 \r
     loadTree.addActionListener(this);\r
     loadAnnotations.addActionListener(this);\r
-    loadScores.addActionListener(this);\r
     outputFeatures.addActionListener(this);\r
     outputAnnotations.addActionListener(this);\r
     selectAllSequenceMenuItem.addActionListener(this);\r
@@ -3183,10 +3110,11 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
     showGroupConservation.setLabel("Group Conservation");\r
     showConsensusHistogram.setLabel("Show Consensus Histogram");\r
     showSequenceLogo.setLabel("Show Consensus Logo");\r
+    normSequenceLogo.setLabel("Normalise Consensus Logo");\r
     applyAutoAnnotationSettings.setLabel("Apply to all groups");\r
     applyAutoAnnotationSettings.setState(true);\r
     autoAnnMenu.setLabel("Autocalculated Annotation");\r
-    \r
+\r
     invertColSel.addActionListener(this);\r
     showColumns.addActionListener(this);\r
     showSeqs.addActionListener(this);\r
@@ -3199,6 +3127,8 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
     showGroupConservation.addItemListener(this);\r
     showConsensusHistogram.addItemListener(this);\r
     showSequenceLogo.addItemListener(this);\r
+    normSequenceLogo.addItemListener(this);\r
+    \r
     applyAutoAnnotationSettings.addItemListener(this);\r
     formatMenu.setLabel("Format");\r
     selectMenu.setLabel("Select");\r
@@ -3216,8 +3146,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
     fileMenu.add(inputText);\r
     fileMenu.add(loadTree);\r
     fileMenu.add(loadAnnotations);\r
-    fileMenu.add(loadScores);\r
-    \r
+\r
     fileMenu.addSeparator();\r
     fileMenu.add(outputTextboxMenu);\r
     fileMenu.add(outputFeatures);\r
@@ -3253,6 +3182,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
     autoAnnMenu.add(applyAutoAnnotationSettings);\r
     autoAnnMenu.add(showConsensusHistogram);\r
     autoAnnMenu.add(showSequenceLogo);\r
+    autoAnnMenu.add(normSequenceLogo);\r
     autoAnnMenu.addSeparator();\r
     autoAnnMenu.add(showGroupConservation);\r
     autoAnnMenu.add(showGroupConsensus);\r
@@ -3375,7 +3305,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
    * Attach the alignFrame panels after embedding menus, if necessary. This used\r
    * to be called setEmbedded, but is now creates the dropdown menus in a\r
    * platform independent manner to avoid OSX/Mac menu appendage daftness.\r
-   * \r
+   *\r
    * @param reallyEmbedded\r
    *          true to attach the view to the applet area on the page rather than\r
    *          in a new window\r
@@ -3403,14 +3333,14 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
       final AlignFrame me = this;\r
       viewport.applet.addFocusListener(new FocusListener()\r
       {\r
-        \r
+\r
         @Override\r
         public void focusLost(FocusEvent e)\r
         {\r
           if (me.viewport.applet.currentAlignFrame==me) {\r
                   me.viewport.applet.currentAlignFrame = null;\r
         }}\r
-        \r
+\r
         @Override\r
         public void focusGained(FocusEvent e)\r
         {\r
@@ -3445,7 +3375,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
    * structures in the original jmol window. Note This method doesn't work\r
    * without an additional javascript library to exchange messages between the\r
    * distinct applets. See http://issues.jalview.org/browse/JAL-621\r
-   * \r
+   *\r
    * @param viewer\r
    *          JmolViewer instance\r
    * @param sequenceIds\r
@@ -3510,7 +3440,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
   }\r
   /**\r
    * bind a pdb file to a sequence in the current view\r
-   * \r
+   *\r
    * @param sequenceId\r
    *          - sequenceId within the dataset.\r
    * @param pdbEntryString\r
@@ -3682,7 +3612,7 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
 \r
   /**\r
    * modify the current selection, providing the user has not made a selection already.\r
-   * @param sel - sequences from this alignment \r
+   * @param sel - sequences from this alignment\r
    * @param csel - columns to be selected on the alignment\r
    */\r
   public void select(SequenceGroup sel, ColumnSelection csel)\r
@@ -3692,15 +3622,15 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
 \r
   public void scrollTo(int row, int column)\r
   {\r
-    alignPanel.seqPanel.scrollTo(row, column);    \r
+    alignPanel.seqPanel.scrollTo(row, column);\r
   }\r
   public void scrollToRow(int row)\r
   {\r
-    alignPanel.seqPanel.scrollToRow(row);    \r
+    alignPanel.seqPanel.scrollToRow(row);\r
   }\r
   public void scrollToColumn(int column)\r
   {\r
-    alignPanel.seqPanel.scrollToColumn(column);    \r
+    alignPanel.seqPanel.scrollToColumn(column);\r
   }\r
   /**\r
    * @return the alignments unique ID.\r
@@ -3708,41 +3638,51 @@ public class AlignFrame extends EmbmenuFrame implements ActionListener,  ItemLis
   public String getSequenceSetId() {\r
     return viewport.getSequenceSetId();\r
   }\r
-  \r
-  \r
+\r
+\r
   /**\r
-   * Load the (T-Coffee) score file from the specified url \r
-   * \r
-   * @param url The absolute path from where download and read the score file\r
-   * @throws IOException \r
+   * Load the (T-Coffee) score file from the specified url\r
+   *\r
+   * @param source File/URL/T-COFFEE score file contents\r
+   * @throws IOException\r
+   * @return true if alignment was annotated with data from source\r
    */\r
-  public void loadScoreFile( URL url ) throws IOException {\r
-         // TODO: refactor to string/standard jalview data importer\r
-         TCoffeeScoreFile file = TCoffeeScoreFile.load( new InputStreamReader( url.openStream() ) );\r
-         if( file == null ) {\r
-        // TODO: raise a dialog box here rather than bomb out.\r
-            \r
-                 throw new RuntimeException("The file provided does not match the T-Coffee scores file format");\r
+  public boolean loadScoreFile( String source ) throws IOException {\r
+\r
+    TCoffeeScoreFile file = new TCoffeeScoreFile(source, AppletFormatAdapter.checkProtocol(source));\r
+         if( !file.isValid()) {\r
+           // TODO: raise dialog for gui\r
+           System.err.println("Problems parsing T-Coffee scores: "+file.getWarningMessage());\r
+           System.err.println("Origin was:\n"+source);\r
+           return false;\r
          }\r
-         \r
+\r
          /*\r
           * check that the score matrix matches the alignment dimensions\r
           */\r
-         AlignmentI aln; \r
+         AlignmentI aln;\r
          if( (aln=viewport.getAlignment()) != null && (aln.getHeight() != file.getHeight() || aln.getWidth() != file.getWidth()) ) {\r
            // TODO: raise a dialog box here rather than bomb out.\r
-           throw new RuntimeException("The scores matrix does not match the alignment dimensions");\r
-                 \r
+           System.err.println("The scores matrix does not match the alignment dimensions");\r
+\r
          }\r
-         \r
+\r
           // TODO add parameter to indicate if matching should be done\r
          if (file.annotateAlignment(alignPanel.getAlignment(), false))\r
          {\r
-                 tcoffeeColour.setEnabled(true);\r
+           alignPanel.fontChanged();\r
+           tcoffeeColour.setEnabled(true);\r
                  // switch to this color\r
                  changeColour(new TCoffeeColourScheme(alignPanel.getAlignment()));\r
+                 return true;\r
+          } else {\r
+            System.err.println("Problems resolving T-Coffee scores:");\r
+            if (file.getWarningMessage()!=null) {\r
+              System.err.println(file.getWarningMessage());\r
+            }\r
           }\r
+         return false;\r
   }\r
-  \r
-  \r
+\r
+\r
 }\r