X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fgui%2FSeqPanel.java;h=48651f99d0ea73aad6f0927a2a8f9d90c880455c;hb=52fd681e0f239e20df19fd7560d4b999104ba222;hp=2fe27b381bf84bffe35aac5fb94700b32ef764fa;hpb=02f2f19fdecb0b7afa5b61b07c56d46d40948674;p=jalview.git

diff --git a/src/jalview/gui/SeqPanel.java b/src/jalview/gui/SeqPanel.java
index 2fe27b3..48651f9 100755
--- a/src/jalview/gui/SeqPanel.java
+++ b/src/jalview/gui/SeqPanel.java
@@ -45,8 +45,6 @@ public class SeqPanel extends JPanel implements MouseListener,
     public AlignmentPanel ap;
     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
@@ -67,7 +65,10 @@ public class SeqPanel extends JPanel implements MouseListener,
     SequenceGroup stretchGroup = null;
     boolean remove = false;
 
+    Point lastMousePress;
     boolean mouseWheelPressed = false;
+    StringBuffer keyboardNo1;
+    StringBuffer keyboardNo2;
 
     /**
      * Creates a new SeqPanel object.
@@ -173,8 +174,211 @@ public class SeqPanel extends JPanel implements MouseListener,
      seqEditOccurred = false;
      editingSeqs = false;
      groupEditing = false;
+     keyboardNo1 = null;
+     keyboardNo2 = null;
     }
 
+    void setCursorRow()
+    {
+      seqCanvas.cursorY = getKeyboardNo(keyboardNo1)-1;
+      scrollToVisible();
+    }
+
+    void setCursorColumn()
+    {
+      seqCanvas.cursorX = getKeyboardNo(keyboardNo1)-1;
+      scrollToVisible();
+    }
+
+    void setCursorRowAndColumn()
+    {
+      if(keyboardNo2==null)
+      {
+        keyboardNo2 = new StringBuffer();
+      }
+      else
+      {
+        seqCanvas.cursorX = getKeyboardNo(keyboardNo1) - 1;
+        seqCanvas.cursorY = getKeyboardNo(keyboardNo2) - 1;
+        scrollToVisible();
+      }
+    }
+
+    void setCursorPosition()
+    {
+      SequenceI sequence =
+          (Sequence) av.getAlignment().getSequenceAt(seqCanvas.cursorY);
+
+      seqCanvas.cursorX = sequence.findIndex(
+          getKeyboardNo(keyboardNo1)-1
+          );
+      scrollToVisible();
+    }
+
+    void moveCursor(int dx, int dy)
+    {
+      seqCanvas.cursorX += dx;
+      seqCanvas.cursorY += dy;
+      scrollToVisible();
+    }
+
+    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;
+
+
+      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);
+        }
+        if (!av.wrapAlignment)
+        {
+          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(); 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.sequences.clear();
+          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());
+    }
+
+
     /**
      * DOCUMENT ME!
      *
@@ -183,6 +387,7 @@ public class SeqPanel extends JPanel implements MouseListener,
     public void mouseReleased(MouseEvent evt)
     {
       mouseDragging = false;
+      mouseWheelPressed = false;
 
       if (!editingSeqs)
       {
@@ -190,13 +395,10 @@ public class SeqPanel extends JPanel implements MouseListener,
          return;
       }
 
-        if (seqEditOccurred)
-        {
-            editOccurred();
-        }
+       editOccurred();
 
-        endEditing();
-        ap.repaint();
+       endEditing();
+       ap.repaint();
     }
 
 
@@ -208,6 +410,8 @@ public class SeqPanel extends JPanel implements MouseListener,
      */
     public void mousePressed(MouseEvent evt)
     {
+      lastMousePress = evt.getPoint();
+
       if (javax.swing.SwingUtilities.isMiddleMouseButton(evt))
       {
         mouseWheelPressed = true;
@@ -230,8 +434,6 @@ public class SeqPanel extends JPanel implements MouseListener,
       }
 
 
-      ap.alignFrame.addHistoryItem(new HistoryItem("Edit Sequence",
-                                                   av.alignment, HistoryItem.EDIT));
 
       int seq = findSeq(evt);
       int res = findRes(evt);
@@ -239,6 +441,9 @@ public class SeqPanel extends JPanel implements MouseListener,
       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()))
         {
@@ -251,9 +456,6 @@ public class SeqPanel extends JPanel implements MouseListener,
             lastres = -1;
         }
 
