JAL-2388 Reverted to previous
authorkiramt <k.mourao@dundee.ac.uk>
Wed, 1 Mar 2017 16:40:07 +0000 (16:40 +0000)
committerkiramt <k.mourao@dundee.ac.uk>
Wed, 1 Mar 2017 16:40:07 +0000 (16:40 +0000)
src/jalview/gui/AlignViewport.java
src/jalview/viewmodel/AlignmentViewport.java
src/jalview/viewmodel/ViewportPositionProps.java

index f1bb8de..7bebfa9 100644 (file)
@@ -150,11 +150,10 @@ public class AlignViewport extends AlignmentViewport implements
   public AlignViewport(AlignmentI al, ColumnSelection hiddenColumns)
   {
     setAlignment(al);
-    posProps.setHiddenColumns(hiddenColumns);
-    /*if (hiddenColumns != null)
+    if (hiddenColumns != null)
     {
       colSel = hiddenColumns;
-    }*/
+    }
     init();
   }
 
@@ -198,11 +197,10 @@ public class AlignViewport extends AlignmentViewport implements
       Cache.log.debug("Setting viewport's view id : " + viewId);
     }
     setAlignment(al);
-    posProps.setHiddenColumns(hiddenColumns);
-    /*if (hiddenColumns != null)
+    if (hiddenColumns != null)
     {
       colSel = hiddenColumns;
-    }*/
+    }
     init();
   }
 
@@ -535,9 +533,7 @@ public class AlignViewport extends AlignmentViewport implements
     {
       end = alignment.getWidth();
     }
-    viscontigs = posProps.getColumnSelection()
-            .getVisibleContigs(start, end);
-    // viscontigs = colSel.getVisibleContigs(start, end);
+    viscontigs = colSel.getVisibleContigs(start, end);
     return viscontigs;
   }
 
index b62c61c..41248b1 100644 (file)
@@ -589,9 +589,9 @@ public abstract class AlignmentViewport implements AlignViewportI,
     return isDataset;
   }
 
+  private Map<SequenceI, SequenceCollectionI> hiddenRepSequences;
 
-
-  // protected ColumnSelection colSel = new ColumnSelection();
+  protected ColumnSelection colSel = new ColumnSelection();
 
   public boolean autoCalculateConsensus = true;
 
@@ -632,7 +632,7 @@ public abstract class AlignmentViewport implements AlignViewportI,
       {
         residueShading.setConservation(hconservation);
       }
