X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fappletgui%2FSeqPanel.java;h=5e2f24323d8320bf074727dba52c064ac0afcf04;hb=174230b4233d9ce80f94527768d2cd2f76da11ab;hp=d7f11bb23ab187b989850f672c450877e422f167;hpb=b90206d60dbcb530a3eeeb00b4802ba0a6d8bad2;p=jalview.git

diff --git a/src/jalview/appletgui/SeqPanel.java b/src/jalview/appletgui/SeqPanel.java
index d7f11bb..5e2f243 100755
--- a/src/jalview/appletgui/SeqPanel.java
+++ b/src/jalview/appletgui/SeqPanel.java
@@ -1,6 +1,6 @@
 /*
  * Jalview - A Sequence Alignment Editor and Viewer
- * Copyright (C) 2005 AM Waterhouse, J Procter, G Barton, M Clamp, S Searle
+ * Copyright (C) 2006 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
@@ -19,16 +19,16 @@
 
 package jalview.appletgui;
 
-import java.util.*;
-
 import java.awt.*;
 import java.awt.event.*;
 
 import jalview.datamodel.*;
 import jalview.schemes.*;
 
+import java.util.Vector;
+
 public class SeqPanel
-    extends Panel
+    extends Panel implements MouseMotionListener, MouseListener
 {
 
   public SeqCanvas seqCanvas;
@@ -36,20 +36,30 @@ public class SeqPanel
 
   protected int lastres;
   protected int startseq;
-  int startEdit = -1;
-  int endEdit = -1;
 
   protected AlignViewport av;
 
   // if character is inserted or deleted, we will need to recalculate the conservation
-  int seqEditOccurred = -1;
+  boolean seqEditOccurred = false;
 
   ScrollThread scrollThread = null;
   boolean mouseDragging = false;
-
   boolean editingSeqs = false;
   boolean groupEditing = false;
 
+  int oldSeq = -1;
+  boolean changeEndSeq = false;
+  boolean changeStartSeq = false;
+  boolean changeEndRes = false;
+  boolean changeStartRes = false;
+  SequenceGroup stretchGroup = null;
+
+  StringBuffer keyboardNo1;
+  StringBuffer keyboardNo2;
+
+  boolean mouseWheelPressed = false;
+  Point lastMousePress;
+
   public SeqPanel(AlignViewport avp, AlignmentPanel p)
   {
     this.av = avp;
@@ -60,96 +70,323 @@ public class SeqPanel
 
     ap = p;
 
-    seqCanvas.addMouseMotionListener(new MouseMotionAdapter()
-    {
-      public void mouseMoved(MouseEvent evt)
-      {
-        doMouseMoved(evt);
-      }
+    seqCanvas.addMouseMotionListener(this);
+    seqCanvas.addMouseListener(this);
 
-      public void mouseDragged(MouseEvent evt)
-      {
-        if (editingSeqs)
-        {
-          doMouseDragged(evt);
-        }
-        else
-        {
-          doMouseDraggedDefineMode(evt);
-        }
-      }
-    });
+    seqCanvas.repaint();
+  }
 
-    seqCanvas.addMouseListener(new MouseAdapter()
-    {
-      public void mouseReleased(MouseEvent evt)
-      {
-        if (editingSeqs)
-        {
-          doMouseReleased(evt);
-        }
-        else
-        {
-          doMouseReleasedDefineMode(evt);
-        }
+  void endEditing()
+  {
+    startseq = -1;
+    lastres = -1;
+    seqEditOccurred = false;
+    editingSeqs = false;
+    groupEditing = false;
+    keyboardNo1 = null;
+    keyboardNo2 = null;
+   }
 
-      }
+   void setCursorRow()
+   {
+     seqCanvas.cursorY = getKeyboardNo(keyboardNo1)-1;
+     scrollToVisible();
+   }
 
-      public void mousePressed(MouseEvent evt)
-      {
-        if (evt.isShiftDown() || evt.isAltDown() || evt.isControlDown())
-        {
-          if (evt.isAltDown() || evt.isControlDown())
-          {
-            groupEditing = true;
-          }
+   void setCursorColumn()
+   {
+     seqCanvas.cursorX = getKeyboardNo(keyboardNo1)-1;
+     scrollToVisible();
+   }
 
-          editingSeqs = true;
-          doMousePressed(evt);
-        }
-        else
-        {
-          doMousePressedDefineMode(evt);
-        }
-      }
+   void setCursorRowAndColumn()
+   {
+     if(keyboardNo2==null)
+     {
+       keyboardNo2 = new StringBuffer();
+     }
+     else
+     {
+       seqCanvas.cursorX = getKeyboardNo(keyboardNo1) - 1;
+       seqCanvas.cursorY = getKeyboardNo(keyboardNo2) - 1;
+       scrollToVisible();
+     }
+   }
 
-      public void mouseExited(MouseEvent evt)
-      {
-        if (editingSeqs)
-        {
-          return;
-        }
-        doMouseExitedDefineMode(evt);
+   void setCursorPosition()
+   {
+     SequenceI sequence =
+         (Sequence) av.getAlignment().getSequenceAt(seqCanvas.cursorY);
 
-      }
+     seqCanvas.cursorX = sequence.findIndex(
+         getKeyboardNo(keyboardNo1)-1
+         );
+     scrollToVisible();
+   }
 
-      public void mouseEntered(MouseEvent evt)
-      {
-        if (editingSeqs)
-        {
-          return;
-        }
-        doMouseEnteredDefineMode(evt);
-      }
+   void moveCursor(int dx, int dy)
+   {
+     seqCanvas.cursorX += dx;
+     seqCanvas.cursorY += dy;
+     scrollToVisible();
+   }
 
-    });
-    seqCanvas.repaint();
-  }
+   void scrollToVisible()
+   {
+     if (seqCanvas.cursorX < 0)
+       seqCanvas.cursorX = 0;
+     else if (seqCanvas.cursorX > av.alignment.getWidth() - 1)
+       seqCanvas.cursorX = av.alignment.getWidth() - 1;
+
+     if (seqCanvas.cursorY < 0)
+       seqCanvas.cursorY = 0;
+     else if (seqCanvas.cursorY > av.alignment.getHeight() - 1)
+       seqCanvas.cursorY = av.alignment.getHeight() - 1;
 
-  public void doMouseReleased(MouseEvent evt)
+
+     endEditing();
+     if (av.wrapAlignment)
+     {
+       ap.scrollToWrappedVisible(seqCanvas.cursorX);
+     }
+     else
+     {
+       while (seqCanvas.cursorY < av.startSeq)
+       {
+         ap.scrollUp(true);
+       }
+       while (seqCanvas.cursorY + 1 > av.endSeq)
+       {
+         ap.scrollUp(false);
+       }
+       while (seqCanvas.cursorX < av.startRes)
+       {
+
+         if (!ap.scrollRight(false))
+           break;
+       }
+       while (seqCanvas.cursorX > av.endRes)
+       {
+         if (!ap.scrollRight(true))
+           break;
+       }
+     }
+     setStatusMessage(av.alignment.getSequenceAt(seqCanvas.cursorY),
+                      seqCanvas.cursorX, seqCanvas.cursorY);
+
+     seqCanvas.repaint();
+   }
+
+   void setSelectionAreaAtCursor(boolean topLeft)
+   {
+     SequenceI sequence =
+         (Sequence) av.getAlignment().getSequenceAt(seqCanvas.cursorY);
+
+     if(av.getSelectionGroup()!=null)
+     {
+       SequenceGroup sg = av.selectionGroup;
+       //Find the top and bottom of this group
+       int min = av.alignment.getHeight(), max = 0;
+       for(int i=0; i<sg.getSize(false); i++)
+       {
+         int index = av.alignment.findIndex( sg.getSequenceAt(i) );
+         if(index > max)
+           max = index;
+         if(index < min)
+           min = index;
+       }
+
+       max ++;
+
+       if(topLeft)
+       {
+         sg.setStartRes(seqCanvas.cursorX);
+         if(sg.getEndRes()<seqCanvas.cursorX)
+           sg.setEndRes(seqCanvas.cursorX);
+
+         min = seqCanvas.cursorY;
+       }
+       else
+       {
+         sg.setEndRes(seqCanvas.cursorX);
+         if(sg.getStartRes()>seqCanvas.cursorX)
+           sg.setStartRes(seqCanvas.cursorX);
+
+         max = seqCanvas.cursorY+1;
+       }
+
+       if(min>max)
+       {
+         // Only the user can do this
+         av.setSelectionGroup(null);
+       }
+       else
+       {
+         // Now add any sequences between min and max
+         sg.getSequences(false).removeAllElements();
+         for (int i = min; i < max; i++)
+         {
+           sg.addSequence(av.alignment.getSequenceAt(i), false);
+         }
+       }
+     }
+
+     if (av.getSelectionGroup() == null)
+     {
+       SequenceGroup sg = new SequenceGroup();
+       sg.setStartRes(seqCanvas.cursorX);
+       sg.setEndRes(seqCanvas.cursorX);
+       sg.addSequence(sequence, false);
+       av.setSelectionGroup(sg);
+     }
+
+
+     ap.repaint();
+   }
+
+   void insertGapAtCursor(boolean group)
+   {
+     ap.alignFrame.addHistoryItem(new HistoryItem("Edit Sequence",
+                                                  av.alignment, HistoryItem.EDIT));
+     groupEditing = group;
+     startseq = seqCanvas.cursorY;
+     lastres = seqCanvas.cursorX;
+     editSequence(true, seqCanvas.cursorX+getKeyboardNo(keyboardNo1));
+     editOccurred();
+   }
+
+   void deleteGapAtCursor(boolean group)
+   {
+     ap.alignFrame.addHistoryItem(new HistoryItem("Edit Sequence",
+                                                  av.alignment, HistoryItem.EDIT));
+     groupEditing = group;
+     startseq = seqCanvas.cursorY;
+     lastres = seqCanvas.cursorX+getKeyboardNo(keyboardNo1);
+     editSequence(false, seqCanvas.cursorX);
+     editOccurred();
+   }
+
+   void numberPressed(char value)
+   {
+     if(keyboardNo1==null)
+       keyboardNo1 = new StringBuffer();
+
+     if(keyboardNo2!=null)
+       keyboardNo2.append(value);
+     else
+       keyboardNo1.append(value);
+   }
+
+   int getKeyboardNo(StringBuffer kb)
+   {
+     if(kb==null)
+       return 1;
+     else
+       return Integer.parseInt(kb.toString());
+   }
+
+   void setStatusMessage(SequenceI sequence, int res, int seq)
+   {
+     StringBuffer text = new StringBuffer("Sequence " + (seq + 1) + " ID: " +
+                                          sequence.getName());
+
+     Object obj = null;
+     if (av.alignment.isNucleotide())
+     {
+       obj = ResidueProperties.nucleotideName.get(sequence.getCharAt(res) +
+                                                  "");
+       if (obj != null)
+         text.append(" Nucleotide: ");
+     }
+     else
+     {
+       obj = ResidueProperties.aa2Triplet.get(sequence.getCharAt(res) + "");
+       if (obj != null)
+         text.append("  Residue: ");
+     }
+
+     if (obj != null)
+     {
+
+       if (obj != "")
+       {
+         text.append(obj + " (" + sequence.findPosition(res) +
+                     ")");
+       }
+     }
+
+     ap.alignFrame.statusBar.setText(text.toString());
+
+    }
+     public void mousePressed(MouseEvent evt)
+     {
+       lastMousePress = evt.getPoint();
+
+       if ( (evt.getModifiers() & InputEvent.BUTTON2_MASK) ==
+         InputEvent.BUTTON2_MASK)
+       {
+         mouseWheelPressed = true;
+         return;
+       }
+
+       if (evt.isShiftDown() || evt.isAltDown() ||
+           evt.isControlDown())
+       {
+         if (evt.isAltDown() || evt.isControlDown())
+         {
+           groupEditing = true;
+         }
+         editingSeqs = true;
+       }
+       else
+       {
+         doMousePressedDefineMode(evt);
+         return;
+       }
+
+
+       int seq = findSeq(evt);
+       int res = findRes(evt);
+
+       if(seq<0 || res<0)
+         return;
+
+       ap.alignFrame.addHistoryItem(new HistoryItem("Edit Sequence",
+                                                          av.alignment, HistoryItem.EDIT));
+
+         if ((seq < av.getAlignment().getHeight()) &&
+                 (res < av.getAlignment().getSequenceAt(seq).getLength()))
+         {
+             startseq = seq;
+             lastres = res;
+         }
+         else
+         {
+             startseq = -1;
+             lastres = -1;
+         }
+
+        return;
+     }
+
+     public void mouseClicked(MouseEvent evt){}
+
+
+  public void mouseReleased(MouseEvent evt)
   {
+    mouseDragging = false;
+    mouseWheelPressed = false;
 
-    if (seqEditOccurred > -1)
+    if (!editingSeqs)
     {
-      editOccurred(seqEditOccurred);
+       doMouseReleasedDefineMode(evt);
+       return;
     }
 
-    startseq = -1;
-    lastres = -1;
-    seqEditOccurred = -1;
-    editingSeqs = false;
-    groupEditing = false;
-    ap.repaint();
+     editOccurred();
+
+     endEditing();
+     ap.repaint();
   }
 
   int startWrapBlock=-1;
@@ -161,40 +398,67 @@ public class SeqPanel
 
   if (av.wrapAlignment)
   {
+
+    int hgap = av.charHeight;
+    if (av.scaleAboveWrapped)
+      hgap += av.charHeight;
+
+    int cHeight = av.getAlignment().getHeight() * av.charHeight
+        + hgap + seqCanvas.getAnnotationHeight();
+
       int y = evt.getY();
-      y -= (2 * av.charHeight);
+      y -= hgap;
       x -= seqCanvas.LABEL_WEST;
 
-      int chunkHeight = (av.getAlignment().getHeight() + 2) * av.charHeight;
-      int cwidth = seqCanvas.getWrappedCanvasWidth(this.getSize().width);
-      wrappedBlock = y/chunkHeight;
-      wrappedBlock += av.getStartRes()/cwidth;
 
-      res = wrappedBlock*cwidth   +   x / av.getCharWidth();
+      int cwidth = seqCanvas.getWrappedCanvasWidth(getSize().width);
+
+      wrappedBlock = y / cHeight;
+      wrappedBlock += av.getStartRes() / cwidth;
+
+      res = wrappedBlock * cwidth + x / av.getCharWidth();
+
   }
   else
   {
       res = (x / av.getCharWidth()) + av.getStartRes();
   }
 
+  if(av.hasHiddenColumns)
+          res = av.getColumnSelection().adjustForHiddenColumns(res);
+
   return res;
 
  }
 
  int findSeq(MouseEvent evt)
  {
+
    int seq = 0;
    int y = evt.getY();
 
    if (av.wrapAlignment)
    {
-     y -= (2 * av.charHeight);
-     int chunkHeight = (av.getAlignment().getHeight() + 2) * av.charHeight;
-     seq = ( (y % chunkHeight) / av.getCharHeight());
+     int hgap = av.charHeight;
+     if (av.scaleAboveWrapped)
+       hgap += av.charHeight;
+
+     int cHeight = av.getAlignment().getHeight() * av.charHeight
+         + hgap + seqCanvas.getAnnotationHeight();
+
+       y -= hgap;
+
+     seq = Math.min( (y % cHeight) / av.getCharHeight(),
+                     av.alignment.getHeight() -1);
+     if(seq<0)
+       seq = 0;
    }
    else
    {
-     seq = (y / av.getCharHeight()) + av.getStartSeq();
+     seq = Math.min( (y / av.getCharHeight()) + av.getStartSeq(),
+                     av.alignment.getHeight() -1);
+     if(seq<0)
+       seq = 0;
    }
 
    return seq;
@@ -228,7 +492,7 @@ public class SeqPanel
     return;
   }
 
-  public void doMouseMoved(MouseEvent evt)
+  public void mouseMoved(MouseEvent evt)
   {
     int res = findRes(evt);
     int seq = findSeq(evt);
@@ -244,149 +508,433 @@ public class SeqPanel
       return;
     }
 
-    Object obj = ResidueProperties.aa2Triplet.get(sequence.getCharAt(res) + "");
-    String aa = "";
+    StringBuffer text = new StringBuffer("Sequence " + (seq + 1) + " ID: " +
+            sequence.getName());
+
+    Object obj = null;
+    if (av.alignment.isNucleotide())
+    {
+      obj = ResidueProperties.nucleotideName.get(sequence.getCharAt(res) +
+          "");
+      if(obj!=null)
+        text.append(" Nucleotide: ");
+    }
+    else
+    {
+      obj = ResidueProperties.aa2Triplet.get(sequence.getCharAt(res) + "");
+      if(obj!=null)
+        text.append("  Residue: ");
+    }
+
     if (obj != null)
     {
-      aa = obj.toString();
+      if (obj != "")
+      {
+        text.append(obj + " (" + sequence.findPosition(res) + ")");
+      }
     }
 
-    StringBuffer text = new StringBuffer("Sequence " + (seq + 1) + " ID: " +
-                                         sequence.getName());
-    if (aa != "")
+    if(seqCanvas.pdbCanvas!=null && sequence==seqCanvas.pdbCanvas.sequence)
     {
-      text.append("  Residue: " + aa + " (" +
-                  av.getAlignment().getSequenceAt(seq).findPosition(res) + ")");
+      seqCanvas.pdbCanvas.highlightRes(sequence.findPosition(res));
     }
 
     ap.alignFrame.statusBar.setText(text.toString());
 
+
     // use aa to see if the mouse pointer is on a
-    if (av.showSequenceFeatures)
+    if (av.showSequenceFeatures
+        && sequence.getSequenceFeatures()!=null
+        && av.featuresDisplayed!=null)
     {
-      Vector features = sequence.getSequenceFeatures();
-      Enumeration e = features.elements();
-      StringBuffer sbuffer = new StringBuffer();
+      StringBuffer featureText = new StringBuffer();
+      Vector allFeatures = getAllFeaturesAtRes(sequence, sequence.findPosition(res));
 
-      while (e.hasMoreElements())
+      int index = 0;
+      while (index < allFeatures.size())
       {
-        SequenceFeature sf = (SequenceFeature) e.nextElement();
-        if (sf.getStart() <= sequence.findPosition(res) &&
-            sf.getEnd() >= sequence.findPosition(res))
+        SequenceFeature sf = (SequenceFeature) allFeatures.elementAt(index);
+
+        featureText.append(sf.getType()+" "+sf.begin+":"+sf.end);
+
+        if (sf.getDescription() != null)
+          featureText.append(" " + sf.getDescription());
+
+        if (sf.getValue("status") != null )
         {
-          if (sbuffer.length() > 0)
-          {
-            sbuffer.append("; ");
-          }
-          sbuffer.append(sf.getType() + " " + sf.getDescription());
-          if (sf.getStatus().length() > 0)
-          {
-            sbuffer.append(" (" + sf.getStatus() + ")");
-          }
+          featureText.append(" (" + sf.getValue("status") + ")");
         }
+        featureText.append("\n");
 
+        index++;
       }
-    }
 
+
+        if (tooltip == null)
+          tooltip = new Tooltip(featureText.toString(), seqCanvas);
+        else
+          tooltip.setTip(featureText.toString());
+
+        tooltip.repaint();
+
+    }
   }
 
-  public void doMouseDragged(MouseEvent evt)
+  Vector getAllFeaturesAtRes(SequenceI seq, int res)
   {
-
-    // If we're dragging we're editing
-    int res = findRes(evt);
-    if (res < 0)
+    Vector allFeatures = new Vector();
+    int index = 0;
+    if(seq.getSequenceFeatures()!=null)
     {
-      res = 0;
+      while (index < seq.getSequenceFeatures().length)
+      {
+        SequenceFeature sf = seq.getSequenceFeatures()[index];
+        if (sf.getBegin() <= res &&
+            sf.getEnd() >= res)
+        {
+          if (av.featuresDisplayed.containsKey(sf.getType()))
+          {
+            allFeatures.addElement(sf);
+          }
+        }
+        index++;
+      }
     }
+    return allFeatures;
+  }
+
+  Tooltip tooltip;
 
-    if (lastres == -1 || lastres == res)
+  public void mouseDragged(MouseEvent evt)
+  {
+    if (mouseWheelPressed)
     {
+      int oldWidth = av.charWidth;
+
+      //Which is bigger, left-right or up-down?
+      if (Math.abs(evt.getY() - lastMousePress.y)
+          > Math.abs(evt.getX() - lastMousePress.x))
+      {
+        int fontSize = av.font.getSize();
+
+        if (evt.getY() < lastMousePress.y && av.charHeight > 1)
+        {
+          fontSize--;
+        }
+        else if (evt.getY() > lastMousePress.y)
+        {
+          fontSize++;
+        }
+
+
+        if(fontSize<1)
+          fontSize = 1;
+
+        av.setFont(new Font(av.font.getName(), av.font.getStyle(), fontSize));
+        av.charWidth = oldWidth;
+      }
+      else
+      {
+        if (evt.getX() < lastMousePress.x && av.charWidth > 1)
+        {
+          av.charWidth--;
+        }
+        else if (evt.getX() > lastMousePress.x)
+        {
+          av.charWidth++;
+        }
+
+        if(av.charWidth<1)
+        {
+          av.charWidth = 1;
+        }
+      }
+
+      ap.fontChanged();
+
+      FontMetrics fm = getFontMetrics(av.getFont());
+      av.validCharWidth = fm.charWidth('M') <= av.charWidth;
+
+      lastMousePress = evt.getPoint();
+
+      ap.repaint();
+      ap.annotationPanel.image = null;
       return;
-    }
+      }
 
-    boolean dragRight = true;
-    if (res < av.getAlignment().getWidth() && res < lastres)
-    {
-      dragRight = false;
-    }
+      if (!editingSeqs)
+      {
+        doMouseDraggedDefineMode(evt);
+        return;
+      }
+
+  int res = findRes(evt);
+
+  if (res < 0)
+  {
+      res = 0;
+  }
+
+  if ((lastres == -1) || (lastres == res))
+  {
+      return;
+  }
+
+  if ( (res < av.getAlignment().getWidth()) && (res < lastres))
+  {
+    // dragLeft, delete gap
+    editSequence(false, res);
+  }
+  else
+    editSequence(true, res);
+
+  mouseDragging = true;
+  if(scrollThread!=null)
+    scrollThread.setEvent(evt);
+
+  }
+
+  synchronized void editSequence(boolean insertGap, int startres)
+  {
+    int fixedLeft = -1;
+    int fixedRight = -1;
+    boolean fixedColumns = false;
+    SequenceGroup sg = av.getSelectionGroup();
+
+
+      if (!groupEditing && av.hasHiddenRows)
+      {
+        if (av.alignment.getSequenceAt(startseq).getHiddenSequences() != null)
+        {
+          groupEditing = true;
+        }
+      }
+
+      //No group, but the sequence may represent a group
+      if (groupEditing
+          && sg == null
+          && av.alignment.getSequenceAt(startseq).getHiddenSequences() == null)
+      {
+        groupEditing = false;
+      }
+
+      SequenceI seq = av.alignment.getSequenceAt(startseq);
+      StringBuffer message = new StringBuffer();
+      if (groupEditing)
+         message.append("Edit group:");
+      else
+         message.append("Edit sequence: "+seq.getName());
+
+     if(insertGap)
+       message.append(" insert ");
+     else
+       message.append(" delete ");
+
+     message.append(Math.abs(startres-lastres)+" gaps.");
+     ap.alignFrame.statusBar.setText(message.toString());
+
+
+      //Are we editing within a selection group?
+      if (groupEditing
+          || (sg != null && sg.getSequences(true).contains(seq)))
+      {
+        fixedColumns = true;
+
+        //sg might be null as the user may only see 1 sequence,
+        //but the sequence represents a group
+        if (sg == null)
+        {
+          sg = new SequenceGroup(null, null, false, false, false, 0,
+                                 av.alignment.getWidth()-1);
+          sg.addSequence(av.alignment.getSequenceAt(startseq), false);
+        }
+
+        fixedLeft = sg.getStartRes();
+        fixedRight = sg.getEndRes();
+
+        if (   (startres < fixedLeft && lastres >= fixedLeft)
+            || (startres >= fixedLeft && lastres < fixedLeft)
+            || (startres > fixedRight && lastres <=fixedRight)
+            || (startres <= fixedRight && lastres > fixedRight))
+        {
+          endEditing();
+          return;
+        }
+
+        if (fixedLeft > startres)
+        {
+          fixedRight = fixedLeft - 1;
+          fixedLeft = 0;
+        }
+        else if (fixedRight < startres)
+        {
+          fixedLeft = fixedRight;
+          fixedRight = -1;
+        }
+      }
+
+
+      if(av.hasHiddenColumns )
+      {
+          fixedColumns = true;
+          int y1 = av.getColumnSelection().getHiddenBoundaryLeft(startres);
+          int y2 = av.getColumnSelection().getHiddenBoundaryRight(startres);
+
+          if ( (insertGap && startres > y1 && lastres < y1)
+              || (!insertGap && startres < y2 && lastres > y2))
+          {
+            endEditing();
+            return;
+          }
+
+          //System.out.print(y1+" "+y2+" "+fixedLeft+" "+fixedRight+"~~");
+          //Selection spans a hidden region
+          if(fixedLeft<y1 && (fixedRight>y2 || fixedRight==-1))
+          {
+            if(startres>=y2)
+            {
+              fixedLeft = y2;
+            }
+            else
+            {
+             fixedRight = y2 - 1;
+           }
+          }
+      }
+
+
+      if (groupEditing)
+      {
+        // drag to right
+        if (insertGap)
+        {
+            //If the user has selected the whole sequence, and is dragging to
+            // the right, we can still extend the alignment and selectionGroup
+            if(   sg.getStartRes() == 0
+                  && sg.getEndRes() == fixedRight
+                  && sg.getEndRes() == av.alignment.getWidth()-1
+               )
+            {
+              sg.setEndRes(av.alignment.getWidth() + startres - lastres);
+              fixedRight = sg.getEndRes();
+            }
+
+          // Is it valid with fixed columns??
+          // Find the next gap before the end
+          // of the visible region boundary
+          boolean blank = false;
+          for (fixedRight = fixedRight;
+               fixedRight > lastres;
+               fixedRight--)
+          {
+            blank = true;
+            for (int s = 0; s < sg.getSize(true); s++)
+            {
+              seq = (SequenceI)sg.getSequences(true).elementAt(s);
+              for (int j = 0; j < startres - lastres; j++)
+              {
+                if (!jalview.util.Comparison.isGap(
+                    seq.getCharAt(fixedRight - j)))
+                {
+                  blank = false;
+                  break;
+                }
+              }
+            }
+            if (blank)
+              break;
+          }
+
+          if (!blank)
+          {
+            if(sg.getSize(false) == av.alignment.getHeight()  )
+            {
+              if((av.hasHiddenColumns
+                  && startres<av.getColumnSelection().getHiddenBoundaryRight(startres)))
+              {
+                endEditing();
+                return;
+              }
 
-    if (res != lastres)
-    {
-      // Group editing
-      if (groupEditing)
-      {
-        SequenceGroup sg = av.getSelectionGroup();
-        if (sg == null)
-        {
-          lastres = -1;
-          return;
+              int alWidth = av.alignment.getWidth();
+              if(av.hasHiddenRows)
+              {
+                int hwidth = av.alignment.getHiddenSequences().getWidth();
+                if(hwidth>alWidth)
+                  alWidth = hwidth;
+              }
+              //We can still insert gaps if the selectionGroup
+              //contains all the sequences
+              sg.setEndRes(sg.getEndRes()+startres-lastres);
+              fixedRight = alWidth+startres-lastres;
+            }
+            else
+            {
+              endEditing();
+              return;
+            }
+          }
         }
 
-        // drag to right
-        if (dragRight)
-        {
-          sg.setEndRes(sg.getEndRes() + (res - lastres));
-        }
 
         // drag to left
-        else
+        else if(!insertGap)
         {
           /// Are we able to delete?
           // ie are all columns blank?
-          boolean deleteAllowed = false;
-          for (int s = 0; s < sg.getSize(); s++)
+
+          for (int s = 0; s < sg.getSize(true); s++)
           {
-            SequenceI seq = sg.getSequenceAt(s);
-            for (int j = res; j < lastres; j++)
+            seq = (SequenceI)sg.getSequences(true).elementAt(s);
+
+            for (int j = startres; j < lastres; j++)
             {
               if (seq.getSequence().length() <= j)
               {
                 continue;
               }
 
-              if (!jalview.util.Comparison.isGap(seq.getSequence().charAt(j)))
+              if (!jalview.util.Comparison.isGap(
+                  seq.getSequence().charAt(j)))
               {
                 // Not a gap, block edit not valid
-                res = j + 1;
-                deleteAllowed = false;
-                continue;
+                endEditing();
+                return;
               }
-              deleteAllowed = true;
             }
           }
-
-          if (!deleteAllowed)
-          {
-            lastres = -1;
-            return;
-          }
-
-          sg.setEndRes(sg.getEndRes() - (lastres - res));
         }
 
-        for (int i = 0; i < sg.getSize(); i++)
+
+        for (int i = 0; i < sg.getSize(true); i++)
         {
-          SequenceI s = sg.getSequenceAt(i);
-          int k = av.alignment.findIndex(s);
+          seq = (SequenceI) sg.getSequences(true).elementAt(i);
 
-          // drag to right
-          if (dragRight)
+          if (insertGap)
           {
-            for (int j = lastres; j < res; j++)
+            // dragging to the right
+            for (int j = lastres; j < startres; j++)
             {
-              insertChar(j, k);
+              if (fixedColumns && fixedRight != -1)
+              {
+                insertChar(j, seq, fixedRight);
+              }
+              else
+                insertChar(j, seq);
             }
           }
-
-          // drag to left
           else
           {
-            for (int j = res; j < lastres; j++)
+            // dragging to the left
+            for (int j = lastres; j > startres; j--)
             {
-              if (s.getLength() > j)
+              if (fixedColumns && fixedRight != -1)
+              {
+                deleteChar(startres, seq, fixedRight);
+              }
+              else
               {
-                deleteChar(res, k);
+                deleteChar(startres, seq);
               }
             }
           }
@@ -394,183 +942,159 @@ public class SeqPanel
       }
       else /////Editing a single sequence///////////
       {
-        if (res < av.getAlignment().getWidth() && res > lastres)
+        if (insertGap)
         {
           // dragging to the right
-          for (int j = lastres; j < res; j++)
+          for (int j = lastres; j < startres; j++)
           {
-            insertChar(j, startseq);
+            if (fixedColumns && fixedRight != -1)
+            {
+                insertChar(j, seq, fixedRight);
+            }
+            else
+              insertChar(j, seq);
           }
         }
-        else if (res < av.getAlignment().getWidth() && res < lastres)
+        else
         {
           // dragging to the left
-          for (int j = lastres; j > res; j--)
+          for (int j = lastres; j > startres; j--)
           {
-            if (jalview.util.Comparison.isGap(
-                av.alignment.getSequenceAt(startseq).getSequence().charAt(res)))
+            if (fixedColumns && fixedRight != -1)
             {
-
-              deleteChar(res, startseq);
+              deleteChar(startres, seq, fixedRight);
             }
             else
             {
-
-              break;
+              deleteChar(startres, seq);
             }
           }
         }
-
       }
-    }
-
-    endEdit = res;
-    lastres = res;
-    seqCanvas.repaint();
-  }
-
-  public void drawChars(int seqstart, int seqend, int start)
-  {
-    seqCanvas.drawPanel(seqCanvas.gg, start, av.getEndRes(), seqstart, seqend,
-                        av.getStartRes(), av.getStartSeq(), 0);
-    seqCanvas.repaint();
-  }
 
-  public void insertChar(int j, int seq)
-  {
-    av.alignment.getSequenceAt(seq).insertCharAt(j, av.getGapCharacter());
-    seqEditOccurred = seq;
+      lastres = startres;
+      seqCanvas.repaint();
   }
 
-  public void deleteChar(int j, int seq)
-  {
-
-    av.alignment.getSequenceAt(seq).deleteCharAt(j);
-    seqEditOccurred = seq;
-    av.alignment.getWidth();
-    repaint();
-  }
 
-  void editOccurred(int i)
-  {
-    if (endEdit == startEdit)
+    /**
+     * DOCUMENT ME!
+     *
+     * @param j DOCUMENT ME!
+     * @param seq DOCUMENT ME!
+     */
+    void insertChar(int j, SequenceI seq)
     {
-        ap.alignFrame.historyList.pop();
-        ap.alignFrame.updateEditMenuBar();
+        seq.insertCharAt(j, av.getGapCharacter());
+        seqEditOccurred = true;
     }
 
