resolution for * JAL-553

[jalview.git] / src / jalview / gui / AnnotationLabels.java

diff --git a/src/jalview/gui/AnnotationLabels.java b/src/jalview/gui/AnnotationLabels.java
index 18bda61..1033936 100755 (executable)
--- a/src/jalview/gui/AnnotationLabels.java
+++ b/src/jalview/gui/AnnotationLabels.java
@@ -73,6 +73,8 @@ public class AnnotationLabels extends JPanel implements MouseListener,
 
   Font font = new Font("Arial", Font.PLAIN, 11);
 
+  private boolean hasHiddenRows;
+
   /**
    * Creates a new AnnotationLabels object.
    * 
@@ -130,15 +132,23 @@ public class AnnotationLabels extends JPanel implements MouseListener,
     repaint();
   }
 
+  /**
+   * sets selectedRow to -2 if no annotation preset, -1 if no visible row is at
+   * y
+   * 
+   * @param y
+   *          coordinate position to search for a row
+   */
   void getSelectedRow(int y)
   {
     int height = 0;
     AlignmentAnnotation[] aa = ap.av.alignment.getAlignmentAnnotation();
-
+    selectedRow = -2;
     if (aa != null)
     {
       for (int i = 0; i < aa.length; i++)
       {
+        selectedRow = -1;
         if (!aa[i].visible)
         {
           continue;
@@ -250,7 +260,8 @@ public class AnnotationLabels extends JPanel implements MouseListener,
   {
     EditNameDialog dialog = new EditNameDialog(annotation.label,
             annotation.description, "       Annotation Name ",
-            "Annotation Description ", "Edit Annotation Name/Description");
+            "Annotation Description ", "Edit Annotation Name/Description",
+            ap.alignFrame);
 
     if (!dialog.accept)
     {
@@ -298,6 +309,10 @@ public class AnnotationLabels extends JPanel implements MouseListener,
       // Swap these annotations
       AlignmentAnnotation startAA = ap.av.alignment
               .getAlignmentAnnotation()[start];
+      if (end == -1)
+      {
+        end = ap.av.alignment.getAlignmentAnnotation().length - 1;
+      }
       AlignmentAnnotation endAA = ap.av.alignment.getAlignmentAnnotation()[end];
 
       ap.av.alignment.getAlignmentAnnotation()[end] = startAA;
@@ -429,7 +444,7 @@ public class AnnotationLabels extends JPanel implements MouseListener,
     AlignmentAnnotation[] aa = ap.av.alignment.getAlignmentAnnotation();
     if (SwingUtilities.isLeftMouseButton(evt))
     {
-      if (selectedRow < aa.length)
+      if (selectedRow > -1 && selectedRow < aa.length)
       {
         if (aa[selectedRow].groupRef != null)
         {
@@ -439,7 +454,9 @@ public class AnnotationLabels extends JPanel implements MouseListener,
             ap.seqPanel.ap.idPanel.highlightSearchResults(null);
             ap.av.setSelectionGroup(// new SequenceGroup(
                     aa[selectedRow].groupRef); // );
+            ap.av.sendSelection();
             ap.paintAlignment(false);
+            PaintRefresher.Refresh(ap, ap.av.getSequenceSetId());
           }
           else
           {
@@ -463,6 +480,7 @@ public class AnnotationLabels extends JPanel implements MouseListener,
             SequenceGroup sg = new SequenceGroup();
             sg.addSequence(aa[selectedRow].sequenceRef, false);
             ap.av.setSelectionGroup(sg);
+            ap.av.sendSelection();
             ap.paintAlignment(false);
             PaintRefresher.Refresh(ap, ap.av.getSequenceSetId());
           }
@@ -478,17 +496,18 @@ public class AnnotationLabels extends JPanel implements MouseListener,
     JPopupMenu pop = new JPopupMenu("Annotations");
     JMenuItem item = new JMenuItem(ADDNEW);
     item.addActionListener(this);
-
-    if ((aa == null) || (aa.length == 0))
+    pop.add(item);
+    if (selectedRow < 0)
     {
-      item = new JMenuItem(SHOWALL);
-      item.addActionListener(this);
-      pop.add(item);
+      if (hasHiddenRows)
+      { // let the user make everything visible again
+        item = new JMenuItem(SHOWALL);
+        item.addActionListener(this);
+        pop.add(item);
+      }
       pop.show(this, evt.getX(), evt.getY());
       return;
     }
-
-    pop.add(item);
     item = new JMenuItem(EDITNAME);
     item.addActionListener(this);
     pop.add(item);
@@ -498,9 +517,12 @@ public class AnnotationLabels extends JPanel implements MouseListener,
     item = new JMenuItem(DELETE);
     item.addActionListener(this);
     pop.add(item);
-    item = new JMenuItem(SHOWALL);
-    item.addActionListener(this);
-    pop.add(item);
+    if (hasHiddenRows)
+    {
+      item = new JMenuItem(SHOWALL);
+      item.addActionListener(this);
+      pop.add(item);
+    }
     item = new JMenuItem(OUTPUT_TEXT);
     item.addActionListener(this);
     pop.add(item);
@@ -750,12 +772,14 @@ public class AnnotationLabels extends JPanel implements MouseListener,
 
     if (aa != null)
     {
+      hasHiddenRows = false;
       for (int i = 0; i < aa.length; i++)
       {
         g.setColor(Color.black);
 
         if (!aa[i].visible)
         {
+          hasHiddenRows = true;
           continue;
         }