JAL-1620 version bump and release notes
[jalview.git] / src / jalview / gui / PopupMenu.java
index 7d953d4..957ca46 100644 (file)
@@ -1,40 +1,85 @@
 /*
- * Jalview - A Sequence Alignment Editor and Viewer (Version 2.8.0b1)
+ * Jalview - A Sequence Alignment Editor and Viewer (Version 2.8.2b1)
  * Copyright (C) 2014 The Jalview Authors
  * 
  * This file is part of Jalview.
  * 
  * 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.
+ * as published by the Free Software Foundation, either version 3
+ * of the License, or (at your option) any later version.
  *  
  * 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/>.
+ * You should have received a copy of the GNU General Public License
+ * along with Jalview.  If not, see <http://www.gnu.org/licenses/>.
  * The Jalview Authors are detailed in the 'AUTHORS' file.
  */
 package jalview.gui;
 
-import java.util.*;
-
-import java.awt.*;
-import java.awt.event.*;
-
-import javax.swing.*;
-
-import jalview.analysis.*;
-import jalview.commands.*;
-import jalview.datamodel.*;
-import jalview.io.*;
-import jalview.schemes.*;
+import jalview.analysis.AAFrequency;
+import jalview.analysis.AlignmentAnnotationUtils;
+import jalview.analysis.Conservation;
+import jalview.commands.ChangeCaseCommand;
+import jalview.commands.EditCommand;
+import jalview.commands.EditCommand.Action;
+import jalview.datamodel.AlignmentAnnotation;
+import jalview.datamodel.AlignmentI;
+import jalview.datamodel.Annotation;
+import jalview.datamodel.DBRefEntry;
+import jalview.datamodel.PDBEntry;
+import jalview.datamodel.Sequence;
+import jalview.datamodel.SequenceFeature;
+import jalview.datamodel.SequenceGroup;
+import jalview.datamodel.SequenceI;
+import jalview.io.FormatAdapter;
+import jalview.io.SequenceAnnotationReport;
+import jalview.schemes.AnnotationColourGradient;
+import jalview.schemes.Blosum62ColourScheme;
+import jalview.schemes.BuriedColourScheme;
+import jalview.schemes.ClustalxColourScheme;
+import jalview.schemes.HelixColourScheme;
+import jalview.schemes.HydrophobicColourScheme;
+import jalview.schemes.NucleotideColourScheme;
+import jalview.schemes.PIDColourScheme;
+import jalview.schemes.PurinePyrimidineColourScheme;
+import jalview.schemes.ResidueProperties;
+import jalview.schemes.StrandColourScheme;
+import jalview.schemes.TaylorColourScheme;
+import jalview.schemes.TurnColourScheme;
+import jalview.schemes.UserColourScheme;
+import jalview.schemes.ZappoColourScheme;
 import jalview.util.GroupUrlLink;
 import jalview.util.GroupUrlLink.UrlStringTooLongException;
-import jalview.util.MessageManager;\r
+import jalview.util.MessageManager;
 import jalview.util.UrlLink;
 