-    av.updateConservation();
-    av.updateConsensus();
-
-
-    //Does the edit affect any groups?
-    //////////////////////////////
-    Vector groupsToUpdate = new Vector();
-    SequenceGroup sg = av.getSelectionGroup();
-
-    int s, g, gSize = av.alignment.getGroups().size();
-    if (sg != null)
+    void insertChar(int j, SequenceI seq, int fixedColumn)
     {
-      groupsToUpdate.addElement(av.getSelectionGroup());
-      int sSize = sg.getSize();
-      for (s = 0; s < sSize; s++)
+      //Find the next gap before the end of the visible region boundary
+      //If lastCol > j, theres a boundary after the gap insertion
+      int blankColumn = fixedColumn;
+      for (blankColumn = fixedColumn; blankColumn > j; blankColumn--)
       {
-        SequenceI seq = sg.getSequenceAt(s);
-
-        for (g = 0; g < gSize; g++)
+        if (jalview.util.Comparison.isGap(seq.getCharAt(blankColumn)))
         {
-          SequenceGroup sg2 = (SequenceGroup) av.alignment.getGroups().
-              elementAt(
-                  g);
-          if (sg2.sequences.contains(seq))
-          {
-            if (!groupsToUpdate.contains(sg2))
-              groupsToUpdate.addElement(sg2);
-            continue;
-          }
+          //Theres a space, so break and insert the gap
+          break;
         }
       }
-    }
-    else
-    {
-      SequenceI seq = av.alignment.getSequenceAt(startseq);
-      for (g = 0; g < gSize; g++)
+
+      if (blankColumn <= j)
       {
-        SequenceGroup sg2 = (SequenceGroup) av.alignment.getGroups().elementAt(
-            g);
-        if (sg2.sequences.contains(seq))
-        {
-          if (!groupsToUpdate.contains(sg2))
-            groupsToUpdate.addElement(sg2);
-          continue;
-        }
+        endEditing();
+        return;
       }
-    }
 