-      residueShading.alignmentChanged(alignment, getHiddenRepSequences());
+      residueShading.alignmentChanged(alignment, hiddenRepSequences);
     }
 
     /*
@@ -651,8 +651,7 @@ public abstract class AlignmentViewport implements AlignViewportI,
         if (cs != null)
         {
           sg.getGroupColourScheme()
-.alignmentChanged(sg,
-                  getHiddenRepSequences());
+                  .alignmentChanged(sg, hiddenRepSequences);
         }
       }
     }
@@ -1095,27 +1094,24 @@ public abstract class AlignmentViewport implements AlignViewportI,
 
   public void setHiddenColumns(ColumnSelection colsel)
   {
-    posProps.setHiddenColumns(colsel);
-    // this.colSel = colsel;
+    this.colSel = colsel;
   }
 
   @Override
   public ColumnSelection getColumnSelection()
   {
-    return posProps.getColumnSelection();
-    // return colSel;
+    return colSel;
   }
 
   @Override
   public void setColumnSelection(ColumnSelection colSel)
   {
-    posProps.setColumnSelection(colSel);
-    /*this.colSel = colSel;
+    this.colSel = colSel;
     if (colSel != null)
     {
       updateHiddenColumns();
     }
-    isColSelChanged(true);*/
+    isColSelChanged(true);
   }
 
   /**
@@ -1125,14 +1121,14 @@ public abstract class AlignmentViewport implements AlignViewportI,
   @Override
   public Map<SequenceI, SequenceCollectionI> getHiddenRepSequences()
   {
-    return posProps.getHiddenRepSequences();
+    return hiddenRepSequences;
   }
 
   @Override
   public void setHiddenRepSequences(
           Map<SequenceI, SequenceCollectionI> hiddenRepSequences)
   {
-    posProps.getHiddenRepSequences();
+    this.hiddenRepSequences = hiddenRepSequences;
   }
 
   @Override
@@ -1145,8 +1141,7 @@ public abstract class AlignmentViewport implements AlignViewportI,
   @Override
   public boolean hasHiddenColumns()
   {
-    return posProps.hasHiddenColumns();
-    // return colSel != null && colSel.hasHiddenColumns();
+    return colSel != null && colSel.hasHiddenColumns();
   }
 
   public void updateHiddenColumns()
@@ -1216,7 +1211,7 @@ public abstract class AlignmentViewport implements AlignViewportI,
 
   }
 
-  private long sgrouphash = -1;
+  private long sgrouphash = -1, colselhash = -1;
 
   /**
    * checks current SelectionGroup against record of last hash value, and
@@ -1252,8 +1247,7 @@ public abstract class AlignmentViewport implements AlignViewportI,
    */
   public boolean isColSelChanged(boolean b)
   {
-    return posProps.isColSelChanged(b);
-    /*int hc = (colSel == null || colSel.isEmpty()) ? -1 : colSel.hashCode();
+    int hc = (colSel == null || colSel.isEmpty()) ? -1 : colSel.hashCode();
     if (hc != -1 && hc != colselhash)
     {
       if (b)
@@ -1262,7 +1256,7 @@ public abstract class AlignmentViewport implements AlignViewportI,
       }
       return true;
     }
-    return false;*/
+    return false;
   }
 
   @Override
@@ -1343,21 +1337,6 @@ public abstract class AlignmentViewport implements AlignViewportI,
 
   // common hide/show column stuff
 
-  public void hideColumns(int start, int end)
-  {
-    posProps.hideColumns(start, end);
-  }
-
-  public void showColumn(int col)
-  {
-    posProps.showColumn(col);
-  }
-
-  public void showAllHiddenColumns()
-  {
-    posProps.showAllHiddenColumns();
-  }
-
   public void hideSelectedColumns()
   {
     if (colSel.isEmpty())
@@ -1370,41 +1349,61 @@ public abstract class AlignmentViewport implements AlignViewportI,
     isColSelChanged(true);
   }
 
+  public void hideColumns(int start, int end)
+  {
+    if (start == end)
+    {
+      colSel.hideColumns(start);
+    }
+    else
+    {
+      colSel.hideColumns(start, end);
+    }
+    isColSelChanged(true);
+  }
+
+  public void showColumn(int col)
+  {
+    colSel.revealHiddenColumns(col);
+    isColSelChanged(true);
+  }
 