-        startEdit = lastres;
-        endEdit = lastres;
-
         return;
     }
 
@@ -274,6 +476,7 @@ public class SeqPanel extends JPanel implements MouseListener,
       int res = findRes(evt);
       int seq = findSeq(evt);
 
+
       if(res<0 || seq<0 || seq >= av.getAlignment().getHeight())
             return;
 
@@ -289,36 +492,7 @@ public class SeqPanel extends JPanel implements MouseListener,
         seqCanvas.pdbCanvas.highlightRes(sequence.findPosition(res));
       }
 
-
-        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 + " (" +
-                        av.getAlignment().getSequenceAt(seq).findPosition(res) + ")");
-          }
-        }
-
-        ap.alignFrame.statusBar.setText(text.toString());
+      setStatusMessage(sequence, res, seq);
 
         // use aa to see if the mouse pointer is on a
         if (av.showSequenceFeatures)
@@ -334,7 +508,8 @@ public class SeqPanel extends JPanel implements MouseListener,
                 if ( (features[i].getBegin() <= sequence.findPosition(res)) &&
                     (features[i].getEnd() >= sequence.findPosition(res)))
                 {
-                  if(!av.featuresDisplayed.containsKey(features[i].getType()))
+                  if(av.featuresDisplayed==null
+                    || !av.featuresDisplayed.containsKey(features[i].getType()))
                   continue;
 
 
@@ -382,6 +557,39 @@ public class SeqPanel extends JPanel implements MouseListener,
         }
     }
 
+    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());
+
+    }
+
     /**
      * DOCUMENT ME!
      *
@@ -389,6 +597,52 @@ public class SeqPanel extends JPanel implements MouseListener,
      */
     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.getY())
+            > Math.abs(evt.getX() - lastMousePress.getX()))
+        {
+          int fontSize = av.font.getSize();
+
+          if (evt.getY() < lastMousePress.getY() && av.charHeight > 1)
+          {
+            fontSize--;
+          }
+          else if (evt.getY() > lastMousePress.getY())
+          {
+            fontSize++;
+          }
+
+          av.setFont(new Font(av.font.getName(), av.font.getStyle(), fontSize));
+          av.charWidth = oldWidth;
+          ap.fontChanged();
+
+        }
+        else
+        {
+          if (evt.getX() < lastMousePress.getX() && av.charWidth > 1)
+          {
+            av.charWidth--;
+          }
+          else if (evt.getX() > lastMousePress.getX())
+          {
+            av.charWidth++;
+          }
+
+          ap.repaint();
+        }
+
+        FontMetrics fm = getFontMetrics(av.getFont());
+        av.validCharWidth = fm.charWidth('M') <= av.charWidth;
+
+        lastMousePress = evt.getPoint();
+
+        return;
+      }
+
       if (!editingSeqs)
       {
         doMouseDraggedDefineMode(evt);
@@ -407,240 +661,382 @@ public class SeqPanel extends JPanel implements MouseListener,
             return;
         }
 
-        boolean dragRight = true;
-
-        if ((res < av.getAlignment().getWidth()) && (res < lastres))
+        if ( (res < av.getAlignment().getWidth()) && (res < lastres))
         {
-            dragRight = false;
+          // dragLeft, delete gap
+          editSequence(false, res);
         }
-        else if(av.hasHiddenColumns)
+        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 && sg==null)
+        return;
+
+        if (!groupEditing && av.hasHiddenRows)
         {
-          //Stop editing if the user has dragged beyond hiddenBoundary
-          int lastCol = av.getColumnSelection().getHiddenRegionBoundary(lastres);
-          if( lastCol < res)
+          //This needs to check all the sequences in a group edit,m
+          // not just the startseq
+          if (av.alignment.getSequenceAt(startseq).getHiddenSequences() != null)
           {
-            if(lastres!=lastCol)
-            {
-              endEditing();
-              return;
-            }
+            groupEditing = true;
           }
         }
 