-    gSize = groupsToUpdate.size();
-    for (g = 0; g < gSize; g++)
-    {
-      ( (SequenceGroup) groupsToUpdate.elementAt(g)).recalcConservation();
+      if (!jalview.util.Comparison.isGap(seq.getCharAt(blankColumn)))
+      {
+        //Just Checking
+        System.out.println("Tried removing residue (INSERT)"+seq.getCharAt(fixedColumn));
+        return;
+      }
+
+      seq.deleteCharAt(blankColumn);
+      seq.insertCharAt(j, av.getGapCharacter());
+      seqEditOccurred = true;
     }
-    ////End of updating groups
-      //////////////////////////////////////////
 
+    void deleteChar(int j, SequenceI seq, int fixedColumn)
+    {
+      if (!jalview.util.Comparison.isGap(seq.getCharAt(j)))
+      {
+        ap.alignFrame.statusBar.setText(
+            "End editing: Tried removing residue " + seq.getCharAt(j));
+        return;
+      }
 
-    // Y O Y CLUSTALX
-    ColourSchemeI cs = av.getGlobalColourScheme();
+      seq.deleteCharAt(j);
+      seq.insertCharAt(fixedColumn, av.getGapCharacter());
+      seqEditOccurred = true;
+    }
 
-    if (av.getSelectionGroup() != null)
+    /**
+     * DOCUMENT ME!
+     *
+     * @param j DOCUMENT ME!
+     * @param seq DOCUMENT ME!
+     */
+    void deleteChar(int j, SequenceI seq)
     {
-        av.getSelectionGroup().recalcConservation();
+      if (!jalview.util.Comparison.isGap(seq.getCharAt(j)))
+      {
+        ap.alignFrame.statusBar.setText(
+            "End editing: Tried removing residue " + seq.getCharAt(j));
+        return;
+      }
+
+        seq.deleteCharAt(j);
+        seqEditOccurred = true;
+        seqCanvas.repaint();
     }
 