+  public void showAllHiddenColumns()
+  {
+    colSel.revealAllHiddenColumns();
+    isColSelChanged(true);
+  }
 
   // common hide/show seq stuff
   public void showAllHiddenSeqs()
   {
-    selectionGroup = posProps.showAllHiddenSeqs(selectionGroup);
-
-    /*    if (alignment.getHiddenSequences().getSize() > 0)
-        {
-          if (selectionGroup == null)
-          {
-            selectionGroup = new SequenceGroup();
-            selectionGroup.setEndRes(alignment.getWidth() - 1);
-          }
-          List<SequenceI> tmp = alignment.getHiddenSequences().showAll(
-                  hiddenRepSequences);
-          for (SequenceI seq : tmp)
-          {
-            selectionGroup.addSequence(seq, false);
-            setSequenceAnnotationsVisible(seq, true);
-          }
+    if (alignment.getHiddenSequences().getSize() > 0)
+    {
+      if (selectionGroup == null)
+      {
+        selectionGroup = new SequenceGroup();
+        selectionGroup.setEndRes(alignment.getWidth() - 1);
+      }
+      List<SequenceI> tmp = alignment.getHiddenSequences().showAll(
+              hiddenRepSequences);
+      for (SequenceI seq : tmp)
+      {
+        selectionGroup.addSequence(seq, false);
+        setSequenceAnnotationsVisible(seq, true);
+      }
 
-          hiddenRepSequences = null;
+      hiddenRepSequences = null;
 
-          firePropertyChange("alignment", null, alignment.getSequences());
-          // used to set hasHiddenRows/hiddenRepSequences here, after the property
-          // changed event
-          sendSelection();
-        }*/
+      firePropertyChange("alignment", null, alignment.getSequences());
+      // used to set hasHiddenRows/hiddenRepSequences here, after the property
+      // changed event
+      sendSelection();
+    }
   }
 
   public void showSequence(int index)
   {
-    posProps.showSequence(index, selectionGroup);
-    /*List<SequenceI> tmp = alignment.getHiddenSequences().showSequence(
+    List<SequenceI> tmp = alignment.getHiddenSequences().showSequence(
             index, hiddenRepSequences);
     if (tmp.size() > 0)
     {
@@ -1421,13 +1420,12 @@ public abstract class AlignmentViewport implements AlignViewportI,
       }
       firePropertyChange("alignment", null, alignment.getSequences());
       sendSelection();
-    }*/
+    }
   }
 
   public void hideAllSelectedSeqs()
   {
-    posProps.hideAllSelectedSeqs(selectionGroup);
-    /*if (selectionGroup == null || selectionGroup.getSize() < 1)
+    if (selectionGroup == null || selectionGroup.getSize() < 1)
     {
       return;
     }
@@ -1436,13 +1434,12 @@ public abstract class AlignmentViewport implements AlignViewportI,
 
     hideSequence(seqs);
 
-    setSelectionGroup(null);*/
+    setSelectionGroup(null);
   }
 
   public void hideSequence(SequenceI[] seq)
   {
-    posProps.hideSequence(seq);
-    /*if (seq != null)
+    if (seq != null)
     {
       for (int i = 0; i < seq.length; i++)
       {
@@ -1450,7 +1447,7 @@ public abstract class AlignmentViewport implements AlignViewportI,
         setSequenceAnnotationsVisible(seq[i], false);
       }
       firePropertyChange("alignment", null, alignment.getSequences());
-    }*/
+    }
   }
 
   /**
@@ -1464,8 +1461,7 @@ public abstract class AlignmentViewport implements AlignViewportI,
    */
   public void hideSequences(SequenceI sequence, boolean representGroup)
   {
-    posProps.hideSequences(sequence, representGroup, selectionGroup);
-    /*if (selectionGroup == null || selectionGroup.getSize() < 1)
+    if (selectionGroup == null || selectionGroup.getSize() < 1)
     {
       hideSequence(new SequenceI[] { sequence });
       return;
@@ -1484,14 +1480,33 @@ public abstract class AlignmentViewport implements AlignViewportI,
 
     hideSequence(hseqs);
     setSelectionGroup(null);
-    sendSelection();*/
+    sendSelection();
   }
 
+  /**
+   * Set visibility for any annotations for the given sequence.
+   * 
+   * @param sequenceI
+   */
+  protected void setSequenceAnnotationsVisible(SequenceI sequenceI,
+          boolean visible)
+  {
+    AlignmentAnnotation[] anns = alignment.getAlignmentAnnotation();
+    if (anns != null)
+    {
+      for (AlignmentAnnotation ann : anns)
+      {
+        if (ann.sequenceRef == sequenceI)
+        {
+          ann.visible = visible;
+        }
+      }
+    }
+  }
 
   public void hideRepSequences(SequenceI repSequence, SequenceGroup sg)
   {
-    posProps.hideRepSequences(repSequence, sg);
-    /*int sSize = sg.getSize();
+    int sSize = sg.getSize();
     if (sSize < 2)
     {
       return;
@@ -1521,7 +1536,7 @@ public abstract class AlignmentViewport implements AlignViewportI,
     }
     sg.setSeqrep(repSequence); // note: not done in 2.7applet
     sg.setHidereps(true); // note: not done in 2.7applet
-    hideSequence(seqs);*/
+    hideSequence(seqs);
 
   }
 