+import java.awt.Color;
+import java.awt.event.ActionEvent;
+import java.awt.event.ActionListener;
+import java.util.ArrayList;
+import java.util.Arrays;
+import java.util.Collection;
+import java.util.Collections;
+import java.util.Hashtable;
+import java.util.LinkedHashMap;
+import java.util.List;
+import java.util.Map;
+import java.util.TreeMap;
+import java.util.Vector;
+
+import javax.swing.ButtonGroup;
+import javax.swing.JCheckBoxMenuItem;
+import javax.swing.JColorChooser;
+import javax.swing.JMenu;
+import javax.swing.JMenuItem;
+import javax.swing.JOptionPane;
+import javax.swing.JPopupMenu;
+import javax.swing.JRadioButtonMenuItem;
+
 /**
  * DOCUMENT ME!
  * 
@@ -43,6 +88,10 @@ import jalview.util.UrlLink;
  */
 public class PopupMenu extends JPopupMenu
 {
+  private static final String ALL_ANNOTATIONS = "All";
+
+  private static final String COMMA = ",";
+
   JMenu groupMenu = new JMenu();
 
   JMenuItem groupName = new JMenuItem();
@@ -72,6 +121,7 @@ public class PopupMenu extends JPopupMenu
   protected JRadioButtonMenuItem BLOSUM62Colour = new JRadioButtonMenuItem();
 
   protected JRadioButtonMenuItem purinePyrimidineColour = new JRadioButtonMenuItem();
+
   protected JRadioButtonMenuItem RNAInteractionColour = new JRadioButtonMenuItem();
 
   // protected JRadioButtonMenuItem covariationColour = new
@@ -91,8 +141,12 @@ public class PopupMenu extends JPopupMenu
 
   JMenuItem sequenceSelDetails = new JMenuItem();
 
+  JMenuItem chooseAnnotations = new JMenuItem();
+
   SequenceI sequence;
+
   JMenuItem createGroupMenuItem = new JMenuItem();
+
   JMenuItem unGroupMenuItem = new JMenuItem();
 
   JMenuItem outline = new JMenuItem();
@@ -124,17 +178,31 @@ public class PopupMenu extends JPopupMenu
   JMenu pdbMenu = new JMenu();
 
   JMenuItem pdbFromFile = new JMenuItem();
-    // JBPNote: Commented these out - Should add these services via the web services menu system.
-    // JMenuItem ContraFold = new JMenuItem();
-  
-    // JMenuItem RNAFold = new JMenuItem();
-  
+
+  // JBPNote: Commented these out - Should add these services via the web
+  // services menu system.
+  // JMenuItem ContraFold = new JMenuItem();
+
+  // JMenuItem RNAFold = new JMenuItem();
+
   JMenuItem enterPDB = new JMenuItem();
 
   JMenuItem discoverPDB = new JMenuItem();
 
   JMenu outputMenu = new JMenu();
 
+  JMenu seqShowAnnotationsMenu = new JMenu();
+
+  JMenu seqHideAnnotationsMenu = new JMenu();
+
+  JMenuItem seqAddReferenceAnnotations = new JMenuItem();
+
+  JMenu groupShowAnnotationsMenu = new JMenu();
+
+  JMenu groupHideAnnotationsMenu = new JMenu();
+
+  JMenuItem groupAddReferenceAnnotations = new JMenuItem();
+
   JMenuItem sequenceFeature = new JMenuItem();
 
   JMenuItem textColour = new JMenuItem();
@@ -209,6 +277,7 @@ public class PopupMenu extends JPopupMenu
 
       item.addActionListener(new java.awt.event.ActionListener()
       {
+        @Override
         public void actionPerformed(ActionEvent e)
         {
           outputText_actionPerformed(e);
@@ -218,6 +287,29 @@ public class PopupMenu extends JPopupMenu
       outputMenu.add(item);
     }
 
+    /*
+     * Build menus for annotation types that may be shown or hidden, and for
+     * 'reference annotations' that may be added to the alignment. First for the
+     * currently selected sequence (if there is one):
+     */
+    final List<SequenceI> selectedSequence = (seq == null ? Collections
+            .<SequenceI> emptyList() : Arrays.asList(seq));
+    buildAnnotationTypesMenus(seqShowAnnotationsMenu,
+            seqHideAnnotationsMenu, selectedSequence);
+    configureReferenceAnnotationsMenu(seqAddReferenceAnnotations,
+            selectedSequence);
+
+    /*
+     * And repeat for the current selection group (if there is one):
+     */
+    final List<SequenceI> selectedGroup = (ap.av.getSelectionGroup() == null ? Collections
+            .<SequenceI> emptyList() : ap.av.getSelectionGroup()
+            .getSequences());
+    buildAnnotationTypesMenus(groupShowAnnotationsMenu,
+            groupHideAnnotationsMenu, selectedGroup);
+    configureReferenceAnnotationsMenu(groupAddReferenceAnnotations,
+            selectedGroup);
+
     try
     {
       jbInit();
@@ -243,17 +335,20 @@ public class PopupMenu extends JPopupMenu
 
           menuItem = new JMenuItem();
           menuItem.setText(pdb.getId());
-          menuItem.addActionListener(new java.awt.event.ActionListener()
+          menuItem.addActionListener(new ActionListener()
           {
+            @Override
             public void actionPerformed(ActionEvent e)
             {
               // TODO re JAL-860: optionally open dialog or provide a menu entry
               // allowing user to open just one structure per sequence
-              new AppJmol(pdb, ap.av.collateForPDB(new PDBEntry[]
-              { pdb })[0], null, ap);
-              // new PDBViewer(pdb, seqs2, null, ap, AppletFormatAdapter.FILE);
+              // new AppJmol(pdb, ap.av.collateForPDB(new PDBEntry[]
+              // { pdb })[0], null, ap);
+              new StructureViewer(ap.getStructureSelectionManager())
+                      .viewStructures(pdb,
+                              ap.av.collateForPDB(new PDBEntry[]
+                              { pdb })[0], null, ap);
             }
-
           });
           viewStructureMenu.add(menuItem);
 
@@ -279,32 +374,35 @@ public class PopupMenu extends JPopupMenu
       {
         AlignmentAnnotation[] aa = ap.av.getAlignment()
                 .getAlignmentAnnotation();
-        for (int i = 0; i < aa.length; i++)
+        for (int i = 0; aa != null && i < aa.length; i++)
         {
-          if (aa[i].getRNAStruc() != null)
+          if (aa[i].isValidStruc() && aa[i].sequenceRef == null)
           {
             final String rnastruc = aa[i].getRNAStruc();
-            final String structureLine = aa[i].label;
+            final String structureLine = aa[i].label + " (alignment)";
             menuItem = new JMenuItem();
-            menuItem.setText(MessageManager.formatMessage("label.2d_rna_structure_line", new String[]{structureLine}));\r
+            menuItem.setText(MessageManager.formatMessage(
+                    "label.2d_rna_structure_line", new String[]
+                    { structureLine }));
             menuItem.addActionListener(new java.awt.event.ActionListener()
-            
             {
+              @Override
               public void actionPerformed(ActionEvent e)
               {
-                       //System.out.println("1:"+structureLine);
-                       System.out.println("1:sname"+seq.getName());
-                       System.out.println("2:seq"+seq);
-                
-                       //System.out.println("3:"+seq.getSequenceAsString());
-                       System.out.println("3:strucseq"+rnastruc);
-                       //System.out.println("4:struc"+seq.getRNA());
-                       System.out.println("5:name"+seq.getName());
-                       System.out.println("6:ap"+ap);
-                       new AppVarna(structureLine, seq, seq.getSequenceAsString(), rnastruc, seq
-                            .getName(), ap);
-                       //new AppVarna(seq.getName(),seq,rnastruc,seq.getRNA(), seq.getName(), ap);
-                       System.out.println("end");
+                // // System.out.println("1:"+structureLine);
+                // System.out.println("1:sname" + seq.getName());
+                // System.out.println("2:seq" + seq);
+                //
+                // // System.out.println("3:"+seq.getSequenceAsString());
+                // System.out.println("3:strucseq" + rnastruc);
+                // // System.out.println("4:struc"+seq.getRNA());
+                // System.out.println("5:name" + seq.getName());
+                // System.out.println("6:ap" + ap);
+                new AppVarna(structureLine, seq, seq.getSequenceAsString(),
+                        rnastruc, seq.getName(), ap);
+                // new AppVarna(seq.getName(),seq,rnastruc,seq.getRNA(),
+                // seq.getName(), ap);
+                System.out.println("end");
               }
             });
             viewStructureMenu.add(menuItem);
@@ -318,19 +416,22 @@ public class PopupMenu extends JPopupMenu
           AlignmentAnnotation seqAnno[] = seq.getAnnotation();
           for (int i = 0; i < seqAnno.length; i++)
           {
-            if (seqAnno[i].getRNAStruc() != null)
+            if (seqAnno[i].isValidStruc())
             {
               final String rnastruc = seqAnno[i].getRNAStruc();
 
               // TODO: make rnastrucF a bit more nice
               menuItem = new JMenuItem();
-              menuItem.setText(MessageManager.formatMessage("label.2d_rna_sequence_name", new String[]{seq.getName()}));\r
+              menuItem.setText(MessageManager.formatMessage(
+                      "label.2d_rna_sequence_name", new String[]
+                      { seq.getName() }));
               menuItem.addActionListener(new java.awt.event.ActionListener()
               {
+                @Override
                 public void actionPerformed(ActionEvent e)
                 {
                   // TODO: VARNA does'nt print gaps in the sequence
-                
+
                   new AppVarna(seq.getName() + " structure", seq, seq
                           .getSequenceAsString(), rnastruc, seq.getName(),
                           ap);
@@ -343,9 +444,11 @@ public class PopupMenu extends JPopupMenu
 
       }
 
-      menuItem = new JMenuItem(MessageManager.getString("action.hide_sequences"));\r
+      menuItem = new JMenuItem(
+              MessageManager.getString("action.hide_sequences"));
       menuItem.addActionListener(new java.awt.event.ActionListener()
       {
+        @Override
         public void actionPerformed(ActionEvent e)
         {
           hideSequences(false);
@@ -356,9 +459,12 @@ public class PopupMenu extends JPopupMenu
       if (ap.av.getSelectionGroup() != null
               && ap.av.getSelectionGroup().getSize() > 1)
       {
-        menuItem = new JMenuItem(MessageManager.formatMessage("label.represent_group_with", new String[]{seq.getName()}));\r
+        menuItem = new JMenuItem(MessageManager.formatMessage(
+                "label.represent_group_with", new String[]
+                { seq.getName() }));
         menuItem.addActionListener(new java.awt.event.ActionListener()
         {
+          @Override
           public void actionPerformed(ActionEvent e)
           {
             hideSequences(true);
@@ -374,9 +480,11 @@ public class PopupMenu extends JPopupMenu
         if (ap.av.adjustForHiddenSeqs(index)
                 - ap.av.adjustForHiddenSeqs(index - 1) > 1)
         {
-          menuItem = new JMenuItem(MessageManager.getString("action.reveal_sequences"));\r
+          menuItem = new JMenuItem(
+                  MessageManager.getString("action.reveal_sequences"));
           menuItem.addActionListener(new ActionListener()
           {
+            @Override
             public void actionPerformed(ActionEvent e)
             {
               ap.av.showSequence(index);
@@ -394,9 +502,11 @@ public class PopupMenu extends JPopupMenu
     if (ap.av.hasHiddenRows())
     {
       {
-        menuItem = new JMenuItem(MessageManager.getString("action.reveal_all"));\r
+        menuItem = new JMenuItem(
+                MessageManager.getString("action.reveal_all"));
         menuItem.addActionListener(new ActionListener()
         {
+          @Override
           public void actionPerformed(ActionEvent e)
           {
             ap.av.showAllHiddenSeqs();
@@ -413,12 +523,16 @@ public class PopupMenu extends JPopupMenu
     }
 
     SequenceGroup sg = ap.av.getSelectionGroup();
-    boolean isDefinedGroup = (sg!=null) ? ap.av.getAlignment().getGroups().contains(sg) : false;
+    boolean isDefinedGroup = (sg != null) ? ap.av.getAlignment()
+            .getGroups().contains(sg) : false;
 
     if (sg != null && sg.getSize() > 0)
-    {      
-      groupName.setText(MessageManager.formatMessage("label.name_param", new String[]{sg.getName()}));\r
-      groupName.setText(MessageManager.getString("label.edit_name_and_description_current_group"));\r
+    {
+      groupName.setText(MessageManager.formatMessage("label.name_param",
+              new String[]
+              { sg.getName() }));
+      groupName.setText(MessageManager
+              .getString("label.edit_name_and_description_current_group"));
 
       if (sg.cs instanceof ZappoColourScheme)
       {
@@ -468,8 +582,7 @@ public class PopupMenu extends JPopupMenu
       {
         purinePyrimidineColour.setSelected(true);
       }
-      
-   
+
       /*
        * else if (sg.cs instanceof CovariationColourScheme) {
        * covariationColour.setSelected(true); }
@@ -493,15 +606,14 @@ public class PopupMenu extends JPopupMenu
         buildGroupURLMenu(sg, groupLinks);
       }
       // Add a 'show all structures' for the current selection
-      Hashtable<String, PDBEntry> pdbe = new Hashtable<String, PDBEntry>(),reppdb=new Hashtable<String,PDBEntry>();
+      Hashtable<String, PDBEntry> pdbe = new Hashtable<String, PDBEntry>(), reppdb = new Hashtable<String, PDBEntry>();
       SequenceI sqass = null;
       for (SequenceI sq : ap.av.getSequenceSelection())
       {
-        Vector<PDBEntry> pes = (Vector<PDBEntry>) sq.getDatasetSequence()
-                .getPDBId();
-        if (pes != null)
+        Vector<PDBEntry> pes = sq.getDatasetSequence().getPDBId();
+        if (pes != null && pes.size() > 0)
         {
-          reppdb.put(pes.get(0).getId(),pes.get(0));
+          reppdb.put(pes.get(0).getId(), pes.get(0));
           for (PDBEntry pe : pes)
           {
             pdbe.put(pe.getId(), pe);
@@ -515,38 +627,50 @@ public class PopupMenu extends JPopupMenu
       if (pdbe.size() > 0)
       {
         final PDBEntry[] pe = pdbe.values().toArray(
-                new PDBEntry[pdbe.size()]),pr = reppdb.values().toArray(
-                        new PDBEntry[reppdb.size()]);
-        final JMenuItem gpdbview,rpdbview;
+                new PDBEntry[pdbe.size()]), pr = reppdb.values().toArray(
+                new PDBEntry[reppdb.size()]);
+        final JMenuItem gpdbview, rpdbview;
         if (pdbe.size() == 1)
         {
-          structureMenu.add(gpdbview = new JMenuItem(MessageManager.formatMessage("label.view_structure_for", new String[]{sqass.getDisplayId(false)})));\r
+          structureMenu.add(gpdbview = new JMenuItem(MessageManager
+                  .formatMessage("label.view_structure_for", new String[]
+                  { sqass.getDisplayId(false) })));
         }
         else
         {
-          structureMenu.add(gpdbview = new JMenuItem(MessageManager.formatMessage("label.view_all_structures", new String[]{new Integer(pdbe.size()).toString()})));          \r
+          structureMenu.add(gpdbview = new JMenuItem(MessageManager
+                  .formatMessage("label.view_all_structures", new String[]
+                  { new Integer(pdbe.size()).toString() })));
         }
-        gpdbview.setToolTipText(MessageManager.getString("label.open_new_jmol_view_with_all_structures_associated_current_selection_superimpose_using_alignment"));\r
+        gpdbview.setToolTipText(MessageManager
+                .getString("label.open_new_jmol_view_with_all_structures_associated_current_selection_superimpose_using_alignment"));
         gpdbview.addActionListener(new ActionListener()
         {
 
           @Override
           public void actionPerformed(ActionEvent e)
           {
-            new AppJmol(ap, pe, ap.av.collateForPDB(pe));
+            new StructureViewer(ap.getStructureSelectionManager())
+                    .viewStructures(ap, pe, ap.av.collateForPDB(pe));
           }
         });
-        if (reppdb.size()>1 && reppdb.size()<pdbe.size())
+        if (reppdb.size() > 1 && reppdb.size() < pdbe.size())
         {
-          structureMenu.add(rpdbview = new JMenuItem(MessageManager.formatMessage("label.view_all_representative_structures", new String[]{new Integer(reppdb.size()).toString()})));
-          rpdbview.setToolTipText(MessageManager.getString("label.open_new_jmol_view_with_all_representative_structures_associated_current_selection_superimpose_using_alignment"));
+          structureMenu.add(rpdbview = new JMenuItem(MessageManager
+                  .formatMessage(
+                          "label.view_all_representative_structures",
+                          new String[]
+                          { new Integer(reppdb.size()).toString() })));
+          rpdbview.setToolTipText(MessageManager
+                  .getString("label.open_new_jmol_view_with_all_representative_structures_associated_current_selection_superimpose_using_alignment"));
           rpdbview.addActionListener(new ActionListener()
           {
 
             @Override
             public void actionPerformed(ActionEvent e)
             {
-              new AppJmol(ap, pr, ap.av.collateForPDB(pr));
+              new StructureViewer(ap.getStructureSelectionManager())
+                      .viewStructures(ap, pr, ap.av.collateForPDB(pr));
             }
           });
         }
@@ -562,11 +686,13 @@ public class PopupMenu extends JPopupMenu
     {
       createGroupMenuItem.setVisible(true);
       unGroupMenuItem.setVisible(false);
-      jMenu1.setText(MessageManager.getString("action.edit_new_group"));\r
-    } else {
+      jMenu1.setText(MessageManager.getString("action.edit_new_group"));
+    }
+    else
+    {
       createGroupMenuItem.setVisible(false);
       unGroupMenuItem.setVisible(true);
-      jMenu1.setText(MessageManager.getString("action.edit_group"));\r
+      jMenu1.setText(MessageManager.getString("action.edit_group"));
     }
 
     if (seq == null)
@@ -578,7 +704,7 @@ public class PopupMenu extends JPopupMenu
     if (links != null && links.size() > 0)
     {
 
-      JMenu linkMenu = new JMenu(MessageManager.getString("action.link"));\r
+      JMenu linkMenu = new JMenu(MessageManager.getString("action.link"));
       Vector linkset = new Vector();
       for (int i = 0; i < links.size(); i++)
       {
@@ -698,6 +824,162 @@ public class PopupMenu extends JPopupMenu
     }
   }
 
+  /**
+   * Add annotation types to 'Show annotations' and/or 'Hide annotations' menus.
+   * "All" is added first, followed by a separator. Then add any annotation
+   * types associated with the current selection. Separate menus are built for
+   * the selected sequence group (if any), and the selected sequence.
+   * <p>
+   * Some annotation rows are always rendered together - these can be identified
+   * by a common graphGroup property > -1. Only one of each group will be marked
+   * as visible (to avoid duplication of the display). For such groups we add a
+   * composite type name, e.g.
+   * <p>
+   * IUPredWS (Long), IUPredWS (Short)
+   * 
+   * @param seq
+   */
+  protected void buildAnnotationTypesMenus(JMenu showMenu, JMenu hideMenu,
+          List<SequenceI> forSequences)
+  {
+    showMenu.removeAll();
+    hideMenu.removeAll();
+
+    final List<String> all = Arrays.asList(ALL_ANNOTATIONS);
+    addAnnotationTypeToShowHide(showMenu, forSequences, "", all, true, true);
+    addAnnotationTypeToShowHide(hideMenu, forSequences, "", all, true,
+            false);
+    showMenu.addSeparator();
+    hideMenu.addSeparator();
+
+    final AlignmentAnnotation[] annotations = ap.getAlignment()
+            .getAlignmentAnnotation();
+
+    /*
+     * Find shown/hidden annotations types, distinguished by source (calcId),
+     * and grouped by graphGroup. Using LinkedHashMap means we will retrieve in
+     * the insertion order, which is the order of the annotations on the
+     * alignment.
+     */
+    Map<String, List<List<String>>> shownTypes = new LinkedHashMap<String, List<List<String>>>();
+    Map<String, List<List<String>>> hiddenTypes = new LinkedHashMap<String, List<List<String>>>();
+    AlignmentAnnotationUtils.getShownHiddenTypes(shownTypes,
+            hiddenTypes,
+            AlignmentAnnotationUtils.asList(annotations),
+            forSequences);
+
+    for (String calcId : hiddenTypes.keySet())
+    {
+      for (List<String> type : hiddenTypes.get(calcId))
+      {
+        addAnnotationTypeToShowHide(showMenu, forSequences,
+                calcId, type, false, true);
+      }
+    }
+    // grey out 'show annotations' if none are hidden
+    showMenu.setEnabled(!hiddenTypes.isEmpty());
+
+    for (String calcId : shownTypes.keySet())
+    {
+      for (List<String> type : shownTypes.get(calcId))
+      {
+        addAnnotationTypeToShowHide(hideMenu, forSequences,
+                calcId, type, false, false);
+      }
+    }
+    // grey out 'hide annotations' if none are shown
+    hideMenu.setEnabled(!shownTypes.isEmpty());
+  }
+
+  /**
+   * Returns a list of sequences - either the current selection group (if there
+   * is one), else the specified single sequence.
+   * 
+   * @param seq
+   * @return
+   */
+  protected List<SequenceI> getSequenceScope(SequenceI seq)
+  {
+    List<SequenceI> forSequences = null;
+    final SequenceGroup selectionGroup = ap.av.getSelectionGroup();
+    if (selectionGroup != null && selectionGroup.getSize() > 0)
+    {
+      forSequences = selectionGroup.getSequences();
+    }
+    else
+    {
+      forSequences = seq == null ? Collections.<SequenceI> emptyList()
+              : Arrays.asList(seq);
+    }
+    return forSequences;
+  }
+
+  /**
+   * Add one annotation type to the 'Show Annotations' or 'Hide Annotations'
+   * menus.
+   * 
+   * @param showOrHideMenu
+   *          the menu to add to
+   * @param forSequences
+   *          the sequences whose annotations may be shown or hidden
+   * @param calcId
+   * @param types
+   *          the label to add
+   * @param allTypes
+   *          if true this is a special label meaning 'All'
+   * @param actionIsShow
+   *          if true, the select menu item action is to show the annotation
+   *          type, else hide
+   */
+  protected void addAnnotationTypeToShowHide(JMenu showOrHideMenu,
+          final List<SequenceI> forSequences, String calcId,
+          final List<String> types, final boolean allTypes,
+          final boolean actionIsShow)
+  {
+    String label = types.toString(); // [a, b, c]
+    label = label.substring(1, label.length() - 1);
+    final JMenuItem item = new JMenuItem(label);
+    item.setToolTipText(calcId);
+    item.addActionListener(new java.awt.event.ActionListener()
+    {
+      @Override
+      public void actionPerformed(ActionEvent e)
+      {
+        showHideAnnotation_actionPerformed(types, forSequences, allTypes,
+                actionIsShow);
+      }
+    });
+    showOrHideMenu.add(item);
+  }
+
+  /**
+   * Action on selecting a list of annotation type (or the 'all types' values)
+   * to show or hide for the specified sequences.
+   * 
+   * @param types
+   * @param forSequences
+   * @param anyType
+   * @param doShow
+   */
+  protected void showHideAnnotation_actionPerformed(
+          Collection<String> types, List<SequenceI> forSequences,
+          boolean anyType, boolean doShow)
+  {
+    for (AlignmentAnnotation aa : ap.getAlignment()
+            .getAlignmentAnnotation())
+    {
+      if (anyType || types.contains(aa.label))
+      {
+        if ((aa.sequenceRef != null)
+                && forSequences.contains(aa.sequenceRef))
+        {
+          aa.visible = doShow;
+        }
+      }
+    }
+    refresh();
+  }
+
   private void buildGroupURLMenu(SequenceGroup sg, Vector groupLinks)
   {
 
@@ -705,11 +987,18 @@ public class PopupMenu extends JPopupMenu
     // menu appears asap
     // sequence only URLs
     // ID/regex match URLs
-    groupLinksMenu = new JMenu(MessageManager.getString("action.group_link"));\r
+    groupLinksMenu = new JMenu(
+            MessageManager.getString("action.group_link"));
     JMenu[] linkMenus = new JMenu[]
-    { null, new JMenu(MessageManager.getString("action.ids")), new JMenu(MessageManager.getString("action.sequences")),\r
-        new JMenu(MessageManager.getString("action.ids_sequences")) }; // three types of url that might be\r
-                                          // created.
+    { null, new JMenu(MessageManager.getString("action.ids")),
+        new JMenu(MessageManager.getString("action.sequences")),
+        new JMenu(MessageManager.getString("action.ids_sequences")) }; // three
+                                                                       // types
+                                                                       // of url
+                                                                       // that
+                                                                       // might
+                                                                       // be
+    // created.
     SequenceI[] seqs = ap.av.getSelectionAsNewSequence();
     String[][] idandseqs = GroupUrlLink.formStrings(seqs);
     Hashtable commonDbrefs = new Hashtable();
@@ -831,7 +1120,8 @@ public class PopupMenu extends JPopupMenu
     }
     if (addMenu)
     {
-      groupLinksMenu = new JMenu(MessageManager.getString("action.group_link"));\r
+      groupLinksMenu = new JMenu(
+              MessageManager.getString("action.group_link"));
       for (int m = 0; m < linkMenus.length; m++)
       {
         if (linkMenus[m] != null
@@ -857,14 +1147,18 @@ public class PopupMenu extends JPopupMenu
   private void addshowLink(JMenu linkMenu, String label, final String url)
   {
     JMenuItem item = new JMenuItem(label);
-    item.setToolTipText(MessageManager.formatMessage("label.open_url_param", new String[]{url}));\r
+    item.setToolTipText(MessageManager.formatMessage(
+            "label.open_url_param", new String[]
+            { url }));
     item.addActionListener(new java.awt.event.ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         new Thread(new Runnable()
         {
 
+          @Override
           public void run()
           {
             showLink(url);
@@ -892,15 +1186,21 @@ public class PopupMenu extends JPopupMenu
           final GroupUrlLink urlgenerator, final Object[] urlstub)
   {
     JMenuItem item = new JMenuItem(label);
-    item.setToolTipText(MessageManager.formatMessage("label.open_url_seqs_param", new Object[]{urlgenerator.getUrl_prefix(),urlgenerator.getNumberInvolved(urlstub)}));\r
-    // TODO: put in info about what is being sent.\r
+    item.setToolTipText(MessageManager.formatMessage(
+            "label.open_url_seqs_param",
+            new Object[]
+            { urlgenerator.getUrl_prefix(),
+                urlgenerator.getNumberInvolved(urlstub) }));
+    // TODO: put in info about what is being sent.
     item.addActionListener(new java.awt.event.ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         new Thread(new Runnable()
         {
 
+          @Override
           public void run()
           {
             try
@@ -926,228 +1226,282 @@ public class PopupMenu extends JPopupMenu
    */
   private void jbInit() throws Exception
   {
-    groupMenu.setText(MessageManager.getString("label.group"));\r
-    groupMenu.setText(MessageManager.getString("label.selection"));\r
-    groupName.setText(MessageManager.getString("label.name"));\r
+    groupMenu.setText(MessageManager.getString("label.group"));
+    groupMenu.setText(MessageManager.getString("label.selection"));
+    groupName.setText(MessageManager.getString("label.name"));
     groupName.addActionListener(new java.awt.event.ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         groupName_actionPerformed();
       }
     });
-    sequenceMenu.setText(MessageManager.getString("label.sequence"));\r
-    sequenceName.setText(MessageManager.getString("label.edit_name_description"));\r
+    sequenceMenu.setText(MessageManager.getString("label.sequence"));
+    sequenceName.setText(MessageManager
+            .getString("label.edit_name_description"));
     sequenceName.addActionListener(new java.awt.event.ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         sequenceName_actionPerformed();
       }
     });
-    sequenceDetails.setText(MessageManager.getString("label.sequence_details") + "...");\r
+    chooseAnnotations.setText(MessageManager
+            .getString("label.choose_annotations") + "...");
+    chooseAnnotations.addActionListener(new java.awt.event.ActionListener()
+    {
+      @Override
+      public void actionPerformed(ActionEvent e)
+      {
+        chooseAnnotations_actionPerformed(e);
+      }
+    });
+    sequenceDetails.setText(MessageManager
+            .getString("label.sequence_details") + "...");
     sequenceDetails.addActionListener(new java.awt.event.ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         sequenceDetails_actionPerformed();
       }
     });
-    sequenceSelDetails.setText(MessageManager.getString("label.sequence_details") + "...");\r
+    sequenceSelDetails.setText(MessageManager
+            .getString("label.sequence_details") + "...");
     sequenceSelDetails
             .addActionListener(new java.awt.event.ActionListener()
             {
+              @Override
               public void actionPerformed(ActionEvent e)
               {
                 sequenceSelectionDetails_actionPerformed();
               }
             });
     PIDColour.setFocusPainted(false);
-    unGroupMenuItem.setText(MessageManager.getString("action.remove_group"));\r
+    unGroupMenuItem
+            .setText(MessageManager.getString("action.remove_group"));
     unGroupMenuItem.addActionListener(new java.awt.event.ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         unGroupMenuItem_actionPerformed();
       }
     });
-    createGroupMenuItem.setText(MessageManager.getString("action.create_group"));\r
-    createGroupMenuItem.addActionListener(new java.awt.event.ActionListener()
-    {
-      public void actionPerformed(ActionEvent e)
-      {
-        createGroupMenuItem_actionPerformed();
-      }
-    });
+    createGroupMenuItem.setText(MessageManager
+            .getString("action.create_group"));
+    createGroupMenuItem
+            .addActionListener(new java.awt.event.ActionListener()
+            {
+              @Override
+              public void actionPerformed(ActionEvent e)
+              {
+                createGroupMenuItem_actionPerformed();
+              }
+            });
 
-    outline.setText(MessageManager.getString("action.border_colour"));\r
+    outline.setText(MessageManager.getString("action.border_colour"));
     outline.addActionListener(new java.awt.event.ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         outline_actionPerformed();
       }
     });
-    nucleotideMenuItem.setText(MessageManager.getString("label.nucleotide"));\r
+    nucleotideMenuItem
+            .setText(MessageManager.getString("label.nucleotide"));
     nucleotideMenuItem.addActionListener(new ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         nucleotideMenuItem_actionPerformed();
       }
     });
-    colourMenu.setText(MessageManager.getString("label.group_colour"));\r
-    showBoxes.setText(MessageManager.getString("action.boxes"));\r
+    colourMenu.setText(MessageManager.getString("label.group_colour"));
+    showBoxes.setText(MessageManager.getString("action.boxes"));
     showBoxes.setState(true);
     showBoxes.addActionListener(new ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         showBoxes_actionPerformed();
       }
     });
-    showText.setText(MessageManager.getString("action.text"));\r
+    showText.setText(MessageManager.getString("action.text"));
     showText.setState(true);
     showText.addActionListener(new ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         showText_actionPerformed();
       }
     });
-    showColourText.setText(MessageManager.getString("label.colour_text"));\r
+    showColourText.setText(MessageManager.getString("label.colour_text"));
     showColourText.addActionListener(new ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         showColourText_actionPerformed();
       }
     });
-    displayNonconserved.setText(MessageManager.getString("label.show_non_conversed"));\r
+    displayNonconserved.setText(MessageManager
+            .getString("label.show_non_conversed"));
     displayNonconserved.setState(true);
     displayNonconserved.addActionListener(new ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         showNonconserved_actionPerformed();
       }
     });
-    editMenu.setText(MessageManager.getString("action.edit"));\r
-    cut.setText(MessageManager.getString("action.cut"));\r
+    editMenu.setText(MessageManager.getString("action.edit"));
+    cut.setText(MessageManager.getString("action.cut"));
     cut.addActionListener(new ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         cut_actionPerformed();
       }
     });
-    upperCase.setText(MessageManager.getString("label.to_upper_case"));\r
+    upperCase.setText(MessageManager.getString("label.to_upper_case"));
     upperCase.addActionListener(new ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         changeCase(e);
       }
     });
-    copy.setText(MessageManager.getString("action.copy"));\r
+    copy.setText(MessageManager.getString("action.copy"));
     copy.addActionListener(new ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         copy_actionPerformed();
       }
     });
-    lowerCase.setText(MessageManager.getString("label.to_lower_case"));\r
+    lowerCase.setText(MessageManager.getString("label.to_lower_case"));
     lowerCase.addActionListener(new ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         changeCase(e);
       }
     });
-    toggle.setText(MessageManager.getString("label.toggle_case"));\r
+    toggle.setText(MessageManager.getString("label.toggle_case"));
     toggle.addActionListener(new ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         changeCase(e);
       }
     });
-    pdbMenu.setText(MessageManager.getString("label.associate_structure_with_sequence"));\r
-    pdbFromFile.setText(MessageManager.getString("label.from_file"));\r
+    pdbMenu.setText(MessageManager
+            .getString("label.associate_structure_with_sequence"));
+    pdbFromFile.setText(MessageManager.getString("label.from_file"));
     pdbFromFile.addActionListener(new ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         pdbFromFile_actionPerformed();
       }
     });
-//    RNAFold.setText("From RNA Fold with predict2D");
-//    RNAFold.addActionListener(new ActionListener()
-//    {
-//      public void actionPerformed(ActionEvent e)
-//      {
-//       try {
-//                     RNAFold_actionPerformed();
-//             } catch (Exception e1) {
-//                     // TODO Auto-generated catch block
-//                     e1.printStackTrace();
-//             }
-//      }   
-//    });
-//    ContraFold.setText("From Contra Fold with predict2D");
-//    ContraFold.addActionListener(new ActionListener()
-//    {
-//      public void actionPerformed(ActionEvent e)
-//      {
-//       try {
-//                     ContraFold_actionPerformed();
-//             } catch (Exception e1) {
-//                     // TODO Auto-generated catch block
-//                     e1.printStackTrace();
-//             }
-//      }   
-//    });
-    enterPDB.setText(MessageManager.getString("label.enter_pdb_id"));\r
+    // RNAFold.setText("From RNA Fold with predict2D");
+    // RNAFold.addActionListener(new ActionListener()
+    // {
+    // public void actionPerformed(ActionEvent e)
+    // {
+    // try {
+    // RNAFold_actionPerformed();
+    // } catch (Exception e1) {
+    // // TODO Auto-generated catch block
+    // e1.printStackTrace();
+    // }
+    // }
+    // });
+    // ContraFold.setText("From Contra Fold with predict2D");
+    // ContraFold.addActionListener(new ActionListener()
+    // {
+    // public void actionPerformed(ActionEvent e)
+    // {
+    // try {
+    // ContraFold_actionPerformed();
+    // } catch (Exception e1) {
+    // // TODO Auto-generated catch block
+    // e1.printStackTrace();
+    // }
+    // }
+    // });
+    enterPDB.setText(MessageManager.getString("label.enter_pdb_id"));
     enterPDB.addActionListener(new ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         enterPDB_actionPerformed();
       }
     });
-    discoverPDB.setText(MessageManager.getString("label.discover_pdb_ids"));\r
+    discoverPDB.setText(MessageManager.getString("label.discover_pdb_ids"));
     discoverPDB.addActionListener(new ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         discoverPDB_actionPerformed();
       }
     });
-    outputMenu.setText(MessageManager.getString("label.out_to_textbox") + "...");\r
-    sequenceFeature.setText(MessageManager.getString("label.create_sequence_feature"));\r
+    outputMenu.setText(MessageManager.getString("label.out_to_textbox")
+            + "...");
+    seqShowAnnotationsMenu.setText(MessageManager
+            .getString("label.show_annotations"));
+    seqHideAnnotationsMenu.setText(MessageManager
+            .getString("label.hide_annotations"));
+    groupShowAnnotationsMenu.setText(MessageManager
+            .getString("label.show_annotations"));
+    groupHideAnnotationsMenu.setText(MessageManager
+            .getString("label.hide_annotations"));
+    sequenceFeature.setText(MessageManager
+            .getString("label.create_sequence_feature"));
     sequenceFeature.addActionListener(new ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         sequenceFeature_actionPerformed();
       }
     });
-    textColour.setText(MessageManager.getString("label.text_colour"));\r
+    textColour.setText(MessageManager.getString("label.text_colour"));
     textColour.addActionListener(new ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         textColour_actionPerformed();
       }
     });
-    jMenu1.setText(MessageManager.getString("label.group"));\r
-    structureMenu.setText(MessageManager.getString("label.structure"));\r
-    viewStructureMenu.setText(MessageManager.getString("label.view_structure"));\r
+    jMenu1.setText(MessageManager.getString("label.group"));
+    structureMenu.setText(MessageManager.getString("label.structure"));
+    viewStructureMenu.setText(MessageManager
+            .getString("label.view_structure"));
     // colStructureMenu.setText("Colour By Structure");
-    editSequence.setText(MessageManager.getString("label.edit_sequence") + "...");\r
+    editSequence.setText(MessageManager.getString("label.edit_sequence")
+            + "...");
     editSequence.addActionListener(new ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent actionEvent)
       {
         editSequence_actionPerformed(actionEvent);
@@ -1164,6 +1518,19 @@ public class PopupMenu extends JPopupMenu
     add(groupMenu);
     add(sequenceMenu);
     this.add(structureMenu);
+    // annotations configuration panel suppressed for now
+    // groupMenu.add(chooseAnnotations);
+
+    /*
+     * Add show/hide annotations to the Sequence menu, and to the Selection menu
+     * (if a selection group is in force).
+     */
+    sequenceMenu.add(seqShowAnnotationsMenu);
+    sequenceMenu.add(seqHideAnnotationsMenu);
+    sequenceMenu.add(seqAddReferenceAnnotations);
+    groupMenu.add(groupShowAnnotationsMenu);
+    groupMenu.add(groupHideAnnotationsMenu);
+    groupMenu.add(groupAddReferenceAnnotations);
     groupMenu.add(editMenu);
     groupMenu.add(outputMenu);
     groupMenu.add(sequenceFeature);
@@ -1185,10 +1552,11 @@ public class PopupMenu extends JPopupMenu
     colourMenu.add(turnColour);
     colourMenu.add(buriedColour);
     colourMenu.add(nucleotideMenuItem);
-    if (ap.getAlignment().isNucleotide()) {
-       // JBPNote - commented since the colourscheme isn't functional
-       //  colourMenu.add(RNAInteractionColour);
-       colourMenu.add(purinePyrimidineColour);
+    if (ap.getAlignment().isNucleotide())
+    {
+      // JBPNote - commented since the colourscheme isn't functional
+      // colourMenu.add(RNAInteractionColour);
+      colourMenu.add(purinePyrimidineColour);
     }
     // colourMenu.add(covariationColour);
     colourMenu.add(userDefinedColour);
@@ -1203,6 +1571,7 @@ public class PopupMenu extends JPopupMenu
         JMenuItem item = new JMenuItem(userColours.nextElement().toString());
         item.addActionListener(new ActionListener()
         {
+          @Override
           public void actionPerformed(ActionEvent evt)
           {
             userDefinedColour_actionPerformed(evt);
@@ -1223,9 +1592,10 @@ public class PopupMenu extends JPopupMenu
     editMenu.add(lowerCase);
     editMenu.add(toggle);
     pdbMenu.add(pdbFromFile);
-    // JBPNote: These shouldn't be added here - should appear in a generic 'apply web service to this sequence menu'
-    //    pdbMenu.add(RNAFold);
-    //    pdbMenu.add(ContraFold);
+    // JBPNote: These shouldn't be added here - should appear in a generic
+    // 'apply web service to this sequence menu'
+    // pdbMenu.add(RNAFold);
+    // pdbMenu.add(ContraFold);
     pdbMenu.add(enterPDB);
     pdbMenu.add(discoverPDB);
     jMenu1.add(groupName);
@@ -1238,133 +1608,155 @@ public class PopupMenu extends JPopupMenu
     structureMenu.add(pdbMenu);
     structureMenu.add(viewStructureMenu);
     // structureMenu.add(colStructureMenu);
-    noColourmenuItem.setText(MessageManager.getString("label.none"));\r
+    noColourmenuItem.setText(MessageManager.getString("label.none"));
     noColourmenuItem.addActionListener(new java.awt.event.ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         noColourmenuItem_actionPerformed();
       }
     });
 
-    clustalColour.setText(MessageManager.getString("label.clustalx_colours"));\r
+    clustalColour.setText(MessageManager
+            .getString("label.clustalx_colours"));
     clustalColour.addActionListener(new java.awt.event.ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         clustalColour_actionPerformed();
       }
     });
-    zappoColour.setText(MessageManager.getString("label.zappo"));\r
+    zappoColour.setText(MessageManager.getString("label.zappo"));
     zappoColour.addActionListener(new java.awt.event.ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         zappoColour_actionPerformed();
       }
     });
-    taylorColour.setText(MessageManager.getString("label.taylor"));\r
+    taylorColour.setText(MessageManager.getString("label.taylor"));
     taylorColour.addActionListener(new java.awt.event.ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         taylorColour_actionPerformed();
       }
     });