-    if(cs!=null)
+    /**
+     * DOCUMENT ME!
+     *
+     * @param i DOCUMENT ME!
+     */
+    void editOccurred()
     {
-      if (cs.conservationApplied())
+      if (!seqEditOccurred)
       {
-        jalview.analysis.Conservation c = new jalview.analysis.Conservation("All",
-                                          ResidueProperties.propHash, 3,
-                                          av.alignment.getSequences(),
-                                          0, av.alignment.getWidth() - 1);
-        c.calculate();
-        c.verdict(false, av.ConsPercGaps);
+        ap.alignFrame.historyList.pop();
+        ap.alignFrame.updateEditMenuBar();
+      }
 
-        if (cs instanceof ClustalxColourScheme)
-        {
-          ((ClustalxColourScheme)cs).
-              resetClustalX(av.alignment.getSequences(),
-                                av.alignment.getWidth());
-          cs.setConservation(c);
-          av.setGlobalColourScheme(cs);
-        }
-        else
-        {
-          cs.setConservation(c);
-          av.setGlobalColourScheme(cs);
-        }
+      endEditing();
 
-      }
+      av.firePropertyChange("alignment", null,av.getAlignment().getSequences());
 
-      if (cs instanceof ClustalxColourScheme)
-      {
-        ( (ClustalxColourScheme) cs).resetClustalX(av.alignment.getSequences(),
-                                                   av.alignment.getWidth());
-        av.setGlobalColourScheme(cs);
-      }
-        }
-  }
+    }
 
 //////////////////////////////////////////
 /////Everything below this is for defining the boundary of the rubberband
 //////////////////////////////////////////
