X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fappletgui%2FSeqCanvas.java;h=5fb956c610b8d80d48a2b5cc33598873c1cfe361;hb=b39bccea66aeb6a34d49649e9215a59dd5be4e27;hp=dc7911d3f268b0c53169923658cd878f6a4ca221;hpb=6d50f49d0e32b01c57733c4b6e5ece33aa9664af;p=jalview.git

diff --git a/src/jalview/appletgui/SeqCanvas.java b/src/jalview/appletgui/SeqCanvas.java
index dc7911d..5fb956c 100755
--- a/src/jalview/appletgui/SeqCanvas.java
+++ b/src/jalview/appletgui/SeqCanvas.java
@@ -1,6 +1,6 @@
 /*
  * Jalview - A Sequence Alignment Editor and Viewer
- * Copyright (C) 2006 AM Waterhouse, J Procter, G Barton, M Clamp, S Searle
+ * Copyright (C) 2007 AM Waterhouse, J Procter, G Barton, M Clamp, S Searle
  *
  * This program is free software; you can redistribute it and/or
  * modify it under the terms of the GNU General Public License
@@ -39,11 +39,9 @@ public class SeqCanvas
 
   boolean fastPaint = false;
 
-
   int cursorX = 0;
   int cursorY = 0;
 
-
   public SeqCanvas(AlignViewport av)
   {
     this.av = av;
@@ -62,18 +60,11 @@ public class SeqCanvas
     return fr;
   }
 
-  MCview.AppletPDBCanvas pdbCanvas;
   public SequenceRenderer getSequenceRenderer()
   {
     return sr;
   }
 
-  public void setPDBCanvas(MCview.AppletPDBCanvas pc)
-  {
-    pdbCanvas = pc;
-  }
-
-
   void drawNorthScale(Graphics g, int startx, int endx, int ypos)
   {
     int scalestartx = startx - startx % 10 + 10;
@@ -84,16 +75,18 @@ public class SeqCanvas
     for (int i = scalestartx; i < endx; i += 10)
     {
       int value = i;
-      if(av.hasHiddenColumns)
-          value = av.getColumnSelection().adjustForHiddenColumns(value);
+      if (av.hasHiddenColumns)
+      {
+        value = av.getColumnSelection().adjustForHiddenColumns(value);
+      }
 
-      g.drawString( String.valueOf(value), (i - startx - 1) * av.charWidth,
-          ypos - (av.charHeight / 2));
+      g.drawString(String.valueOf(value), (i - startx - 1) * av.charWidth,
+                   ypos - (av.charHeight / 2));
 
-      g.drawLine(((i - startx - 1) * av.charWidth) + (av.charWidth / 2),
-          (ypos + 2) - (av.charHeight / 2),
-          ((i - startx - 1) * av.charWidth) + (av.charWidth / 2), ypos -
-                2);
+      g.drawLine( ( (i - startx - 1) * av.charWidth) + (av.charWidth / 2),
+                 (ypos + 2) - (av.charHeight / 2),
+                 ( (i - startx - 1) * av.charWidth) + (av.charWidth / 2), ypos -
+                 2);
     }
   }
 
@@ -109,88 +102,92 @@ public class SeqCanvas
 
     int maxwidth = av.alignment.getWidth();
     if (av.hasHiddenColumns)
-        maxwidth = av.getColumnSelection().findColumnPosition(maxwidth) - 1;
+    {
+      maxwidth = av.getColumnSelection().findColumnPosition(maxwidth) - 1;
+    }
 
-      // WEST SCALE
-      for (int i = 0; i < av.alignment.getHeight(); i++)
-      {
-        SequenceI seq = av.alignment.getSequenceAt(i);
-        int index = startx;
-        int value = -1;
+    // WEST SCALE
+    for (int i = 0; i < av.alignment.getHeight(); i++)
+    {
+      SequenceI seq = av.alignment.getSequenceAt(i);
+      int index = startx;
+      int value = -1;
 
-        while (index < endx)
+      while (index < endx)
+      {
+        if (jalview.util.Comparison.isGap(seq.getCharAt(index)))
         {
-          if (jalview.util.Comparison.isGap(seq.getCharAt(index)))
-          {
-            index++;
+          index++;
 
-            continue;
-          }
+          continue;
+        }
 
-          value = av.alignment.getSequenceAt(i).findPosition(index);
+        value = av.alignment.getSequenceAt(i).findPosition(index);
 
-          break;
-        }
+        break;
+      }
 
-        if (value != -1)
-        {
-          int x = LABEL_WEST - fm.stringWidth(String.valueOf(value)) -
-              av.charWidth / 2;
-          g.drawString(value + "", x,
-                       (ypos + (i * av.charHeight)) - (av.charHeight / 5));
-        }
+      if (value != -1)
+      {
+        int x = LABEL_WEST - fm.stringWidth(String.valueOf(value)) -
+            av.charWidth / 2;
+        g.drawString(value + "", x,
+                     (ypos + (i * av.charHeight)) - (av.charHeight / 5));
       }
     }
+  }
+
   void drawEastScale(Graphics g, int startx, int endx, int ypos)
   {
-        ypos += av.charHeight;
+    ypos += av.charHeight;
 
-        if(av.hasHiddenColumns)
-                endx = av.getColumnSelection().adjustForHiddenColumns(endx);
+    if (av.hasHiddenColumns)
+    {
+      endx = av.getColumnSelection().adjustForHiddenColumns(endx);
+    }
 
-        SequenceI seq;
-        // EAST SCALE
-        for (int i = 0; i < av.alignment.getHeight(); i++)
-        {
-            seq = av.alignment.getSequenceAt(i);
-            int index = endx;
-            int value = -1;
+    SequenceI seq;
+    // EAST SCALE
+    for (int i = 0; i < av.alignment.getHeight(); i++)
+    {
+      seq = av.alignment.getSequenceAt(i);
+      int index = endx;
+      int value = -1;
 
-            while (index > startx)
-            {
-                if (jalview.util.Comparison.isGap(seq.getCharAt(index)))
-                {
-                    index--;
+      while (index > startx)
+      {
+        if (jalview.util.Comparison.isGap(seq.getCharAt(index)))
+        {
+          index--;
 
-                    continue;
-                }
+          continue;
+        }
 
-                value = seq.findPosition(index);
+        value = seq.findPosition(index);
 
-                break;
-            }
+        break;
+      }
 
-            if (value != -1)
-            {
-                g.drawString(String.valueOf(value), 0,
-                    (ypos + (i * av.charHeight)) - (av.charHeight / 5));
-            }
-        }
+      if (value != -1)
+      {
+        g.drawString(String.valueOf(value), 0,
+                     (ypos + (i * av.charHeight)) - (av.charHeight / 5));
+      }
     }
+  }
 
-  int lastsr=0;
+  int lastsr = 0;
   void fastPaint(int horizontal, int vertical)
   {
-    if ( fastPaint || gg == null)
+    if (fastPaint || gg == null)
     {
       return;
     }
 
-
     // Its possible on certain browsers that the call to fastpaint
     // is faster than it can paint, so this check here catches
     // this possibility
-    if(lastsr + horizontal != av.startRes)
+    if (lastsr + horizontal != av.startRes)
     {
       horizontal = av.startRes - lastsr;
     }
@@ -205,8 +202,6 @@ public class SeqCanvas
                 -horizontal * av.charWidth,
                 -vertical * av.charHeight);
 
-
-
     int sr = av.startRes, er = av.endRes, ss = av.startSeq, es = av.endSeq,
         transX = 0, transY = 0;
 
@@ -307,7 +302,6 @@ public class SeqCanvas
     gg.setColor(Color.white);
     gg.fillRect(0, 0, imgWidth, imgHeight);
 
-
     if (av.getWrapAlignment())
     {
       drawWrappedPanel(gg, imgWidth, imgHeight, av.startRes);
@@ -319,36 +313,31 @@ public class SeqCanvas
 
     g.drawImage(img, 0, 0, this);
 
-    if (pdbCanvas != null)
-    {
-      pdbCanvas.updateSeqColours();
-    }
   }
 
   int LABEL_WEST, LABEL_EAST;
   public int getWrappedCanvasWidth(int cwidth)
   {
-      cwidth -= cwidth % av.charWidth;
+    cwidth -= cwidth % av.charWidth;
 
-      FontMetrics fm = getFontMetrics(av.getFont());
+    FontMetrics fm = getFontMetrics(av.getFont());
 
-      LABEL_EAST = 0;
-      LABEL_WEST = 0;
+    LABEL_EAST = 0;
+    LABEL_WEST = 0;
 
-      if (av.scaleRightWrapped)
-      {
-          LABEL_EAST = fm.stringWidth(getMask());
-      }
+    if (av.scaleRightWrapped)
+    {
+      LABEL_EAST = fm.stringWidth(getMask());
+    }
 
-      if (av.scaleLeftWrapped)
-      {
-          LABEL_WEST = fm.stringWidth(getMask());
-      }
+    if (av.scaleLeftWrapped)
+    {
+      LABEL_WEST = fm.stringWidth(getMask());
+    }
 
-      return (cwidth - LABEL_EAST - LABEL_WEST) / av.charWidth;
+    return (cwidth - LABEL_EAST - LABEL_WEST) / av.charWidth;
   }
 
-
   /**
    * Generates a string of zeroes.
    * @return String
@@ -362,7 +351,9 @@ public class SeqCanvas
     {
       tmp = av.alignment.getSequenceAt(i).getEnd();
       if (tmp > maxWidth)
+      {
         maxWidth = tmp;
+      }
     }
 
     for (int i = maxWidth; i > 0; i /= 10)
@@ -379,20 +370,21 @@ public class SeqCanvas
 
     FontMetrics fm = getFontMetrics(av.getFont());
 
-
     if (av.scaleRightWrapped)
     {
-        LABEL_EAST = fm.stringWidth(getMask());
+      LABEL_EAST = fm.stringWidth(getMask());
     }
 
     if (av.scaleLeftWrapped)
     {
-        LABEL_WEST = fm.stringWidth(getMask());
+      LABEL_WEST = fm.stringWidth(getMask());
     }
 
     int hgap = av.charHeight;
-    if(av.scaleAboveWrapped)
+    if (av.scaleAboveWrapped)
+    {
       hgap += av.charHeight;
+    }
 
     int cWidth = (canvasWidth - LABEL_EAST - LABEL_WEST) / av.charWidth;
     int cHeight = av.getAlignment().getHeight() * av.charHeight;
@@ -401,396 +393,427 @@ public class SeqCanvas
 
     av.endRes = av.startRes + cWidth;
 
-
     int endx;
     int ypos = hgap;
 
-    int maxwidth = av.alignment.getWidth();
+    int maxwidth = av.alignment.getWidth() - 1;
 
-    if(av.hasHiddenColumns)
-          maxwidth = av.getColumnSelection().findColumnPosition(maxwidth)-1;
+    if (av.hasHiddenColumns)
+    {
+      maxwidth = av.getColumnSelection().findColumnPosition(maxwidth) - 1;
+    }
 
-    while ((ypos <= canvasHeight) && (startRes < maxwidth))
+    while ( (ypos <= canvasHeight) && (startRes < maxwidth))
     {
-      endx = startRes + cWidth -1;
+      endx = startRes + cWidth - 1;
 
       if (endx > maxwidth)
       {
         endx = maxwidth;
       }
 
-        g.setColor(Color.black);
+      g.setColor(Color.black);
 
-        if (av.scaleLeftWrapped)
-        {
-            drawWestScale(g, startRes, endx, ypos);
-        }
+      if (av.scaleLeftWrapped)
+      {
+        drawWestScale(g, startRes, endx, ypos);
+      }
 
-        if (av.scaleRightWrapped)
-        {
-            g.translate(canvasWidth - LABEL_EAST, 0);
-            drawEastScale(g, startRes, endx, ypos);
-            g.translate(-(canvasWidth - LABEL_EAST), 0);
-        }
+      if (av.scaleRightWrapped)
+      {
+        g.translate(canvasWidth - LABEL_EAST, 0);
+        drawEastScale(g, startRes, endx, ypos);
+        g.translate( - (canvasWidth - LABEL_EAST), 0);
+      }
 
-        g.translate(LABEL_WEST, 0);
+      g.translate(LABEL_WEST, 0);
 
-        if (av.scaleAboveWrapped)
-        {
-          drawNorthScale(g, startRes, endx, ypos);
-        }
-        if (av.hasHiddenColumns && av.showHiddenMarkers)
+      if (av.scaleAboveWrapped)
+      {
+        drawNorthScale(g, startRes, endx, ypos);
+      }
+      if (av.hasHiddenColumns && av.showHiddenMarkers)
+      {
+        g.setColor(Color.blue);
+        int res;
+        for (int i = 0; i < av.getColumnSelection().getHiddenColumns().size();
+             i++)
         {
-          g.setColor(Color.blue);
-          int res;
-          for (int i = 0; i < av.getColumnSelection().getHiddenColumns().size();
-               i++)
-          {
-            res = av.getColumnSelection().findHiddenRegionPosition(i) -
-                startRes;
-
-            if (res < 0 || res > endx - startRes)
-              continue;
-
-            gg.fillPolygon(new int[]
-                           {res * av.charWidth - av.charHeight / 4,
-                           res * av.charWidth + av.charHeight / 4,
-                           res * av.charWidth},
-                           new int[]
-                           {
-                           ypos - (av.charHeight / 2),
-                           ypos - (av.charHeight / 2),
-                           ypos - (av.charHeight / 2) + 8
-            }, 3);
+          res = av.getColumnSelection().findHiddenRegionPosition(i) -
+              startRes;
 
+          if (res < 0 || res > endx - startRes)
+          {
+            continue;
           }
-        }
 
-        if(g.getClip()==null)
-          g.setClip(0, 0, cWidth * av.charWidth, canvasHeight);
+          gg.fillPolygon(new int[]
+                         {res * av.charWidth - av.charHeight / 4,
+                         res * av.charWidth + av.charHeight / 4,
+                         res * av.charWidth},
+                         new int[]
+                         {
+                         ypos - (av.charHeight / 2),
+                         ypos - (av.charHeight / 2),
+                         ypos - (av.charHeight / 2) + 8
+          }, 3);
+
+        }
+      }
 
-        drawPanel(g, startRes, endx, 0, al.getHeight(), ypos);
-         g.setClip(null);
+      if (g.getClip() == null)
+      {
+        g.setClip(0, 0, cWidth * av.charWidth, canvasHeight);
+      }
 
+      drawPanel(g, startRes, endx, 0, al.getHeight(), ypos);
+      g.setClip(null);
 
-        if(av.showAnnotation)
+      if (av.showAnnotation)
+      {
+        g.translate(0, cHeight + ypos + 4);
+        if (annotations == null)
         {
-          g.translate(0, cHeight + ypos+4);
-          if(annotations==null)
-            annotations = new AnnotationPanel(av);
-
-          annotations.drawComponent( g, startRes, endx+1 );
-          g.translate(0, -cHeight - ypos-4);
+          annotations = new AnnotationPanel(av);
         }
-        g.translate(-LABEL_WEST, 0);
 
-        ypos += cHeight+getAnnotationHeight()+hgap;
+        annotations.drawComponent(g, startRes, endx + 1);
+        g.translate(0, -cHeight - ypos - 4);
+      }
+      g.translate( -LABEL_WEST, 0);
 
+      ypos += cHeight + getAnnotationHeight() + hgap;
 
-        startRes += cWidth;
-        }
+      startRes += cWidth;
+    }
 
   }
 
   AnnotationPanel annotations;
   int getAnnotationHeight()
   {
-    if(!av.showAnnotation)
+    if (!av.showAnnotation)
+    {
       return 0;
+    }
 
-    if(annotations==null)
+    if (annotations == null)
+    {
       annotations = new AnnotationPanel(av);
+    }
 
     return annotations.adjustPanelHeight();
-    }
+  }
 
-    void drawPanel(Graphics g1, int startRes, int endRes,
-                    int startSeq, int endSeq, int offset)
+  void drawPanel(Graphics g1, int startRes, int endRes,
+                 int startSeq, int endSeq, int offset)
+  {
+    if (!av.hasHiddenColumns)
+    {
+      draw(g1, startRes, endRes, startSeq, endSeq, offset);
+    }
+    else
     {
-      if(!av.hasHiddenColumns)
+      java.util.Vector regions = av.getColumnSelection().getHiddenColumns();
+
+      int screenY = 0;
+      int blockStart = startRes;
+      int blockEnd = endRes;
+
+      for (int i = 0; i < regions.size(); i++)
       {
-        draw(g1, startRes, endRes, startSeq, endSeq, offset);
+        int[] region = (int[]) regions.elementAt(i);
+        int hideStart = region[0];
+        int hideEnd = region[1];
+
+        if (hideStart <= blockStart)
+        {
+          blockStart += (hideEnd - hideStart) + 1;
+          continue;
+        }
+
+        blockEnd = hideStart - 1;
+
+        g1.translate(screenY * av.charWidth, 0);
+
+        draw(g1, blockStart, blockEnd, startSeq, endSeq, offset);
+
+        if (av.getShowHiddenMarkers())
+        {
+          g1.setColor(Color.blue);
+          g1.drawLine( (blockEnd - blockStart + 1) * av.charWidth - 1,
+                      0 + offset,
+                      (blockEnd - blockStart + 1) * av.charWidth - 1,
+                      (endSeq - startSeq) * av.charHeight + offset);
+        }
+
+        g1.translate( -screenY * av.charWidth, 0);
+        screenY += blockEnd - blockStart + 1;
+        blockStart = hideEnd + 1;
       }
-      else
+
+      if (screenY <= (endRes - startRes))
       {
-        java.util.Vector regions = av.getColumnSelection().getHiddenColumns();
+        blockEnd = blockStart + (endRes - startRes) - screenY;
+        g1.translate(screenY * av.charWidth, 0);
+        draw(g1, blockStart, blockEnd, startSeq, endSeq, offset);
 
-        int screenY = 0;
-        int blockStart = startRes;
-        int blockEnd = endRes;
+        g1.translate( -screenY * av.charWidth, 0);
+      }
+    }
 
-        for (int i = 0; i < regions.size(); i++)
-        {
-          int[] region = (int[]) regions.elementAt(i);
-          int hideStart = region[0];
-          int hideEnd = region[1];
+  }
 
-          if (hideStart <= blockStart)
-          {
-            blockStart += (hideEnd - hideStart) + 1;
-            continue;
-          }
+  //int startRes, int endRes, int startSeq, int endSeq, int x, int y,
+  // int x1, int x2, int y1, int y2, int startx, int starty,
+  void draw(Graphics g,
+            int startRes, int endRes,
+            int startSeq, int endSeq,
+            int offset)
+  {
+    g.setFont(av.getFont());
+    sr.prepare(g, av.renderGaps);
 
-          blockEnd = hideStart - 1;
+    SequenceI nextSeq;
 
-          g1.translate(screenY * av.charWidth, 0);
+    /// First draw the sequences
+    /////////////////////////////
+    for (int i = startSeq; i < endSeq; i++)
+    {
+      nextSeq = av.alignment.getSequenceAt(i);
 
-          draw(g1, blockStart, blockEnd, startSeq, endSeq, offset);
+      if (nextSeq == null)
+      {
+        continue;
+      }
 
-          if(av.getShowHiddenMarkers())
-          {
-            g1.setColor(Color.blue);
-            g1.drawLine( (blockEnd - blockStart + 1) * av.charWidth - 1,
-                         0 + offset,
-                         (blockEnd - blockStart + 1) * av.charWidth - 1,
-                         (endSeq - startSeq) * av.charHeight + offset);
-          }
+      sr.drawSequence(nextSeq, av.alignment.findAllGroups(nextSeq),
+                      startRes, endRes,
+                      offset + ( (i - startSeq) * av.charHeight));
 
-          g1.translate( -screenY * av.charWidth, 0);
-          screenY += blockEnd - blockStart + 1;
-          blockStart = hideEnd + 1;
-        }
+      if (av.showSequenceFeatures)
+      {
+        fr.drawSequence(g, nextSeq, startRes, endRes,
+                        offset + ( (i - startSeq) * av.charHeight));
+      }
 
-        if (screenY <= (endRes - startRes))
+      /// Highlight search Results once all sequences have been drawn
+      //////////////////////////////////////////////////////////
+      if (searchResults != null)
+      {
+        int[] visibleResults = searchResults.getResults(nextSeq, startRes,
+            endRes);
+        if (visibleResults != null)
         {
-          blockEnd = blockStart + (endRes - startRes) - screenY;
-          g1.translate(screenY * av.charWidth, 0);
-          draw(g1, blockStart, blockEnd, startSeq, endSeq, offset);
-
-          g1.translate( -screenY * av.charWidth, 0);
+          for (int r = 0; r < visibleResults.length; r += 2)
+          {
+            sr.drawHighlightedText(nextSeq, visibleResults[r],
+                                   visibleResults[r + 1],
+                                   (visibleResults[r] - startRes) *
+                                   av.charWidth,
+                                   offset + ( (i - startSeq) * av.charHeight));
+          }
         }
       }
 
+      if (av.cursorMode && cursorY == i
+          && cursorX >= startRes && cursorX <= endRes)
+      {
+        sr.drawCursor(nextSeq, cursorX, (cursorX - startRes) * av.charWidth,
+                      offset + ( (i - startSeq) * av.charHeight));
+      }
     }
 
+    if (av.getSelectionGroup() != null || av.alignment.getGroups().size() > 0)
+    {
+      drawGroupsBoundaries(g, startRes, endRes, startSeq, endSeq, offset);
+    }
 
+  }
 
+  void drawGroupsBoundaries(Graphics g,
+                            int startRes, int endRes,
+                            int startSeq, int endSeq,
+                            int offset)
+  {
+    //
+    /////////////////////////////////////
+    // Now outline any areas if necessary
+    /////////////////////////////////////
+    SequenceGroup group = av.getSelectionGroup();
+
+    int sx = -1;
+    int sy = -1;
+    int ex = -1;
+    int groupIndex = -1;
+
+    if ( (group == null) && (av.alignment.getGroups().size() > 0))
+    {
+      group = (SequenceGroup) av.alignment.getGroups().elementAt(0);
+      groupIndex = 0;
+    }
 
-    //int startRes, int endRes, int startSeq, int endSeq, int x, int y,
-    // int x1, int x2, int y1, int y2, int startx, int starty,
-    void draw(Graphics g,
-                   int startRes, int endRes,
-                   int startSeq, int endSeq,
-                   int offset)
-   {
-      g.setFont(av.getFont());
-      sr.prepare(g, av.renderGaps);
-
-      SequenceI nextSeq;
-
-        /// First draw the sequences
-        /////////////////////////////
-        for (int i = startSeq; i < endSeq; i++)
+    if (group != null)
+    {
+      do
+      {
+        int oldY = -1;
+        int i = 0;
+        boolean inGroup = false;
+        int top = -1;
+        int bottom = -1;
+        int alHeight = av.alignment.getHeight() - 1;
+
+        for (i = startSeq; i < endSeq; i++)
         {
-            nextSeq = av.alignment.getSequenceAt(i);
-
-            if(nextSeq==null)
-              continue;
+          sx = (group.getStartRes() - startRes) * av.charWidth;
+          sy = offset + ( (i - startSeq) * av.charHeight);
+          ex = ( ( (group.getEndRes() + 1) - group.getStartRes()) *
+                av.charWidth) -
+              1;
 
-            sr.drawSequence(nextSeq, av.alignment.findAllGroups(nextSeq),
-                            startRes, endRes,
-                            offset + ( (i - startSeq) * av.charHeight));
+          if (sx + ex < 0 || sx > imgWidth)
+          {
+            continue;
+          }
 
-            if (av.showSequenceFeatures)
+          if ( (sx <= (endRes - startRes) * av.charWidth) &&
+              group.getSequences(null).
+              contains(av.alignment.getSequenceAt(i)))
+          {
+            if ( (bottom == -1) &&
+                (i >= alHeight ||
+                 !group.getSequences(null).contains(
+                     av.alignment.getSequenceAt(i + 1))))
             {
-                fr.drawSequence(g, nextSeq, startRes, endRes,
-                    offset + ((i - startSeq) * av.charHeight));
+              bottom = sy + av.charHeight;
             }
 
-            /// Highlight search Results once all sequences have been drawn
-            //////////////////////////////////////////////////////////
-            if (searchResults != null)
+            if (!inGroup)
             {
-              int[] visibleResults = searchResults.getResults(nextSeq, startRes, endRes);
-              if (visibleResults != null)
-                for (int r = 0; r < visibleResults.length; r += 2)
-                {
-                  sr.drawHighlightedText(nextSeq, visibleResults[r],
-                                         visibleResults[r + 1],
-                                         (visibleResults[r] - startRes) * av.charWidth,
-                                         offset + ( (i - startSeq) * av.charHeight));
-                }
+              if ( ( (top == -1) && (i == 0)) ||
+                  !group.getSequences(null).contains(
+                      av.alignment.getSequenceAt(i - 1)))
+              {
+                top = sy;
+              }
+
+              oldY = sy;
+              inGroup = true;
+
+              if (group == av.getSelectionGroup())
+              {
+                g.setColor(Color.red);
+              }
+              else
+              {
+                g.setColor(group.getOutlineColour());
+              }
             }
-
-            if(av.cursorMode && cursorY==i
-               && cursorX>=startRes && cursorX<=endRes)
+          }
+          else
+          {
+            if (inGroup)
             {
-              sr.drawCursor(nextSeq, cursorX, (cursorX - startRes) * av.charWidth,
-                            offset + ( (i - startSeq) * av.charHeight));
+              if (sx >= 0 && sx < imgWidth)
+              {
+                g.drawLine(sx, oldY, sx, sy);
+              }
+
+              if (sx + ex < imgWidth)
+              {
+                g.drawLine(sx + ex, oldY, sx + ex, sy);
+              }
+
+              if (sx < 0)
+              {
+                ex += sx;
+                sx = 0;
+              }
+
+              if (sx + ex > imgWidth)
+              {
+                ex = imgWidth;
+              }
+
+              else if (sx + ex >= (endRes - startRes + 1) * av.charWidth)
+              {
+                ex = (endRes - startRes + 1) * av.charWidth;
+              }
+
+              if (top != -1)
+              {
+                g.drawLine(sx, top, sx + ex, top);
+                top = -1;
+              }
+
+              if (bottom != -1)
+              {
+                g.drawLine(sx, bottom, sx + ex, bottom);
+                bottom = -1;
+              }
+
+              inGroup = false;
             }
           }
+        }
 
-          if(av.getSelectionGroup()!=null || av.alignment.getGroups().size()>0)
-            drawGroupsBoundaries(g, startRes, endRes, startSeq, endSeq, offset);
-
-   }
-
-   void drawGroupsBoundaries(Graphics g,
-                   int startRes, int endRes,
-                   int startSeq, int endSeq,
-                   int offset)
-   {
-     //
-     /////////////////////////////////////
-     // Now outline any areas if necessary
-     /////////////////////////////////////
-     SequenceGroup group = av.getSelectionGroup();
-
-     int sx = -1;
-     int sy = -1;
-     int ex = -1;
-     int groupIndex = -1;
-
-     if ( (group == null) && (av.alignment.getGroups().size() > 0))
-     {
-       group = (SequenceGroup) av.alignment.getGroups().elementAt(0);
-       groupIndex = 0;
-     }
-
-     if (group != null)
-     {
-       do
-       {
-         int oldY = -1;
-         int i = 0;
-         boolean inGroup = false;
-         int top = -1;
-         int bottom = -1;
-         int alHeight = av.alignment.getHeight()-1;
-
-         for (i = startSeq; i < endSeq; i++)
-         {
-           sx = (group.getStartRes() - startRes) * av.charWidth;
-           sy = offset + ( (i - startSeq) * av.charHeight);
-           ex = ( ( (group.getEndRes() + 1) - group.getStartRes()) * av.charWidth) -
-               1;
-
-           if (sx + ex < 0 || sx > imgWidth)
-           {
-             continue;
-           }
-
-           if ( (sx <= (endRes - startRes) * av.charWidth) &&
-               group.getSequences(false).
-               contains(av.alignment.getSequenceAt(i)))
-           {
-             if ( (bottom == -1) &&
-             (i >= alHeight ||
-                 !group.getSequences(false).contains(
-                     av.alignment.getSequenceAt(i + 1))))
-             {
-               bottom = sy + av.charHeight;
-             }
-
-             if (!inGroup)
-             {
-               if ( ( (top == -1) && (i == 0)) ||
-                   !group.getSequences(false).contains(
-                       av.alignment.getSequenceAt(i - 1)))
-               {
-                 top = sy;
-               }
-
-               oldY = sy;
-               inGroup = true;
-
-               if (group == av.getSelectionGroup())
-               {
-                 g.setColor(Color.red);
-               }
-               else
-               {
-                 g.setColor(group.getOutlineColour());
-               }
-             }
-           }
-           else
-           {
-             if (inGroup)
-             {
-               if (sx >= 0 && sx < imgWidth)
-                 g.drawLine(sx, oldY, sx, sy);
-
-               if (sx + ex < imgWidth)
-                 g.drawLine(sx + ex, oldY, sx + ex, sy);
-
-               if (sx < 0)
-               {
-                 ex += sx;
-                 sx = 0;
-               }
-
-               if (sx + ex > imgWidth)
-                 ex = imgWidth;
-
-               else if (sx + ex >= (endRes - startRes + 1) * av.charWidth)
-                 ex = (endRes - startRes + 1) * av.charWidth;
-
-               if (top != -1)
-               {
-                 g.drawLine(sx, top, sx + ex, top);
-                 top = -1;
-               }
-
-               if (bottom != -1)
-               {
-                 g.drawLine(sx, bottom, sx + ex, bottom);
-                 bottom = -1;
-               }
-
-               inGroup = false;
-             }
-           }
-         }
-
-         if (inGroup)
-         {
-           sy = offset + ( (i - startSeq) * av.charHeight);
-           if (sx >= 0 && sx < imgWidth)
-             g.drawLine(sx, oldY, sx, sy);
-
-           if (sx + ex < imgWidth)
-             g.drawLine(sx + ex, oldY, sx + ex, sy);
-
-           if (sx < 0)
-           {
-             ex += sx;
-             sx = 0;
-           }
-
-           if (sx + ex > imgWidth)
-             ex = imgWidth;
-           else if (sx + ex >= (endRes - startRes + 1) * av.charWidth)
-             ex = (endRes - startRes + 1) * av.charWidth;
-
-           if (top != -1)
-           {
-             g.drawLine(sx, top, sx + ex, top);
-             top = -1;
-           }
-
-           if (bottom != -1)
-           {
-             g.drawLine(sx, bottom - 1, sx + ex, bottom - 1);
-             bottom = -1;
-           }
-
-           inGroup = false;
-         }
-
-         groupIndex++;
-
-         if (groupIndex >= av.alignment.getGroups().size())
-         {
-           break;
-         }
-
-         group = (SequenceGroup) av.alignment.getGroups().elementAt(groupIndex);
-       }
-       while (groupIndex < av.alignment.getGroups().size());
-
-     }
-   }
+        if (inGroup)
+        {
+          sy = offset + ( (i - startSeq) * av.charHeight);
+          if (sx >= 0 && sx < imgWidth)
+          {
+            g.drawLine(sx, oldY, sx, sy);
+          }
+
+          if (sx + ex < imgWidth)
+          {
+            g.drawLine(sx + ex, oldY, sx + ex, sy);
+          }
+
+          if (sx < 0)
+          {
+            ex += sx;
+            sx = 0;
+          }
+
+          if (sx + ex > imgWidth)
+          {
+            ex = imgWidth;
+          }
+          else if (sx + ex >= (endRes - startRes + 1) * av.charWidth)
+          {
+            ex = (endRes - startRes + 1) * av.charWidth;
+          }
+
+          if (top != -1)
+          {
+            g.drawLine(sx, top, sx + ex, top);
+            top = -1;
+          }
+
+          if (bottom != -1)
+          {
+            g.drawLine(sx, bottom - 1, sx + ex, bottom - 1);
+            bottom = -1;
+          }
+
+          inGroup = false;
+        }
+
+        groupIndex++;
+
+        if (groupIndex >= av.alignment.getGroups().size())
+        {
+          break;
+        }
+
+        group = (SequenceGroup) av.alignment.getGroups().elementAt(groupIndex);
+      }
+      while (groupIndex < av.alignment.getGroups().size());
+
+    }
+  }
 
   public void highlightSearchResults(SearchResults results)
   {