-    hydrophobicityColour.setText(MessageManager.getString("label.hydrophobicity"));\r
+    hydrophobicityColour.setText(MessageManager
+            .getString("label.hydrophobicity"));
     hydrophobicityColour
             .addActionListener(new java.awt.event.ActionListener()
             {
+              @Override
               public void actionPerformed(ActionEvent e)
               {
                 hydrophobicityColour_actionPerformed();
               }
             });
-    helixColour.setText(MessageManager.getString("label.helix_propensity"));\r
+    helixColour.setText(MessageManager.getString("label.helix_propensity"));
     helixColour.addActionListener(new java.awt.event.ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         helixColour_actionPerformed();
       }
     });
-    strandColour.setText(MessageManager.getString("label.strand_propensity"));\r
+    strandColour.setText(MessageManager
+            .getString("label.strand_propensity"));
     strandColour.addActionListener(new java.awt.event.ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         strandColour_actionPerformed();
       }
     });
-    turnColour.setText(MessageManager.getString("label.turn_propensity"));\r
+    turnColour.setText(MessageManager.getString("label.turn_propensity"));
     turnColour.addActionListener(new java.awt.event.ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         turnColour_actionPerformed();
       }
     });
-    buriedColour.setText(MessageManager.getString("label.buried_index"));\r
+    buriedColour.setText(MessageManager.getString("label.buried_index"));
     buriedColour.addActionListener(new java.awt.event.ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         buriedColour_actionPerformed();
       }
     });