-        if(!groupEditing && av.hasHiddenRows)
+        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.sequences.contains(seq)))
         {
-          if(av.alignment.getSequenceAt(startseq).getHiddenSequences()!=null)
+          fixedColumns = true;
+
+          fixedLeft = sg.getStartRes();
+          fixedRight = sg.getEndRes();
+
+          if (   (startres < fixedLeft && lastres >= fixedLeft)
+              || (startres >= fixedLeft && lastres < fixedLeft)
+              || (startres > fixedRight && lastres <=fixedRight)
+              || (startres <= fixedRight && lastres > fixedRight))
           {
-            groupEditing = true;
+            endEditing();
+            return;
+          }
+
+          if (fixedLeft > startres)
+          {
+            fixedRight = fixedLeft - 1;
+            fixedLeft = 0;
+          }
+          else if (fixedRight < startres)
+          {
+            fixedLeft = fixedRight;
+            fixedRight = -1;
           }
         }
 
 
-        if (res != lastres)
+
+        if(av.hasHiddenColumns)
         {
-            SequenceI seq;
-            // Group editing
-            if (groupEditing)
-            {
-                SequenceGroup sg = av.getSelectionGroup();
+          fixedColumns = true;
+          int y1 = av.getColumnSelection().getHiddenBoundaryLeft(startres);
+          int y2 = av.getColumnSelection().getHiddenBoundaryRight(startres);
 
-                if (av.hasHiddenRows)
-                {
-                  //sg might be null as the user may only see 1 sequence
-                  if(sg==null)
-                  {
-                    sg = new SequenceGroup();
-                    sg.addSequence(av.alignment.getSequenceAt(startseq), false);
-                  }
+          if(    ( insertGap && startres>y1 && lastres<y1)
+              || (!insertGap && startres<y2 && lastres>y2) )
+          {
+            endEditing();
+            return;
+          }
 
-                  SequenceGroup tmp = new SequenceGroup();
+          if(fixedRight<y2 && fixedRight==-1 && y2!=startres)
+            fixedRight = y2 -1;
+          if(y1>fixedLeft && fixedLeft==-1)
+            fixedLeft = y1;
+        }
 
-                  //Do any of the sequences have hidden associates?
-                  for (int s = 0; s < sg.getSize(); s++)
-                  {
-                    seq = sg.getSequenceAt(s);
-                    tmp.addSequence(seq, false);
-                    if (seq.getHiddenSequences()!=null)
-                    {
-                      for(int h=0; h<seq.getHiddenSequences().getSize(); h++)
-                        tmp.addSequence(seq.getHiddenSequences().getSequenceAt(h),
-                                        false);
-                    }
-                  }
+        if (groupEditing)
+        {
 
-                  sg = tmp;
-                }
+          /*if (av.hasHiddenRows)
+          {
+            //sg might be null as the user may only see 1 sequence
+            if (sg == null)
+            {
+              sg = new SequenceGroup();
+              sg.addSequence(av.alignment.getSequenceAt(startseq), false);
+            }
 
+            SequenceGroup tmp = new SequenceGroup();
 
-                if (sg == null)
-                {
-                    endEditing();
-                    return;
-                }
+            //Do any of the sequences have hidden associates?
+            for (int s = 0; s < sg.getSize(); s++)
+            {
+              seq = sg.getSequenceAt(s);
+              tmp.addSequence(seq, false);
+              if (seq.getHiddenSequences() != null)
+              {
+                for (int h = 0; h < seq.getHiddenSequences().getSize(); h++)
+                  tmp.addSequence(seq.getHiddenSequences().getSequenceAt(h),
+                                  false);
+              }
+            }
 
-                int blankColumn = -1;
+            sg = tmp;
+          }*/
+          // int blankColumn = -1;
 
-                // drag to right
-                if (dragRight)
-                {
 
-                  // Is it valid??
-                  // Find the next gap before the end
-                  // of the visible region boundary
-                  if (av.hasHiddenColumns)
+          // 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() + 1 == av.alignment.getWidth())
+              {
+                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(); s++)
+              {
+                seq = sg.getSequenceAt(s);
+                for (int j = 0; j < startres - lastres; j++)
+                {
+                  if (!jalview.util.Comparison.isGap(
+                      seq.getCharAt(fixedRight - j)))
                   {
-                    if(res-lastres > 1)
-                    {
-                      res = lastres+1;
-                    }
+                    blank = false;
+                    break;
+                  }
+                }
+              }
+              if (blank)
+                break;
+            }
 
+            if (!blank)
+            {
+              if(sg.getSize() == av.alignment.getHeight())
+              {
+                //We can still insert gaps if the selectionGroup
+                //contains all the sequences
+                sg.setEndRes(sg.getEndRes()+startres-lastres);
+                fixedRight = av.alignment.getWidth()+startres-lastres;
+              }
+              else
+              {
+                endEditing();
+                return;
+              }
+            }
+          }
 
-                    int lastCol = av.getColumnSelection().
-                          getHiddenRegionBoundary(res);
 
-                    if(lastCol!=res)
-                    {
-                      for (blankColumn = lastCol;
-                           blankColumn > lastres;
-                           blankColumn--)
-                      {
-                        boolean blank = true;
-                        for (int s = 0; s < sg.getSize(); s++)
-                        {
-                          seq = sg.getSequenceAt(s);
-
-                          if (seq.getSequence().length() <= blankColumn)
-                          {
-                            continue;
-                          }
-
-                          if (!jalview.util.Comparison.isGap(
-                              seq.getSequence().charAt(blankColumn)))
-                          {
-                            blank = false;
-                            continue;
-                          }
-                        }
-                        if (blank)
-                          break;
-                      }
-
-
-                      if (blankColumn <= lastres)
-                      {
-                        endEditing();
-                        return;
-                      }
-                    }
-                    else
-                      blankColumn = -1;
+          // drag to left
+          else if(!insertGap)
+          {
+            /// Are we able to delete?
+            // ie are all columns blank?
 
-                  }
+            for (int s = 0; s < sg.getSize(); s++)
+            {
+              seq = sg.getSequenceAt(s);
 
-                  sg.setEndRes(sg.getEndRes() + (res - lastres));
+              for (int j = startres; j < lastres; j++)
+              {
+                if (seq.getSequence().length() <= j)
+                {
+                  continue;
                 }
 
-                // drag to left
-                else
+                if (!jalview.util.Comparison.isGap(
+                    seq.getSequence().charAt(j)))
                 {
-                    /// Are we able to delete?
-                    // ie are all columns blank?
-
-                    for (int s = 0; s < sg.getSize(); s++)
-                    {
-                        seq = sg.getSequenceAt(s);
-
-                        for (int j = res; j < lastres; j++)
-                        {
-                            if (seq.getSequence().length() <= j)
-                            {
-                                continue;
-                            }
-
-                            if (!jalview.util.Comparison.isGap(
-                                        seq.getSequence().charAt(j)))
-                            {
-                                // Not a gap, block edit not valid
-                                endEditing();
-                                return;
-                            }
-                        }
-                    }
-
-                    if(res<sg.getStartRes())
-                    {
-                      sg.setStartRes(sg.getStartRes() - 1);
-                    }
-                    sg.setEndRes(sg.getEndRes() - (lastres - res));
+                  // Not a gap, block edit not valid
+                  endEditing();
+                  return;
                 }
+              }
+            }
+          }
 