-  int oldSeq = -1;
   public void doMousePressedDefineMode(MouseEvent evt)
   {
+
+
+    if (scrollThread != null)
+    {
+      scrollThread.running = false;
+      scrollThread = null;
+    }
+
     int res = findRes(evt);
     int seq = findSeq(evt);
     oldSeq = seq;
     startWrapBlock=wrappedBlock;
 
+    if(seq==-1)
+      return;
+
     SequenceI sequence = (Sequence) av.getAlignment().getSequenceAt(seq);
 
     if (sequence == null || res > sequence.getLength())
@@ -594,7 +1118,7 @@ public class SeqPanel
       }
     }
 
-    else if (!stretchGroup.sequences.contains(sequence)
+    else if (!stretchGroup.getSequences(false).contains(sequence)
              || stretchGroup.getStartRes() > res
              || stretchGroup.getEndRes() < res)
     {
@@ -610,84 +1134,99 @@ public class SeqPanel
               allGroups[i].getEndRes() >= res)
           {
             stretchGroup = allGroups[i];
-            av.setSelectionGroup(stretchGroup);
             break;
           }
         }
       }
+      av.setSelectionGroup(stretchGroup);
     }
 
-    if (stretchGroup == null)
-    {
-      // define a new group here
-      SequenceGroup sg = new SequenceGroup();
-      sg.setStartRes(res);
-      sg.setEndRes(res);
-      sg.addSequence(sequence, false);
-      av.setSelectionGroup(sg);
-      stretchGroup = sg;
-
-      if (av.getConservationSelected())
-      {
-        SliderPanel.setConservationSlider(ap, av.getGlobalColourScheme(),
-                                          "Background");
-      }
-      if (av.getAbovePIDThreshold())
-      {
-        SliderPanel.setPIDSliderSource(ap, av.getGlobalColourScheme(),
-                                       "Background");
-      }
-
-    }
 
     // DETECT RIGHT MOUSE BUTTON IN AWT