-    abovePIDColour.setText(MessageManager.getString("label.above_identity_percentage"));\r
+    abovePIDColour.setText(MessageManager
+            .getString("label.above_identity_percentage"));
     abovePIDColour.addActionListener(new java.awt.event.ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         abovePIDColour_actionPerformed();
       }
     });
-    userDefinedColour.setText(MessageManager.getString("action.user_defined"));\r
+    userDefinedColour.setText(MessageManager
+            .getString("action.user_defined"));
     userDefinedColour.addActionListener(new java.awt.event.ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         userDefinedColour_actionPerformed(e);
       }
     });
-    PIDColour.setText(MessageManager.getString("label.percentage_identity"));\r
+    PIDColour
+            .setText(MessageManager.getString("label.percentage_identity"));
     PIDColour.addActionListener(new java.awt.event.ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         PIDColour_actionPerformed();
       }
     });
-    BLOSUM62Colour.setText(MessageManager.getString("label.blosum62"));\r
+    BLOSUM62Colour.setText(MessageManager.getString("label.blosum62"));
     BLOSUM62Colour.addActionListener(new java.awt.event.ActionListener()
     {
+      @Override
       public void actionPerformed(ActionEvent e)
       {
         BLOSUM62Colour_actionPerformed();
       }
     });
-    purinePyrimidineColour.setText(MessageManager.getString("label.purine_pyrimidine"));\r
+    purinePyrimidineColour.setText(MessageManager
+            .getString("label.purine_pyrimidine"));
     purinePyrimidineColour
             .addActionListener(new java.awt.event.ActionListener()
             {
+              @Override
               public void actionPerformed(ActionEvent e)
               {
                 purinePyrimidineColour_actionPerformed();
               }
             });