-                for (int i = 0; i < sg.getSize(); i++)
-                {
-                    seq = sg.getSequenceAt(i);
 
-                    // drag to right
-                    if (dragRight)
-                    {
-                        for (int j = lastres; j < res; j++)
-                        {
-                            insertChar(j, seq, blankColumn);
-                        }
-                    }
+          for (int i = 0; i < sg.getSize(); i++)
+          {
+            seq = sg.getSequenceAt(i);
 
-                    // drag to left
-                    else
-                    {
-                        for (int j = res; j < lastres; j++)
-                        {
-                            if (seq.getLength()-1 > res)
-                            {
-                                deleteChar(res, seq);
-                            }
-                        }
-                    }
+            if (insertGap)
+            {
+              // dragging to the right
+              for (int j = lastres; j < startres; j++)
+              {
+                if (fixedColumns && fixedRight != -1)
+                {
+                  insertChar(j, seq, fixedRight);
                 }
+                else
+                  insertChar(j, seq);
+              }
             }
-            else /////Editing a single sequence///////////
+            else
             {
-                seq = av.alignment.getSequenceAt(startseq);
-                if ((res < av.getAlignment().getWidth()) && (res > lastres))
+              // dragging to the left
+              for (int j = lastres; j > startres; j--)
+              {
+                if (fixedColumns && fixedRight != -1)
                 {
-                    // dragging to the right
-                    for (int j = lastres; j < res; j++)
-                    {
-                        insertChar(j, seq, j);
-                    }
+                  deleteChar(startres, seq, fixedRight);
                 }
-                else if ((res < av.getAlignment().getWidth()) &&
-                        (res < lastres))
+                else
                 {
-                    // dragging to the left
-                    for (int j = lastres; j > res; j--)
-                    {
-                        if (jalview.util.Comparison.isGap(
-                                    av.alignment.getSequenceAt(startseq)
-                                                    .getSequence().charAt(res)))
-                        {
-                            deleteChar(res, seq);
-                        }
-                        else
-                        {
-                            break;
-                        }
-                    }
+                  deleteChar(startres, seq);
                 }
+              }
             }
+          }
+        }
+        else /////Editing a single sequence///////////
+        {
+          if (insertGap)
+          {
+            // dragging to the right
+            for (int j = lastres; j < startres; j++)
+            {
+              if (fixedColumns && fixedRight != -1)
+              {
+                if (sg.getStartRes() == 0
+                    && sg.getEndRes() + 1 == av.alignment.getWidth()
+                    && !jalview.util.Comparison.isGap(seq.getCharAt(fixedRight)))
+                {
+                  //Single sequence edit, whole sequence selected,
+                  //extend the selection group
+                  sg.setEndRes(av.alignment.getWidth() -1 + startres - lastres);
+                  fixedColumns = false;
+                  insertChar(j, seq);
+                }
+                else
+                  insertChar(j, seq, fixedRight);
+              }
+              else
+                insertChar(j, seq);
+            }
+          }
+          else
+          {
+            // dragging to the left
+            for (int j = lastres; j > startres; j--)
+            {
+              if (fixedColumns && fixedRight != -1)
+              {
+                deleteChar(startres, seq, fixedRight);
+              }
+              else
+              {
+                deleteChar(startres, seq);
+              }
+            }
+          }
         }
 