@@ -1551,9 +1566,8 @@ public abstract class AlignmentViewport implements AlignViewportI,
    */
   public boolean isHiddenRepSequence(SequenceI seq)
   {
-    return posProps.isHiddenRepSequence(seq);
-    // return (hiddenRepSequences != null && hiddenRepSequences
-    // .containsKey(seq));
+    return (hiddenRepSequences != null && hiddenRepSequences
+            .containsKey(seq));
   }
 
   /**
@@ -1564,24 +1578,21 @@ public abstract class AlignmentViewport implements AlignViewportI,
    */
   public SequenceGroup getRepresentedSequences(SequenceI seq)
   {
-    return posProps.getRepresentedSequences(seq);
-    // return (SequenceGroup) (hiddenRepSequences == null ? null
-    // : hiddenRepSequences.get(seq));
+    return (SequenceGroup) (hiddenRepSequences == null ? null
+            : hiddenRepSequences.get(seq));
   }
 
   @Override
   public int adjustForHiddenSeqs(int alignmentIndex)
   {
-    return posProps.adjustForHiddenSeqs(alignmentIndex);
-    // return alignment.getHiddenSequences().adjustForHiddenSeqs(
-    // alignmentIndex);
+    return alignment.getHiddenSequences().adjustForHiddenSeqs(
+            alignmentIndex);
   }
 
   @Override
   public void invertColumnSelection()
   {
-    posProps.invertColumnSelection();
-    // colSel.invertColumnSelection(0, alignment.getWidth());
+    colSel.invertColumnSelection(0, alignment.getWidth());
   }
 
   @Override