-    else if ( (evt.getModifiers() & InputEvent.BUTTON3_MASK) ==
+    if ( (evt.getModifiers() & InputEvent.BUTTON3_MASK) ==
              InputEvent.BUTTON3_MASK)
     {
-      APopupMenu popup = new APopupMenu(ap, null, null);
+      Vector allFeatures = getAllFeaturesAtRes(sequence,
+                                               sequence.findPosition(res));
+
+      Vector links = null;
+      if(allFeatures!=null)
+      {
+        for (int i = 0; i < allFeatures.size(); i++)
+        {
+          SequenceFeature sf = (SequenceFeature) allFeatures.elementAt(i);
+          if (sf.links != null)
+          {
+            links = new Vector();
+            for (int j = 0; j < sf.links.size(); j++)
+            {
+              links.addElement(sf.links.elementAt(j));
+            }
+          }
+        }
+      }
+      APopupMenu popup = new APopupMenu(ap, null, links);
       this.add(popup);
       popup.show(this, evt.getX(), evt.getY());
+      ap.repaint();
+      return;
     }
 
-    if (stretchGroup != null && stretchGroup.getEndRes() == res)
+    if (av.cursorMode)
     {
-      // Edit end res position of selected group
-      changeEndRes = true;
+      seqCanvas.cursorX = findRes(evt);
+      seqCanvas.cursorY = findSeq(evt);
+      seqCanvas.repaint();
+      return;
     }
 
-    else if (stretchGroup != null && stretchGroup.getStartRes() == res)
-    {
-      // Edit end res position of selected group
-      changeStartRes = true;
-    }
+      //Only if left mouse button do we want to change group sizes
 
-  }
+      if (stretchGroup == null)
+      {
+        // define a new group here
+        SequenceGroup sg = new SequenceGroup();
+        sg.setStartRes(res);
+        sg.setEndRes(res);
+        sg.addSequence(sequence, false);
+        av.setSelectionGroup(sg);
+        stretchGroup = sg;
+
+        if (av.getConservationSelected())
+        {
+          SliderPanel.setConservationSlider(ap, av.getGlobalColourScheme(),
+                                            "Background");
+        }
+        if (av.getAbovePIDThreshold())
+        {
+          SliderPanel.setPIDSliderSource(ap, av.getGlobalColourScheme(),
+                                         "Background");
+        }
 
-  boolean changeEndSeq = false;
-  boolean changeStartSeq = false;
-  boolean changeEndRes = false;
-  boolean changeStartRes = false;
-  SequenceGroup stretchGroup = null;
+      }
+  }
 
   public void doMouseReleasedDefineMode(MouseEvent evt)
   {
-    if(mouseDragging)
-     {
-       stretchGroup.recalcConservation();
-       mouseDragging = false;
-     }
-
     if (stretchGroup == null)
     {
-      return;
+        return;
     }
 
     if(stretchGroup.cs!=null)
     {
+      if (stretchGroup.cs instanceof ClustalxColourScheme)
+      {
+        ( (ClustalxColourScheme) stretchGroup.cs).resetClustalX(
+            stretchGroup.getSequences(true),
+            stretchGroup.getWidth());
+      }
+
       if (stretchGroup.cs.conservationApplied())
       {
         SliderPanel.setConservationSlider(ap, stretchGroup.cs,
                                           stretchGroup.getName());
+        stretchGroup.recalcConservation();
       }
       else
       {
@@ -698,126 +1237,170 @@ public class SeqPanel
     changeEndRes = false;
     changeStartRes = false;
     stretchGroup = null;
-    seqCanvas.repaint();
+    PaintRefresher.Refresh(av.alignment);
     ap.repaint();
   }
 
-  boolean remove = false;
   public void doMouseDraggedDefineMode(MouseEvent evt)
   {
     int res = findRes(evt);
     int y = findSeq(evt);
 
     if(wrappedBlock!=startWrapBlock)
-        return;
+      return;
+
+     if (stretchGroup == null)
+     {
+          return;
+     }
 
-    if(y>=av.alignment.getHeight())
-      y = av.alignment.getHeight()-1;
+     mouseDragging = true;
 
-    if (stretchGroup == null)
-    {
-      return;
-    }
 
-    if (res > av.alignment.getWidth())
-    {
-      res = av.alignment.getWidth() - 1;
-    }
+      if(y > av.alignment.getHeight())
+      {
+        y = av.alignment.getHeight() -1;
+      }
 
-    if (stretchGroup.getEndRes() == res)
-    {
-      // Edit end res position of selected group
-      changeEndRes = true;
-    }
+      if(res>av.alignment.getWidth())
+        res = av.alignment.getWidth()-1;
 
-    else if (stretchGroup.getStartRes() == res)
-    {
-      // Edit start res position of selected group
-      changeStartRes = true;
-    }
+      if (stretchGroup.getEndRes() == res)
+      {
+          // Edit end res position of selected group
+          changeEndRes = true;
+      }
+      else if (stretchGroup.getStartRes() == res)
+      {
+          // Edit start res position of selected group
+          changeStartRes = true;
+      }
 
-    if (res < av.getStartRes())
-    {
-      res = av.getStartRes();
-    }
-    else if (res > av.getEndRes() && !av.getWrapAlignment())
-    {
-      res = av.getEndRes();
-    }
+      if (res < 0)
+      {
+          res = 0;
+      }
 
-    if (changeEndRes)
-    {
-      if (res > stretchGroup.getStartRes() - 1)
+      if (changeEndRes)
       {
-        stretchGroup.setEndRes(res);
+          if (res > (stretchGroup.getStartRes() - 1))
+          {
+              stretchGroup.setEndRes(res);
+          }
       }
-    }
-    else if (changeStartRes)
-    {
-      if (res < stretchGroup.getEndRes() + 1)
+      else if (changeStartRes)
       {
-        stretchGroup.setStartRes(res);
+          if (res < (stretchGroup.getEndRes() + 1))
+          {
+              stretchGroup.setStartRes(res);
+          }
       }
-    }
 
-    int dragDirection = 0;
-    if (y > oldSeq)
-    {
-      dragDirection = 1;
-    }
-    else if (y < oldSeq)
-    {
-      dragDirection = -1;
-    }
+      int dragDirection = 0;
 
-    while (y != oldSeq && oldSeq > 0 && y < av.alignment.getHeight())
-    {
-      // This routine ensures we don't skip any sequences, as the
-      // selection is quite slow.
-      Sequence seq = (Sequence) av.getAlignment().getSequenceAt(oldSeq);
+      if (y > oldSeq)
+      {
+          dragDirection = 1;
+      }
+      else if (y < oldSeq)
+      {
+          dragDirection = -1;
+      }
+
+
+      while ((y != oldSeq) && (oldSeq > -1) && (y < av.alignment.getHeight()))
+      {
+          // This routine ensures we don't skip any sequences, as the
+          // selection is quite slow.
+          Sequence seq = (Sequence) av.getAlignment().getSequenceAt(oldSeq);
+
+          oldSeq += dragDirection;
+
+          if(oldSeq<0)
+            break;
+
+          Sequence nextSeq = (Sequence) av.getAlignment().getSequenceAt(oldSeq);
 
-      oldSeq += dragDirection;
-      Sequence nextSeq = (Sequence) av.getAlignment().getSequenceAt(oldSeq);
+          if (stretchGroup.getSequences(false).contains(nextSeq))
+          {
+              stretchGroup.deleteSequence(seq, false);
+          }
+          else
+          {
+              if (seq != null)
+              {
+                  stretchGroup.addSequence(seq, false);
+              }
+
+              stretchGroup.addSequence(nextSeq, false);
+          }
+      }
+
+      if(oldSeq < 0)
+        oldSeq = -1;
 
-      if (stretchGroup.sequences.contains(nextSeq))
+
+      if(res>av.endRes || res<av.startRes
+          || y<av.startSeq || y>av.endSeq)
       {
-        stretchGroup.deleteSequence(seq, false);
+        mouseExited(evt);
       }
-      else
+
+      if (scrollThread != null)
       {
-        if (seq != null)
-        {
-          stretchGroup.addSequence(seq, false);
-        }
-        stretchGroup.addSequence(nextSeq, false);
+        scrollThread.setEvent(evt);
       }
+
+      seqCanvas.repaint();
     }
-    oldSeq = y;
-    mouseDragging = true;
-    if (scrollThread != null)
+
+    public void mouseEntered(MouseEvent e)
     {
-      scrollThread.setEvent(evt);
-    }
+      if (oldSeq < 0)
+        oldSeq = 0;
 
-    seqCanvas.repaint();
-  }
+      if (scrollThread != null)
+      {
+        scrollThread.running = false;
+        scrollThread = null;
+      }
+    }
 
-  public void doMouseEnteredDefineMode(MouseEvent e)
+  public void mouseExited(MouseEvent e)
   {
-    if (scrollThread != null)
+    if (av.getWrapAlignment())
     {
-      scrollThread.running = false;
+      return;
     }
-  }
 
-  public void doMouseExitedDefineMode(MouseEvent e)
-  {
-    if (mouseDragging)
+    if (mouseDragging && scrollThread==null)
     {
       scrollThread = new ScrollThread();
     }
   }
 
+  void scrollCanvas(MouseEvent evt)
+  {
+    if(evt==null)
+    {
+      if(scrollThread!=null)
+      {
+        scrollThread.running = false;
+        scrollThread = null;
+      }
+      mouseDragging = false;
+    }
+    else
+    {
+      if (scrollThread == null)
+        scrollThread = new ScrollThread();
+
+      mouseDragging = true;
+      scrollThread.setEvent(evt);
+    }
+
+    }
+
   // this class allows scrolling off the bottom of the visible alignment
   class ScrollThread
       extends Thread
@@ -844,6 +1427,7 @@ public class SeqPanel
       running = true;
       while (running)
       {
+
         if (evt != null)
         {
 
@@ -860,12 +1444,12 @@ public class SeqPanel
 
           if (mouseDragging && evt.getX() < 0)
           {
-            running = ap.scrollRight(true);
+            running = ap.scrollRight(false);
           }
 
           else if (mouseDragging && evt.getX() >= getSize().width)
           {
-            running = ap.scrollRight(false);
+            running = ap.scrollRight(true);
           }
         }