-        mouseDragging = true;
-        if(scrollThread!=null)
-          scrollThread.setEvent(evt);
-
-        endEdit = res;
-        lastres = res;
+        lastres = startres;
         seqCanvas.repaint();
     }
 
+
+    /**
+     * DOCUMENT ME!
+     *
+     * @param j DOCUMENT ME!
+     * @param seq DOCUMENT ME!
+     */
+    void insertChar(int j, SequenceI seq)
+    {
+        seq.insertCharAt(j, av.getGapCharacter());
+        seqEditOccurred = true;
+    }
+
+    void insertChar(int j, SequenceI seq, int fixedColumn)
+    {
+      //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--)
+      {
+        if (jalview.util.Comparison.isGap(seq.getCharAt(blankColumn)))
+        {
+          //Theres a space, so break and insert the gap
+          break;
+        }
+      }
+
+      if (blankColumn <= j)
+      {
+        endEditing();
+        return;
+      }
+
+      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;
+    }
+
+    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;
+      }
+
+      seq.deleteCharAt(j);
+      seq.insertCharAt(fixedColumn, av.getGapCharacter());
+      seqEditOccurred = true;
+    }
+
+    /**
+     * DOCUMENT ME!
+     *
+     * @param j DOCUMENT ME!
+     * @param seq DOCUMENT ME!
+     */
+    void deleteChar(int j, SequenceI seq)
+    {
+      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();
+    }
     /**
      * DOCUMENT ME!
      *
@@ -681,7 +1077,8 @@ public class SeqPanel extends JPanel implements MouseListener,
 
     public void mouseWheelMoved(MouseWheelEvent e)
     {
-      if (mouseWheelPressed)
+      e.consume();
+     /* if (mouseWheelPressed)
       {
         Font font = av.getFont();
         int fontSize = font.getSize();
@@ -690,10 +1087,13 @@ public class SeqPanel extends JPanel implements MouseListener,
         else if (fontSize > 1)
           fontSize--;
 
+
+
         av.setFont(new Font(font.getName(), font.getStyle(), fontSize));
+
         ap.fontChanged();
       }
-      else
+      else*/
       {
         if (e.getWheelRotation() > 0)
           ap.scrollUp(false);
@@ -704,78 +1104,6 @@ public class SeqPanel extends JPanel implements MouseListener,
     }
 
 