-    
-   
+
     /*
      * covariationColour.addActionListener(new java.awt.event.ActionListener() {
      * public void actionPerformed(ActionEvent e) {
      * covariationColour_actionPerformed(); } });
      */
 
-    conservationMenuItem.setText(MessageManager.getString("label.conservation"));\r
+    conservationMenuItem.setText(MessageManager
+            .getString("label.conservation"));
     conservationMenuItem
             .addActionListener(new java.awt.event.ActionListener()
             {
+              @Override
               public void actionPerformed(ActionEvent e)
               {
                 conservationMenuItem_actionPerformed();
@@ -1372,6 +1764,155 @@ public class PopupMenu extends JPopupMenu
             });
   }
 
+  /**
+   * Check for any annotations on the underlying dataset sequences (for the
+   * current selection group) which are not 'on the alignment'.If any are found,
+   * enable the option to add them to the alignment. The criteria for 'on the
+   * alignment' is finding an alignment annotation on the alignment, matched on
+   * calcId, label and sequenceRef.
+   * 
+   * A tooltip is also constructed that displays the source (calcId) and type
+   * (label) of the annotations that can be added.
+   * 
+   * @param menuItem
+   * @param forSequences
+   */
+  protected void configureReferenceAnnotationsMenu(
+          JMenuItem menuItem, List<SequenceI> forSequences)
+  {
+    menuItem.setText(MessageManager
+            .getString("label.add_reference_annotations"));
+    menuItem.setEnabled(false);
+    if (forSequences == null)
+    {
+      return;
+    }
+
+    /*
+     * Temporary store to hold distinct calcId / type pairs for the tooltip.
+     * Using TreeMap means calcIds are shown in alphabetical order.
+     */
+    Map<String, String> tipEntries = new TreeMap<String, String>();
+    StringBuilder tooltip = new StringBuilder(64);
+    tooltip.append(MessageManager.getString("label.add_annotations_for"));
+
+    /*
+     * For each sequence selected in the alignment, make a list of any
+     * annotations on the underlying dataset sequence which are not already on
+     * the alignment.
+     * 
+     * Build a map of { alignmentSequence, <List of annotations to add> }
+     */
+    AlignmentI al = this.ap.av.getAlignment();
+    final Map<SequenceI, List<AlignmentAnnotation>> candidates = new LinkedHashMap<SequenceI, List<AlignmentAnnotation>>();
+    for (SequenceI seq : forSequences)
+    {
+      SequenceI dataset = seq.getDatasetSequence();
+      if (dataset == null)
+      {
+        continue;
+      }
+      AlignmentAnnotation[] datasetAnnotations = dataset.getAnnotation();
+      if (datasetAnnotations == null)
+      {
+        continue;
+      }
+      final List<AlignmentAnnotation> result = new ArrayList<AlignmentAnnotation>();
+      for (AlignmentAnnotation dsann : datasetAnnotations)
+      {
+        /*
+         * Find matching annotations on the alignment.
+         */
+        final Iterable<AlignmentAnnotation> matchedAlignmentAnnotations = al
+                .findAnnotations(seq, dsann.getCalcId(),
+                        dsann.label);
+        if (!matchedAlignmentAnnotations.iterator().hasNext())
+        {
+          result.add(dsann);
+          tipEntries.put(dsann.getCalcId(), dsann.label);
+        }
+      }
+      /*
+       * Save any addable annotations for this sequence
+       */
+      if (!result.isEmpty())
+      {
+        candidates.put(seq, result);
+      }
+    }
+    if (!candidates.isEmpty())
+    {
+      /*
+       * Found annotations that could be added. Enable the menu item, and
+       * configure its tooltip and action.
+       */
+      menuItem.setEnabled(true);
+      for (String calcId : tipEntries.keySet())
+      {
+        tooltip.append("<br/>" + calcId + "/" + tipEntries.get(calcId));
+      }
+      String tooltipText = JvSwingUtils.wrapTooltip(true,
+              tooltip.toString());
+      menuItem.setToolTipText(tooltipText);
+
+      menuItem.addActionListener(new ActionListener()
+      {
+        @Override
+        public void actionPerformed(ActionEvent e)
+        {
+          addReferenceAnnotations_actionPerformed(candidates);
+        }
+      });
+    }
+  }
+
+  /**
+   * Add annotations to the sequences and to the alignment.
+   * 
+   * @param candidates
+   *          a map whose keys are sequences on the alignment, and values a list
+   *          of annotations to add to each sequence
+   */
+  protected void addReferenceAnnotations_actionPerformed(
+          Map<SequenceI, List<AlignmentAnnotation>> candidates)
+  {
+    /*
+     * Add annotations at the top of the annotation, in the same order as their
+     * related sequences.
+     */
+    for (SequenceI seq : candidates.keySet())
+    {
+      for (AlignmentAnnotation ann : candidates.get(seq))
+      {
+        AlignmentAnnotation copyAnn = new AlignmentAnnotation(ann);
+        int startRes = 0;
+        int endRes = ann.annotations.length;
+        final SequenceGroup selectionGroup = this.ap.av.getSelectionGroup();
+        if (selectionGroup != null)
+        {
+          startRes = selectionGroup.getStartRes();
+          endRes = selectionGroup.getEndRes();
+        }
+        copyAnn.restrict(startRes, endRes);
+
+        /*
+         * Add to the sequence (sets copyAnn.datasetSequence), unless the
+         * original annotation is already on the sequence.
+         */
+        if (!seq.hasAnnotation(ann))
+        {
+          seq.addAlignmentAnnotation(copyAnn);
+        }
+        // adjust for gaps
+        copyAnn.adjustForAlignment();
+        // add to the alignment and set visible
+        this.ap.getAlignment().addAnnotation(copyAnn);
+        copyAnn.visible = true;
+      }
+    }
+    refresh();
+  }
+
   protected void sequenceSelectionDetails_actionPerformed()
   {
     createSequenceDetailsReport(ap.av.getSequenceSelection());
@@ -1389,8 +1930,12 @@ public class PopupMenu extends JPopupMenu
     StringBuffer contents = new StringBuffer();
     for (SequenceI seq : sequences)
     {
-      contents.append("<p><h2>" + MessageManager.formatMessage("label.create_sequence_details_report_annotation_for", new String[]{seq.getDisplayId(true)})\r
-              + "</h2></p><p>");
+      contents.append("<p><h2>"
+              + MessageManager
+                      .formatMessage(
+                              "label.create_sequence_details_report_annotation_for",
+                              new String[]
+                              { seq.getDisplayId(true) }) + "</h2></p><p>");
       new SequenceAnnotationReport(null)
               .createSequenceAnnotationReport(
                       contents,
@@ -1404,8 +1949,11 @@ public class PopupMenu extends JPopupMenu
     }
     cap.setText("<html>" + contents.toString() + "</html>");
 
-    Desktop.instance.addInternalFrame(cap, MessageManager.formatMessage("label.sequece_details_for", (sequences.length == 1 ? new String[]{sequences[0].getDisplayId(true)}: new String[]{MessageManager.getString("label.selection")}))\r
-               ,500, 400);\r
+    Desktop.instance.addInternalFrame(cap, MessageManager.formatMessage(
+            "label.sequece_details_for",
+            (sequences.length == 1 ? new String[]
+            { sequences[0].getDisplayId(true) } : new String[]
+            { MessageManager.getString("label.selection") })), 500, 400);
 
   }
 
@@ -1541,7 +2089,6 @@ public class PopupMenu extends JPopupMenu
     refresh();
   }
 