@@ -1629,7 +1640,7 @@ public abstract class AlignmentViewport implements AlignViewportI,
   @Override
   public CigarArray getViewAsCigars(boolean selectedRegionOnly)
   {
-    return new CigarArray(alignment, posProps.getColumnSelection(),
+    return new CigarArray(alignment, colSel,
             (selectedRegionOnly ? selectionGroup : null));
   }
 
@@ -1644,8 +1655,8 @@ public abstract class AlignmentViewport implements AlignViewportI,
   public jalview.datamodel.AlignmentView getAlignmentView(
           boolean selectedOnly, boolean markGroups)
   {
-    return new AlignmentView(alignment, posProps.getColumnSelection(),
-            selectionGroup, posProps.hasHiddenColumns(), selectedOnly,
+    return new AlignmentView(alignment, colSel, selectionGroup,
+            colSel != null && colSel.hasHiddenColumns(), selectedOnly,
             markGroups);
   }
 
@@ -1689,9 +1700,9 @@ public abstract class AlignmentViewport implements AlignViewportI,
     }
 
     selection = new String[iSize];
-    if (posProps.hasHiddenColumns())
+    if (colSel != null && colSel.hasHiddenColumns())
     {
-      selection = posProps.getVisibleSequenceStrings(start, end, seqs);
+      selection = colSel.getVisibleSequenceStrings(start, end, seqs);
     }
     else
     {
@@ -1707,52 +1718,49 @@ public abstract class AlignmentViewport implements AlignViewportI,
   @Override
   public List<int[]> getVisibleRegionBoundaries(int min, int max)
   {
-    return posProps.getVisibleRegionBoundaries(min, max);
-    /*    ArrayList<int[]> regions = new ArrayList<int[]>();
-        int start = min;
-        int end = max;
+    ArrayList<int[]> regions = new ArrayList<int[]>();
+    int start = min;
+    int end = max;
 
-        do
+    do
+    {
+      if (colSel != null && colSel.hasHiddenColumns())
+      {
+        if (start == 0)
         {
-          if (colSel != null && colSel.hasHiddenColumns())
-          {
-            if (start == 0)
-            {
-              start = colSel.adjustForHiddenColumns(start);
-            }
-
-            end = colSel.getHiddenBoundaryRight(start);
-            if (start == end)
-            {
-              end = max;
-            }
-            if (end > max)
-            {
-              end = max;
-            }
-          }
+          start = colSel.adjustForHiddenColumns(start);
+        }
 
-          regions.add(new int[] { start, end });
+        end = colSel.getHiddenBoundaryRight(start);
+        if (start == end)
+        {
+          end = max;
+        }
+        if (end > max)
+        {
+          end = max;
+        }
+      }
 
-          if (posProps.hasHiddenColumns())
-          {
-            start = colSel.adjustForHiddenColumns(end);
-            start = colSel.getHiddenBoundaryLeft(start) + 1;
-          }
-        } while (end < max);
+      regions.add(new int[] { start, end });
+
+      if (colSel != null && colSel.hasHiddenColumns())
+      {
+        start = colSel.adjustForHiddenColumns(end);
+        start = colSel.getHiddenBoundaryLeft(start) + 1;
+      }
+    } while (end < max);
 
-        int[][] startEnd = new int[regions.size()][2];
+    int[][] startEnd = new int[regions.size()][2];
 
-        return regions;*/
+    return regions;
   }
 
   @Override
   public List<AlignmentAnnotation> getVisibleAlignmentAnnotation(
           boolean selectedOnly)
   {
-    return posProps.getVisibleAlignmentAnnotation(selectedOnly,
-            selectionGroup);
-    /*ArrayList<AlignmentAnnotation> ala = new ArrayList<AlignmentAnnotation>();
+    ArrayList<AlignmentAnnotation> ala = new ArrayList<AlignmentAnnotation>();
     AlignmentAnnotation[] aa;
     if ((aa = alignment.getAlignmentAnnotation()) != null)
     {
@@ -1771,7 +1779,7 @@ public abstract class AlignmentViewport implements AlignViewportI,
         ala.add(clone);
       }
     }
-    return ala;*/
+    return ala;
   }
 
   @Override
@@ -1844,7 +1852,7 @@ public abstract class AlignmentViewport implements AlignViewportI,
     ResidueShaderI rs = residueShading;
     if (rs != null)
     {
-      rs.alignmentChanged(alignment, posProps.getHiddenRepSequences());
+      rs.alignmentChanged(alignment, hiddenRepSequences);
 
       rs.setConsensus(hconsensus);
       if (rs.conservationApplied())
@@ -1859,7 +1867,7 @@ public abstract class AlignmentViewport implements AlignViewportI,
     {
       if (sg.cs != null)
       {
-        sg.cs.alignmentChanged(sg, posProps.getHiddenRepSequences());
+        sg.cs.alignmentChanged(sg, hiddenRepSequences);
       }
       sg.recalcConservation();
     }
@@ -2800,11 +2808,7 @@ public abstract class AlignmentViewport implements AlignViewportI,
    */
   public void expandColSelection(SequenceGroup sg, boolean wholewidth)
   {
-    if (!this.hasSelectedColumns())
-    {
-      posProps.expandColSelection(sg, wholewidth);
-    }
-    /*int sgs, sge;
+    int sgs, sge;
     if (sg != null && (sgs = sg.getStartRes()) >= 0
             && sg.getStartRes() <= (sge = sg.getEndRes())
             && !this.hasSelectedColumns())
@@ -2822,7 +2826,7 @@ public abstract class AlignmentViewport implements AlignViewportI,
       {
         colSel.addElement(cspos);
       }
-    }*/
+    }
   }
 
   /**
index 4f67cc6..1bb49af 100644 (file)
 package jalview.viewmodel;
 
 import jalview.api.ViewStyleI;
-import jalview.datamodel.AlignmentAnnotation;
 import jalview.datamodel.AlignmentI;
-import jalview.datamodel.ColumnSelection;
-import jalview.datamodel.SequenceCollectionI;
-import jalview.datamodel.SequenceGroup;
-import jalview.datamodel.SequenceI;
-
-import java.util.ArrayList;
-import java.util.Hashtable;
-import java.util.List;
-import java.util.Map;
 
 /**
  * Supplies and updates viewport properties relating to position such as: start
@@ -64,14 +54,6 @@ public class ViewportPositionProps extends ViewportProperties
   // viewstyle
   private ViewStyleI viewstyle;
 
-  // hidden column data
-  protected ColumnSelection colSel = new ColumnSelection();
-
-  private long colselhash = -1;
-
-  // hidden sequence data
-  private Map<SequenceI, SequenceCollectionI> hiddenRepSequences;
-
   /**
    * Constructor
    * @param alignment TODO
@@ -227,6 +209,7 @@ public class ViewportPositionProps extends ViewportProperties
   public int convertPixelsToResidues(int x)
   {
     return Math.round((float) x / viewstyle.getCharWidth());
+    // return (int) ((float) x / viewstyle.getCharWidth());
   }
 
   /**
@@ -239,6 +222,7 @@ public class ViewportPositionProps extends ViewportProperties
   public int convertPixelsToSequences(int y)
   {
     return Math.round((float) y / viewstyle.getCharHeight());
+    // return (int) ((float) y / viewstyle.getCharHeight());
   }
   
   /**
@@ -264,389 +248,4 @@ public class ViewportPositionProps extends ViewportProperties
   {
     return (r * viewstyle.getCharWidth());
   }
-
-  public void setHiddenColumns(ColumnSelection colsel)
-  {
-    this.colSel = colsel;
-  }
-
-  public ColumnSelection getColumnSelection()
-  {
-    return colSel;
-  }
-
-  public void setColumnSelection(ColumnSelection colSel)
-  {
-    this.colSel = colSel;
-    if (colSel != null)
-    {
-      // updateHiddenColumns(); - does nothing
-    }
-    isColSelChanged(true);
-  }
-
-  public boolean hasHiddenColumns()
-  {
-    return colSel != null && colSel.hasHiddenColumns();
-  }
-
-  /**
-   * checks current colsel against record of last hash value, and optionally
-   * updates record.
-   * 
-   * @param b
-   *          update the record of last hash value
-   * @return true if colsel changed since last call (when b is true)
-   */
-  public boolean isColSelChanged(boolean b)
-  {
-    int hc = (colSel == null || colSel.isEmpty()) ? -1 : colSel.hashCode();
-    if (hc != -1 && hc != colselhash)
-    {
-      if (b)
-      {
-        colselhash = hc;
-      }
-      return true;
-    }
-    return false;
-  }
-
-  public void hideColumns(int start, int end)
-  {
-    if (start == end)
-    {
-      colSel.hideColumns(start);
-    }
-    else
-    {
-      colSel.hideColumns(start, end);
-    }
-    isColSelChanged(true);
-  }
-
-  public void showColumn(int col)
-  {
-    colSel.revealHiddenColumns(col);
-    isColSelChanged(true);
-  }
-
-  public void showAllHiddenColumns()
-  {
-    colSel.revealAllHiddenColumns();
-    isColSelChanged(true);
-  }
-
-  public void invertColumnSelection()
-  {
-    colSel.invertColumnSelection(0, al.getWidth());
-  }
-
-  public List<int[]> getVisibleRegionBoundaries(int min, int max)
-  {
-    ArrayList<int[]> regions = new ArrayList<int[]>();
-    int start = min;
-    int end = max;
-
-    do
-    {
-      if (hasHiddenColumns())
-      {
-        if (start == 0)
-        {
-          start = colSel.adjustForHiddenColumns(start);
-        }
-
-        end = colSel.getHiddenBoundaryRight(start);
-        if (start == end)
-        {
-          end = max;
-        }
-        if (end > max)
-        {
-          end = max;
-        }
-      }
-
-      regions.add(new int[] { start, end });
-
-      if (hasHiddenColumns())
-      {
-        start = colSel.adjustForHiddenColumns(end);
-        start = colSel.getHiddenBoundaryLeft(start) + 1;
-      }
-    } while (end < max);
-
-    int[][] startEnd = new int[regions.size()][2];
-
-    return regions;
-  }
-
-  /**
-   * synthesize a column selection if none exists so it covers the given
-   * selection group. if wholewidth is false, no column selection is made if the
-   * selection group covers the whole alignment width.
-   * 
-   * @param sg
-   * @param wholewidth
-   */
-  public void expandColSelection(SequenceGroup sg, boolean wholewidth)
-  {
-    int sgs, sge;
-    if (sg != null && (sgs = sg.getStartRes()) >= 0
-            && sg.getStartRes() <= (sge = sg.getEndRes()))
-    {
-      if (!wholewidth && al.getWidth() == (1 + sge - sgs))
-      {
-        // do nothing
-        return;
-      }
-      if (colSel == null)
-      {
-        colSel = new ColumnSelection();
-      }
-      for (int cspos = sg.getStartRes(); cspos <= sg.getEndRes(); cspos++)
-      {
-        colSel.addElement(cspos);
-      }
-    }
-  }
-
-  public List<AlignmentAnnotation> getVisibleAlignmentAnnotation(
-          boolean selectedOnly, SequenceGroup selectionGroup)
-  {
-    ArrayList<AlignmentAnnotation> ala = new ArrayList<AlignmentAnnotation>();
-    AlignmentAnnotation[] aa;
-    if ((aa = al.getAlignmentAnnotation()) != null)
-    {
-      for (AlignmentAnnotation annot : aa)
-      {
-        AlignmentAnnotation clone = new AlignmentAnnotation(annot);
-        if (selectedOnly && selectionGroup != null)
-        {
-          colSel.makeVisibleAnnotation(selectionGroup.getStartRes(),
-                  selectionGroup.getEndRes(), clone);
-        }
-        else
-        {
-          colSel.makeVisibleAnnotation(clone);
-        }
-        ala.add(clone);
-      }
-    }
-    return ala;
-  }
-
-  public String[] getVisibleSequenceStrings(int start, int end,
-          SequenceI[] seqs)
-  {
-    return colSel.getVisibleSequenceStrings(start, end, seqs);
-  }
-
-  /**
-   * Set visibility for any annotations for the given sequence.
-   * 
-   * @param sequenceI
-   */
-  protected void setSequenceAnnotationsVisible(SequenceI sequenceI,
-          boolean visible)
-  {
-    AlignmentAnnotation[] anns = al.getAlignmentAnnotation();
-    if (anns != null)
-    {
-      for (AlignmentAnnotation ann : anns)
-      {
-        if (ann.sequenceRef == sequenceI)
-        {
-          ann.visible = visible;
-        }
-      }
-    }
-  }
-
-  public void setHiddenRepSequences(
-          Map<SequenceI, SequenceCollectionI> hiddenRepSequences)
-  {
-    this.hiddenRepSequences = hiddenRepSequences;
-  }
-
-  public Map<SequenceI, SequenceCollectionI> getHiddenRepSequences()
-  {
-    return hiddenRepSequences;
-  }
-
-  // common hide/show seq stuff
-  public SequenceGroup showAllHiddenSeqs(SequenceGroup selectionGroup)
-  {
-    if (al.getHiddenSequences().getSize() > 0)
-    {
-      if (selectionGroup == null)
-      {
-        selectionGroup = new SequenceGroup();
-        selectionGroup.setEndRes(al.getWidth() - 1);
-      }
-      List<SequenceI> tmp = al.getHiddenSequences().showAll(
-              hiddenRepSequences);
-      for (SequenceI seq : tmp)
-      {
-        selectionGroup.addSequence(seq, false);
-        setSequenceAnnotationsVisible(seq, true);
-      }
-
-      hiddenRepSequences = null;
-
-      firePropertyChange("alignment", null, al.getSequences());
-      // used to set hasHiddenRows/hiddenRepSequences here, after the property
-      // changed event
-      sendSelection();
-    }
-  }
-
-  public void showSequence(int index, SequenceGroup selectionGroup)
-  {
-    List<SequenceI> tmp = al.getHiddenSequences().showSequence(index,
-            hiddenRepSequences);
-    if (tmp.size() > 0)
-    {
-      if (selectionGroup == null)
-      {
-        selectionGroup = new SequenceGroup();
-        selectionGroup.setEndRes(al.getWidth() - 1);
-      }
-
-      for (SequenceI seq : tmp)
-      {
-        selectionGroup.addSequence(seq, false);
-        setSequenceAnnotationsVisible(seq, true);
-      }
-      firePropertyChange("alignment", null, al.getSequences());
-      sendSelection();
-    }
-  }
-
-  public void hideAllSelectedSeqs(SequenceGroup selectionGroup)
-  {
-    if (selectionGroup == null || selectionGroup.getSize() < 1)
-    {
-      return;
-    }
-
-    SequenceI[] seqs = selectionGroup.getSequencesInOrder(al);
-
-    hideSequence(seqs);
-
-    setSelectionGroup(null);
-  }
-
-  public void hideSequence(SequenceI[] seq)
-  {
-    if (seq != null)
-    {
-      for (int i = 0; i < seq.length; i++)
-      {
-        al.getHiddenSequences().hideSequence(seq[i]);
-        setSequenceAnnotationsVisible(seq[i], false);
-      }
-      firePropertyChange("alignment", null, al.getSequences());
-    }
-  }
-
-  /**
-   * Hides the specified sequence, or the sequences it represents
-   * 
-   * @param sequence
-   *          the sequence to hide, or keep as representative
-   * @param representGroup
-   *          if true, hide the current selection group except for the
-   *          representative sequence
-   */
-  public void hideSequences(SequenceI sequence, boolean representGroup,
-          SequenceGroup selectionGroup)
-  {
-    if (selectionGroup == null || selectionGroup.getSize() < 1)
-    {
-      hideSequence(new SequenceI[] { sequence });
-      return;
-    }
-
-    if (representGroup)
-    {
-      hideRepSequences(sequence, selectionGroup);
-      setSelectionGroup(null);
-      return;
-    }
-
-    int gsize = selectionGroup.getSize();
-    SequenceI[] hseqs = selectionGroup.getSequences().toArray(
-            new SequenceI[gsize]);
-
-    hideSequence(hseqs);
-    setSelectionGroup(null);
-    sendSelection();
-  }
-
-  public void hideRepSequences(SequenceI repSequence, SequenceGroup sg)
-  {
-    int sSize = sg.getSize();
-    if (sSize < 2)
-    {
-      return;
-    }
-
-    if (hiddenRepSequences == null)
-    {
-      hiddenRepSequences = new Hashtable<SequenceI, SequenceCollectionI>();
-    }
-
-    hiddenRepSequences.put(repSequence, sg);
-
-    // Hide all sequences except the repSequence
-    SequenceI[] seqs = new SequenceI[sSize - 1];
-    int index = 0;
-    for (int i = 0; i < sSize; i++)
-    {
-      if (sg.getSequenceAt(i) != repSequence)
-      {
-        if (index == sSize - 1)
-        {
-          return;
-        }
-
-        seqs[index++] = sg.getSequenceAt(i);
-      }
-    }
-    sg.setSeqrep(repSequence); // note: not done in 2.7applet
-    sg.setHidereps(true); // note: not done in 2.7applet
-    hideSequence(seqs);
-
-  }
-
-  /**
-   * 
-   * @param seq
-   * @return true if there are sequences represented by this sequence that are
-   *         currently hidden
-   */
-  public boolean isHiddenRepSequence(SequenceI seq)
-  {
-    return (hiddenRepSequences != null && hiddenRepSequences
-            .containsKey(seq));
-  }
-
-  /**
-   * 
-   * @param seq
-   * @return null or a sequence group containing the sequences that seq
-   *         represents
-   */
-  public SequenceGroup getRepresentedSequences(SequenceI seq)
-  {
-    return (SequenceGroup) (hiddenRepSequences == null ? null
-            : hiddenRepSequences.get(seq));
-  }
-
-  public int adjustForHiddenSeqs(int alignmentIndex)
-  {
-    return al.getHiddenSequences().adjustForHiddenSeqs(alignmentIndex);
-  }
 }