-    /**
-     * DOCUMENT ME!
-     *
-     * @param j DOCUMENT ME!
-     * @param seq DOCUMENT ME!
-     */
-    void insertChar(int j, SequenceI seq, int blankColumn)
-    {
-        if(av.hasHiddenColumns)
-        {
-          //Find the next gap before the end of the visible region boundary
-          int lastCol = av.getColumnSelection().getHiddenRegionBoundary(j);
-          if(lastCol != j)
-          {
-            if (!groupEditing || lastCol <= j)
-            {
-              blankColumn = lastCol;
-              //If lastCol > j, theres a boundary after the gap insertion
-              if (lastCol > j)
-              {
-                for (blankColumn = lastCol; blankColumn > j; blankColumn--)
-                {
-                  if (jalview.util.Comparison.isGap(seq.getCharAt(blankColumn)))
-                  {
-                    //Theres a space, so break and insert the gap
-                    break;
-                  }
-                }
-
-                if (blankColumn <= j)
-                {
-                  endEditing();
-                  return;
-                }
-              }
-
-            }
-
-            // Editing with hidden regions only!!
-            seq.deleteCharAt(blankColumn);
-          }
-      }
-
-        seq.insertCharAt(j, av.getGapCharacter());
-        seqEditOccurred = true;
-    }
-
-    /**
-     * DOCUMENT ME!
-     *
-     * @param j DOCUMENT ME!
-     * @param seq DOCUMENT ME!
-     */
-    public void deleteChar(int j, SequenceI seq)
-    {
-      if (av.hasHiddenColumns)
-      {
-        //Find the next gap before the end of the visible region boundary
-        int lastCol = av.getColumnSelection().getHiddenRegionBoundary(j);
-
-        //If lastCol > j, theres a boundary after the gap insertion
-        if (lastCol > j)
-        {
-          seq.insertCharAt(lastCol, av.getGapCharacter());
-        }
-      }
-
-        seq.deleteCharAt(j);
-        seqEditOccurred = true;
-
-        seqCanvas.repaint();
-    }
 
     /**
      * DOCUMENT ME!
@@ -784,12 +1112,14 @@ public class SeqPanel extends JPanel implements MouseListener,
      */
     void editOccurred()
     {
-      if (endEdit == startEdit)
+      if (!seqEditOccurred)
       {
         ap.alignFrame.historyList.pop();
         ap.alignFrame.updateEditMenuBar();
       }
 
+      endEditing();
+
       av.firePropertyChange("alignment", null,av.getAlignment().getSequences());
 
     }
@@ -819,7 +1149,6 @@ public class SeqPanel extends JPanel implements MouseListener,
       if(seq<0 || res<0)
             return;
 
-
         SequenceI sequence = (Sequence) av.getAlignment().getSequenceAt(seq);
 
         if ((sequence == null) || (res > sequence.getLength()))
@@ -867,6 +1196,15 @@ public class SeqPanel extends JPanel implements MouseListener,
             }
         }
 
+        if (av.cursorMode)
+        {
+          seqCanvas.cursorX = findRes(evt);
+          seqCanvas.cursorY = findSeq(evt);
+          seqCanvas.repaint();
+          return;
+        }
+
+
         if (stretchGroup == null)
         {
             // define a new group here
@@ -976,6 +1314,10 @@ public class SeqPanel extends JPanel implements MouseListener,
         {
           y = av.alignment.getHeight() -1;
         }
+        if(res> av.alignment.getWidth())
+        {
+          res = av.alignment.getWidth()-1;
+        }
 
         if (stretchGroup.getEndRes() == res)
         {
@@ -1061,6 +1403,27 @@ public class SeqPanel extends JPanel implements MouseListener,
         seqCanvas.repaint();
     }
 
+    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
@@ -1106,11 +1469,11 @@ public class SeqPanel extends JPanel implements MouseListener,
 
                     if (mouseDragging && (evt.getX() < 0))
                     {
-                        running = ap.scrollRight(true);
+                        running = ap.scrollRight(false);
                     }
                     else if (mouseDragging && (evt.getX() >= getWidth()))
                     {
-                        running = ap.scrollRight(false);
+                        running = ap.scrollRight(true);
                     }
                 }