-
   /*
    * protected void covariationColour_actionPerformed() { getGroup().cs = new
    * CovariationColourScheme(sequence.getAnnotation()[0]); refresh(); }
@@ -1592,7 +2139,7 @@ public class PopupMenu extends JPopupMenu
   {
     SequenceGroup sg = getGroup();
 
-    if (e.getSource().equals(userDefinedColour))\r
+    if (e.getSource().equals(userDefinedColour))
     {
       new UserDefinedColours(ap, sg);
     }
@@ -1607,6 +2154,18 @@ public class PopupMenu extends JPopupMenu
   }
 
   /**
+   * Open a panel where the user can choose which types of sequence annotation
+   * to show or hide.
+   * 
+   * @param e
+   */
+  protected void chooseAnnotations_actionPerformed(ActionEvent e)
+  {
+    // todo correct way to guard against opening a duplicate panel?
+    new AnnotationChooser(ap);
+  }
+
+  /**
    * DOCUMENT ME!
    * 
    * @param e
@@ -1669,7 +2228,7 @@ public class PopupMenu extends JPopupMenu
 
     if (conservationMenuItem.isSelected())
     {
-    // JBPNote: Conservation name shouldn't be i18n translated
+      // JBPNote: Conservation name shouldn't be i18n translated
       Conservation c = new Conservation("Group",
               ResidueProperties.propHash, 3, sg.getSequences(ap.av
                       .getHiddenRepSequences()), sg.getStartRes(),
@@ -1721,8 +2280,10 @@ public class PopupMenu extends JPopupMenu
 
     SequenceGroup sg = getGroup();
     EditNameDialog dialog = new EditNameDialog(sg.getName(),
-            sg.getDescription(), "       " + MessageManager.getString("label.group_name") + " ",\r
-            MessageManager.getString("label.group_description") + " ", MessageManager.getString("label.edit_group_name_description"),\r
+            sg.getDescription(), "       "
+                    + MessageManager.getString("label.group_name") + " ",
+            MessageManager.getString("label.group_description") + " ",
+            MessageManager.getString("label.edit_group_name_description"),
             ap.alignFrame);
 
     if (!dialog.accept)
@@ -1761,8 +2322,12 @@ public class PopupMenu extends JPopupMenu
   void sequenceName_actionPerformed()
   {
     EditNameDialog dialog = new EditNameDialog(sequence.getName(),
-            sequence.getDescription(), "       " + MessageManager.getString("label.sequence_name") + " ",\r
-            MessageManager.getString("label.sequence_description") + " ", MessageManager.getString("label.edit_sequence_name_description"),\r
+            sequence.getDescription(),
+            "       " + MessageManager.getString("label.sequence_name")
+                    + " ",
+            MessageManager.getString("label.sequence_description") + " ",
+            MessageManager
+                    .getString("label.edit_sequence_name_description"),
             ap.alignFrame);
 
     if (!dialog.accept)
@@ -1774,10 +2339,14 @@ public class PopupMenu extends JPopupMenu
     {
       if (dialog.getName().indexOf(" ") > -1)
       {
-        JOptionPane.showMessageDialog(ap,
-                MessageManager.getString("label.spaces_converted_to_backslashes"),\r
-                MessageManager.getString("label.no_spaces_allowed_sequence_name"),\r
-                JOptionPane.WARNING_MESSAGE);
+        JOptionPane
+                .showMessageDialog(
+                        ap,
+                        MessageManager
+                                .getString("label.spaces_converted_to_backslashes"),
+                        MessageManager
+                                .getString("label.no_spaces_allowed_sequence_name"),
+                        JOptionPane.WARNING_MESSAGE);
       }
 
       sequence.setName(dialog.getName().replace(' ', '_'));
@@ -1804,9 +2373,11 @@ public class PopupMenu extends JPopupMenu
     ap.av.setSelectionGroup(null);
     refresh();
   }
+
   void createGroupMenuItem_actionPerformed()
   {
-    getGroup(); // implicitly creates group - note - should apply defaults / use standard alignment window logic for this
+    getGroup(); // implicitly creates group - note - should apply defaults / use
+                // standard alignment window logic for this
     refresh();
   }
 
@@ -1819,7 +2390,8 @@ public class PopupMenu extends JPopupMenu
   protected void outline_actionPerformed()
   {
     SequenceGroup sg = getGroup();
-    Color col = JColorChooser.showDialog(this, MessageManager.getString("label.select_outline_colour"),\r
+    Color col = JColorChooser.showDialog(this,
+            MessageManager.getString("label.select_outline_colour"),
             Color.BLUE);
 
     if (col != null)
@@ -1873,11 +2445,10 @@ public class PopupMenu extends JPopupMenu
       jalview.util.BrowserLauncher.openURL(url);
     } catch (Exception ex)
     {
-      JOptionPane
-              .showInternalMessageDialog(
-                      Desktop.desktop,
-                      MessageManager.getString("label.web_browser_not_found_unix"),\r
-                      MessageManager.getString("label.web_browser_not_found"), JOptionPane.WARNING_MESSAGE);\r
+      JOptionPane.showInternalMessageDialog(Desktop.desktop,
+              MessageManager.getString("label.web_browser_not_found_unix"),
+              MessageManager.getString("label.web_browser_not_found"),
+              JOptionPane.WARNING_MESSAGE);
 
       ex.printStackTrace();
     }
@@ -1943,17 +2514,17 @@ public class PopupMenu extends JPopupMenu
 
       if (source == toggle)
       {
-        description = MessageManager.getString("label.toggle_case");\r
+        description = MessageManager.getString("label.toggle_case");
         caseChange = ChangeCaseCommand.TOGGLE_CASE;
       }
       else if (source == upperCase)
       {
-        description = MessageManager.getString("label.to_upper_case");\r
+        description = MessageManager.getString("label.to_upper_case");
         caseChange = ChangeCaseCommand.TO_UPPER;
       }
       else
       {
-        description = MessageManager.getString("label.to_lower_case");\r
+        description = MessageManager.getString("label.to_lower_case");
         caseChange = ChangeCaseCommand.TO_LOWER;
       }
 
@@ -1973,31 +2544,18 @@ public class PopupMenu extends JPopupMenu
   {
     CutAndPasteTransfer cap = new CutAndPasteTransfer();
     cap.setForInput(null);
-    Desktop.addInternalFrame(cap,
-            MessageManager.formatMessage("label.alignment_output_command", new String[]{e.getActionCommand()}), 600, 500);\r
+    Desktop.addInternalFrame(cap, MessageManager.formatMessage(
+            "label.alignment_output_command", new String[]
+            { e.getActionCommand() }), 600, 500);
 
     String[] omitHidden = null;
 
     System.out.println("PROMPT USER HERE"); // TODO: decide if a prompt happens
     // or we simply trust the user wants
     // wysiwig behaviour
-    SequenceGroup sg = ap.av.getSelectionGroup();
-    ColumnSelection csel = new ColumnSelection(ap.av.getColumnSelection());
-    omitHidden = ap.av.getViewAsString(true);
-    Alignment oal = new Alignment(ap.av.getSequenceSelection());
-    AlignmentAnnotation[] nala = ap.av.getAlignment()
-            .getAlignmentAnnotation();
-    if (nala != null)
-    {
-      for (int i = 0; i < nala.length; i++)
-      {
-        AlignmentAnnotation na = nala[i];
-        oal.addAnnotation(na);
-      }
-    }
+
     cap.setText(new FormatAdapter().formatSequences(e.getActionCommand(),
-            oal, omitHidden, csel, sg));
-    oal = null;
+            ap.av, true));
   }
 
   public void pdbFromFile_actionPerformed()
@@ -2005,8 +2563,12 @@ public class PopupMenu extends JPopupMenu
     jalview.io.JalviewFileChooser chooser = new jalview.io.JalviewFileChooser(
             jalview.bin.Cache.getProperty("LAST_DIRECTORY"));
     chooser.setFileView(new jalview.io.JalviewFileView());
-    chooser.setDialogTitle(MessageManager.formatMessage("label.select_pdb_file_for", new String[]{sequence.getDisplayId(false)}));\r
-    chooser.setToolTipText(MessageManager.formatMessage("label.load_pdb_file_associate_with_sequence", new String[]{new Integer(sequence.getDisplayId(false)).toString()}));\r
+    chooser.setDialogTitle(MessageManager.formatMessage(
+            "label.select_pdb_file_for", new String[]
+            { sequence.getDisplayId(false) }));
+    chooser.setToolTipText(MessageManager.formatMessage(
+            "label.load_pdb_file_associate_with_sequence", new String[]
+            { sequence.getDisplayId(false) }));
 
     int value = chooser.showOpenDialog(null);
 
@@ -2015,26 +2577,30 @@ public class PopupMenu extends JPopupMenu
       String choice = chooser.getSelectedFile().getPath();
       jalview.bin.Cache.setProperty("LAST_DIRECTORY", choice);
       new AssociatePdbFileWithSeq().associatePdbWithSeq(choice,
-              jalview.io.AppletFormatAdapter.FILE, sequence, true);
+              jalview.io.AppletFormatAdapter.FILE, sequence, true,
+              Desktop.instance);
     }
 
   }
-    // JBNote: commented out - these won't be instantiated here...!  
-//  public void RNAFold_actionPerformed() throws Exception
-//  {
-//       Predict2D P2D = new Predict2D();
-//       P2D.getStructure2DFromRNAFold("toto");
-//  }
-//  
-//  public void ContraFold_actionPerformed() throws Exception
-//  {
-//       Predict2D P2D = new Predict2D();
-//       P2D.getStructure2DFromContraFold("toto");
-//  }
+
+  // JBNote: commented out - these won't be instantiated here...!
+  // public void RNAFold_actionPerformed() throws Exception
+  // {
+  // Predict2D P2D = new Predict2D();
+  // P2D.getStructure2DFromRNAFold("toto");
+  // }
+  //
+  // public void ContraFold_actionPerformed() throws Exception
+  // {
+  // Predict2D P2D = new Predict2D();
+  // P2D.getStructure2DFromContraFold("toto");
+  // }
   public void enterPDB_actionPerformed()
   {
     String id = JOptionPane.showInternalInputDialog(Desktop.desktop,
-            MessageManager.getString("label.enter_pdb_id"), MessageManager.getString("label.enter_pdb_id"), JOptionPane.QUESTION_MESSAGE);\r
+            MessageManager.getString("label.enter_pdb_id"),
+            MessageManager.getString("label.enter_pdb_id"),
+            JOptionPane.QUESTION_MESSAGE);
 
     if (id != null && id.length() > 0)
     {
@@ -2052,6 +2618,7 @@ public class PopupMenu extends JPopupMenu
             : ap.av.getSequenceSelection());
     Thread discpdb = new Thread(new Runnable()
     {
+      @Override
       public void run()
       {
 
@@ -2136,17 +2703,22 @@ public class PopupMenu extends JPopupMenu
     if (sg != null)
     {
       if (sequence == null)
-        sequence = (Sequence) sg.getSequenceAt(0);
+      {
+        sequence = sg.getSequenceAt(0);
+      }
 
       EditNameDialog dialog = new EditNameDialog(
               sequence.getSequenceAsString(sg.getStartRes(),
-                      sg.getEndRes() + 1), null, MessageManager.getString("label.edit_sequence"), null,\r
-                      MessageManager.getString("label.edit_sequence"), ap.alignFrame);\r
+                      sg.getEndRes() + 1), null,
+              MessageManager.getString("label.edit_sequence"), null,
+              MessageManager.getString("label.edit_sequence"),
+              ap.alignFrame);
 
       if (dialog.accept)
       {
-        EditCommand editCommand = new EditCommand(MessageManager.getString("label.edit_sequences"),\r
-                EditCommand.REPLACE, dialog.getName().replace(' ',
+        EditCommand editCommand = new EditCommand(
+                MessageManager.getString("label.edit_sequences"),
+                Action.REPLACE, dialog.getName().replace(' ',
                         ap.av.getGapCharacter()),
                 sg.getSequencesAsArray(ap.av.getHiddenRepSequences()),
                 sg.getStartRes(), sg.getEndRes() + 1, ap.av.getAlignment());