Unique name for new group and treegroup
[jalview.git] / src / jalview / gui / SeqPanel.java
index 4b7a388..ed9ce8d 100755 (executable)
 package jalview.gui;\r
 \r
 import jalview.datamodel.*;\r
-\r
+import jalview.commands.EditCommand;\r
 import jalview.schemes.*;\r
 \r
 import java.awt.*;\r
 import java.awt.event.*;\r
-\r
 import javax.swing.*;\r
-\r
 import java.util.Vector;\r
 \r
 \r
@@ -40,75 +38,78 @@ public class SeqPanel extends JPanel implements MouseListener,
     MouseMotionListener, MouseWheelListener\r
 \r
 {\r
-    /** DOCUMENT ME!! */\r
-    public SeqCanvas seqCanvas;\r
-\r
-    /** DOCUMENT ME!! */\r
-    public AlignmentPanel ap;\r
-    protected int lastres;\r
-    protected int startseq;\r
-    protected AlignViewport av;\r
-\r
-    // if character is inserted or deleted, we will need to recalculate the conservation\r
-    boolean seqEditOccurred = false;\r
-    ScrollThread scrollThread = null;\r
-    boolean mouseDragging = false;\r
-    boolean editingSeqs = false;\r
-    boolean groupEditing = false;\r
-\r
-    //////////////////////////////////////////\r
-    /////Everything below this is for defining the boundary of the rubberband\r
-    //////////////////////////////////////////\r
-    int oldSeq = -1;\r
-    boolean changeEndSeq = false;\r
-    boolean changeStartSeq = false;\r
-    boolean changeEndRes = false;\r
-    boolean changeStartRes = false;\r
-    SequenceGroup stretchGroup = null;\r
-    boolean remove = false;\r
-\r
-    Point lastMousePress;\r
-    boolean mouseWheelPressed = false;\r
-    StringBuffer keyboardNo1;\r
-    StringBuffer keyboardNo2;\r
-\r
-    java.net.URL linkImageURL;\r
-\r
-    /**\r
-     * Creates a new SeqPanel object.\r
-     *\r
-     * @param avp DOCUMENT ME!\r
-     * @param p DOCUMENT ME!\r
-     */\r
-    public SeqPanel(AlignViewport avp, AlignmentPanel p)\r
+  /** DOCUMENT ME!! */\r
+  public SeqCanvas seqCanvas;\r
+\r
+  /** DOCUMENT ME!! */\r
+  public AlignmentPanel ap;\r
+  protected int lastres;\r
+  protected int startseq;\r
+  protected AlignViewport av;\r
+\r
+  ScrollThread scrollThread = null;\r
+  boolean mouseDragging = false;\r
+  boolean editingSeqs = false;\r
+  boolean groupEditing = false;\r
+\r
+  //////////////////////////////////////////\r
+  /////Everything below this is for defining the boundary of the rubberband\r
+  //////////////////////////////////////////\r
+  int oldSeq = -1;\r
+  boolean changeEndSeq = false;\r
+  boolean changeStartSeq = false;\r
+  boolean changeEndRes = false;\r
+  boolean changeStartRes = false;\r
+  SequenceGroup stretchGroup = null;\r
+  boolean remove = false;\r
+\r
+  Point lastMousePress;\r
+  boolean mouseWheelPressed = false;\r
+  StringBuffer keyboardNo1;\r
+  StringBuffer keyboardNo2;\r
+\r
+  java.net.URL linkImageURL;\r
+\r
+  StringBuffer tooltipText = new StringBuffer("<html>");\r
+  String tmpString;\r
+\r
+  EditCommand editCommand;\r
+\r
+  /**\r
+   * Creates a new SeqPanel object.\r
+   *\r
+   * @param avp DOCUMENT ME!\r
+   * @param p DOCUMENT ME!\r
+   */\r
+  public SeqPanel(AlignViewport avp, AlignmentPanel p)\r
+  {\r
+    linkImageURL = getClass().getResource("/images/link.gif");\r
+    ToolTipManager.sharedInstance().registerComponent(this);\r
+    ToolTipManager.sharedInstance().setInitialDelay(0);\r
+    ToolTipManager.sharedInstance().setDismissDelay(10000);\r
+    this.av = avp;\r
+    setBackground(Color.white);\r
+\r
+    seqCanvas = new SeqCanvas(avp);\r
+    setLayout(new BorderLayout());\r
+    add(seqCanvas, BorderLayout.CENTER);\r
+\r
+    ap = p;\r
+\r
+    if (!av.isDataset())\r
     {\r
-        linkImageURL = getClass().getResource("/images/link.gif");\r
-        ToolTipManager.sharedInstance().registerComponent(this);\r
-        ToolTipManager.sharedInstance().setInitialDelay(0);\r
-        ToolTipManager.sharedInstance().setDismissDelay(10000);\r
-        this.av = avp;\r
-        setBackground(Color.white);\r
-\r
-        seqCanvas = new SeqCanvas(avp);\r
-        setLayout(new BorderLayout());\r
-        add(seqCanvas, BorderLayout.CENTER);\r
-\r
-        ap = p;\r
-\r
-        if(!av.isDataset())\r
-        {\r
-          addMouseMotionListener(this);\r
-          addMouseListener(this);\r
-          addMouseWheelListener(this);\r
-        }\r
+      addMouseMotionListener(this);\r
+      addMouseListener(this);\r
+      addMouseWheelListener(this);\r
     }\r
+  }\r
 \r
-    int startWrapBlock=-1;\r
-    int wrappedBlock=-1;\r
-    int findRes(MouseEvent evt)\r
-   {\r
-     int res = 0;\r
-     int x = evt.getX();\r
+  int startWrapBlock = -1;\r
+  int wrappedBlock = -1;\r
+  int findRes(MouseEvent evt)\r
+  {\r
+    int res = 0;\r
+    int x = evt.getX();\r
 \r
     if (av.wrapAlignment)\r
     {\r
@@ -120,60 +121,88 @@ public class SeqPanel extends JPanel implements MouseListener,
       int cHeight = av.getAlignment().getHeight() * av.charHeight\r
           + hgap + seqCanvas.getAnnotationHeight();\r
 \r
-        int y = evt.getY();\r
-        y -= hgap;\r
-        x -= seqCanvas.LABEL_WEST;\r
+      int y = evt.getY();\r
+      y -= hgap;\r
+      x -= seqCanvas.LABEL_WEST;\r
 \r
+      int cwidth = seqCanvas.getWrappedCanvasWidth(this.getWidth());\r
+      if (cwidth < 1)\r
+        return 0;\r
 \r
-        int cwidth = seqCanvas.getWrappedCanvasWidth(this.getWidth());\r
-        if(cwidth<1)\r
-          return 0;\r
+      wrappedBlock = y / cHeight;\r
+      wrappedBlock += av.getStartRes() / cwidth;\r
 \r
-        wrappedBlock = y / cHeight;\r
-        wrappedBlock += av.getStartRes() / cwidth;\r
-\r
-        res = wrappedBlock * cwidth + x / av.getCharWidth();\r
+      res = wrappedBlock * cwidth + x / av.getCharWidth();\r
 \r
     }\r
     else\r
     {\r
-        res = (x / av.getCharWidth()) + av.getStartRes();\r
+      res = (x / av.getCharWidth()) + av.getStartRes();\r
     }\r
 \r
-    if(av.hasHiddenColumns)\r
-          res = av.getColumnSelection().adjustForHiddenColumns(res);\r
+    if (av.hasHiddenColumns)\r
+      res = av.getColumnSelection().adjustForHiddenColumns(res);\r
 \r
     return res;\r
 \r
-   }\r
+  }\r
 \r
-   int findSeq(MouseEvent evt)\r
-   {\r
-     int seq = 0;\r
-     int y = evt.getY();\r
+  int findSeq(MouseEvent evt)\r
+  {\r
+    int seq = 0;\r
+    int y = evt.getY();\r
 \r
-     if (av.wrapAlignment)\r
-     {\r
-       int hgap = av.charHeight;\r
-       if (av.scaleAboveWrapped)\r
-         hgap += av.charHeight;\r
+    if (av.wrapAlignment)\r
+    {\r
+      int hgap = av.charHeight;\r
+      if (av.scaleAboveWrapped)\r
+        hgap += av.charHeight;\r
 \r
-       int cHeight = av.getAlignment().getHeight() * av.charHeight\r
-           + hgap + seqCanvas.getAnnotationHeight();\r
+      int cHeight = av.getAlignment().getHeight() * av.charHeight\r
+          + hgap + seqCanvas.getAnnotationHeight();\r
 \r
-         y -= hgap;\r
+      y -= hgap;\r
 \r
-       seq = Math.min( (y % cHeight) / av.getCharHeight(),\r
-                       av.alignment.getHeight() -1);\r
-     }\r
-     else\r
-     {\r
-       seq = Math.min( (y / av.getCharHeight()) + av.getStartSeq(),\r
-                       av.alignment.getHeight() -1);\r
-     }\r
+      seq = Math.min( (y % cHeight) / av.getCharHeight(),\r
+                     av.alignment.getHeight() - 1);\r
+    }\r
+    else\r
+    {\r
+      seq = Math.min( (y / av.getCharHeight()) + av.getStartSeq(),\r
+                     av.alignment.getHeight() - 1);\r
+    }\r
+\r
+    return seq;\r
+  }\r
+\r
+  SequenceFeature[] findFeaturesAtRes(SequenceI sequence, int res)\r
+  {\r
+    Vector tmp = new Vector();\r
+    SequenceFeature[] features = sequence.getSequenceFeatures();\r
+    if (features != null)\r
+    {\r
+      for (int i = 0; i < features.length; i++)\r
+      {\r
+        if (av.featuresDisplayed == null\r
+            || !av.featuresDisplayed.containsKey(features[i].getType()))\r
+          continue;\r
 \r
-     return seq;\r
-   }\r
+        if ( (features[i].getBegin() <= res) &&\r
+            (features[i].getEnd() >= res))\r
+        {\r
+          tmp.addElement(features[i]);\r
+        }\r
+      }\r
+    }\r
+\r
+    features = new SequenceFeature[tmp.size()];\r
+    for(int i=0; i<tmp.size(); i++)\r
+    {\r
+      features[i] = (SequenceFeature)tmp.elementAt(i);\r
+    }\r
+\r
+    return features;\r
+  }\r
 \r
    Vector getAllFeaturesAtRes(SequenceI seq, int res)\r
    {\r
@@ -200,13 +229,20 @@ public class SeqPanel extends JPanel implements MouseListener,
 \r
    void endEditing()\r
    {\r
+     if (editCommand!=null && editCommand.getSize() > 0)\r
+     {\r
+       ap.alignFrame.addHistoryItem(editCommand);\r
+       av.firePropertyChange("alignment", null,\r
+                             av.getAlignment().getSequences());\r
+     }\r
+\r
      startseq = -1;\r
      lastres = -1;\r
-     seqEditOccurred = false;\r
      editingSeqs = false;\r
      groupEditing = false;\r
      keyboardNo1 = null;\r
      keyboardNo2 = null;\r
+     editCommand = null;\r
     }\r
 \r
     void setCursorRow()\r
@@ -250,6 +286,25 @@ public class SeqPanel extends JPanel implements MouseListener,
     {\r
       seqCanvas.cursorX += dx;\r
       seqCanvas.cursorY += dy;\r
+      if (av.hasHiddenColumns && !av.colSel.isVisible(seqCanvas.cursorX))\r
+      {\r
+        int original = seqCanvas.cursorX - dx;\r
+        int maxWidth = av.alignment.getWidth();\r
+\r
+        while(!av.colSel.isVisible(seqCanvas.cursorX)\r
+              && seqCanvas.cursorX<maxWidth\r
+              && seqCanvas.cursorX>0)\r
+        {\r
+          seqCanvas.cursorX += dx;\r
+        }\r
+\r
+        if(seqCanvas.cursorX>=maxWidth\r
+           || !av.colSel.isVisible(seqCanvas.cursorX) )\r
+        {\r
+          seqCanvas.cursorX = original;\r
+        }\r
+      }\r
+\r
       scrollToVisible();\r
     }\r
 \r
@@ -283,12 +338,12 @@ public class SeqPanel extends JPanel implements MouseListener,
         }\r
         if (!av.wrapAlignment)\r
         {\r
-          while (seqCanvas.cursorX < av.startRes)\r
+          while (seqCanvas.cursorX < av.colSel.adjustForHiddenColumns(av.startRes))\r
           {\r
             if (!ap.scrollRight(false))\r
               break;\r
           }\r
-          while (seqCanvas.cursorX > av.endRes)\r
+          while (seqCanvas.cursorX > av.colSel.adjustForHiddenColumns(av.endRes))\r
           {\r
             if (!ap.scrollRight(true))\r
               break;\r
@@ -311,7 +366,7 @@ public class SeqPanel extends JPanel implements MouseListener,
         SequenceGroup sg = av.selectionGroup;\r
         //Find the top and bottom of this group\r
         int min = av.alignment.getHeight(), max = 0;\r
-        for(int i=0; i<sg.getSize(false); i++)\r
+        for(int i=0; i<sg.getSize(); i++)\r
         {\r
           int index = av.alignment.findIndex( sg.getSequenceAt(i) );\r
           if(index > max)\r
@@ -347,7 +402,7 @@ public class SeqPanel extends JPanel implements MouseListener,
         else\r
         {\r
           // Now add any sequences between min and max\r
-          sg.getSequences(false).clear();\r
+          sg.getSequences(null).clear();\r
           for (int i = min; i < max; i++)\r
           {\r
             sg.addSequence(av.alignment.getSequenceAt(i), false);\r
@@ -370,24 +425,20 @@ public class SeqPanel extends JPanel implements MouseListener,
 \r
     void insertGapAtCursor(boolean group)\r
     {\r
-      ap.alignFrame.addHistoryItem(new HistoryItem("Edit Sequence",\r
-                                                   av.alignment, HistoryItem.EDIT));\r
       groupEditing = group;\r
       startseq = seqCanvas.cursorY;\r
       lastres = seqCanvas.cursorX;\r
       editSequence(true, seqCanvas.cursorX+getKeyboardNo(keyboardNo1));\r
-      editOccurred();\r
+      endEditing();\r
     }\r
 \r
     void deleteGapAtCursor(boolean group)\r
     {\r
-      ap.alignFrame.addHistoryItem(new HistoryItem("Edit Sequence",\r
-                                                   av.alignment, HistoryItem.EDIT));\r
       groupEditing = group;\r
       startseq = seqCanvas.cursorY;\r
       lastres = seqCanvas.cursorX+getKeyboardNo(keyboardNo1);\r
       editSequence(false, seqCanvas.cursorX);\r
-      editOccurred();\r
+      endEditing();\r
     }\r
 \r
     void numberPressed(char value)\r
@@ -426,7 +477,7 @@ public class SeqPanel extends JPanel implements MouseListener,
          return;\r
       }\r
 \r
-       editOccurred();\r
+       endEditing();\r
 \r
        ap.repaint();\r
     }\r
@@ -463,17 +514,12 @@ public class SeqPanel extends JPanel implements MouseListener,
         return;\r
       }\r
 \r
-\r
-\r
       int seq = findSeq(evt);\r
       int res = findRes(evt);\r
 \r
       if(seq<0 || res<0)\r
         return;\r
 \r
-      ap.alignFrame.addHistoryItem(new HistoryItem("Edit Sequence",\r
-                                                         av.alignment, HistoryItem.EDIT));\r
-\r
         if ((seq < av.getAlignment().getHeight()) &&\r
                 (res < av.getAlignment().getSequenceAt(seq).getLength()))\r
         {\r
@@ -524,84 +570,127 @@ public class SeqPanel extends JPanel implements MouseListener,
 \r
       setStatusMessage(sequence, res, seq);\r
 \r
+\r
+\r
+      tooltipText.setLength(6); // Cuts the buffer back to <html>\r
+\r
+      SequenceGroup []  groups = av.alignment.findAllGroups(sequence);\r
+      if(groups!=null)\r
+      {\r
+        for(int g=0; g<groups.length; g++)\r
+        {\r
+          if(groups[g].getStartRes()<=res && groups[g].getEndRes()>=res)\r
+          {\r
+            if (!groups[g].getName().startsWith("JTreeGroup") &&\r
+                !groups[g].getName().startsWith("JGroup"))\r
+              tooltipText.append(groups[g].getName() + " ");\r
+\r
+            if(groups[g].getDescription()!=null)\r
+              tooltipText.append(groups[g].getDescription()+"<BR>");\r
+          }\r
+        }\r
+      }\r
+\r
+\r
         // use aa to see if the mouse pointer is on a\r
         if (av.showSequenceFeatures)\r
         {\r
-            SequenceFeature [] features = sequence.getDatasetSequence().getSequenceFeatures();\r
-            if(features!=null)\r
-            {\r
-              StringBuffer sbuffer = new StringBuffer("<html>");\r
-              StringBuffer seqSpecific =  new StringBuffer();\r
+          SequenceFeature[] features = findFeaturesAtRes(\r
+              sequence.getDatasetSequence(), sequence.findPosition(res));\r
 \r
-              for (int i = 0; i < features.length; i++)\r
+          if (features != null)\r
+          {\r
+            for (int i = 0; i < features.length; i++)\r
+            {\r
+              if (features[i].getType().equals("disulfide bond"))\r
               {\r
-\r
-                if ( (features[i].getBegin() <= sequence.findPosition(res)) &&\r
-                    (features[i].getEnd() >= sequence.findPosition(res)))\r
+                if (features[i].getBegin() == sequence.findPosition(res)\r
+                    || features[i].getEnd() == sequence.findPosition(res))\r
                 {\r
-                  if(av.featuresDisplayed==null\r
-                    || !av.featuresDisplayed.containsKey(features[i].getType()))\r
-                  continue;\r
+                  if (tooltipText.length() > 6)\r
+                    tooltipText.append("<br>");\r
+                  tooltipText.append("disulfide bond " + features[i].getBegin() + ":" +\r
+                                     features[i].getEnd());\r
+                  if (features[i].links != null)\r
+                    tooltipText.append(" <img src=\"" + linkImageURL + "\">");\r
+                }\r
+              }\r
+              else\r
+              {\r
+                if (tooltipText.length() > 6)\r
+                  tooltipText.append("<br>");\r
 \r
+                tooltipText.append(features[i].getType() + " " +\r
+                                   features[i].begin);\r
+                if (features[i].begin != features[i].end)\r
+                  tooltipText.append(" " + features[i].end);\r
 \r
-                  if (features[i].getType().equals("disulfide bond"))\r
+                if (features[i].getDescription() != null\r
+                    && !features[i].description.equals(features[i].getType()))\r
+                {\r
+                  tmpString = features[i].getDescription();\r
+                  int startTag = tmpString.toUpperCase().indexOf("<HTML>");\r
+                  if (startTag > -1)\r
                   {\r
-                    if (features[i].getBegin() == sequence.findPosition(res)\r
-                        || features[i].getEnd() == sequence.findPosition(res))\r
-                    {\r
-                      if (sbuffer.length() > 6)\r
-                        sbuffer.append("<br>");\r
-                      sbuffer.append("disulfide bond " + features[i].getBegin() + ":" +\r
-                                     features[i].getEnd());\r
-                      if (features[i].links != null)\r
-                      sbuffer.append(" <img src=\"" + linkImageURL + "\">");\r
-                    }\r
+                    tmpString = tmpString.substring(startTag + 6);\r
                   }\r
-                  else\r
+                  int endTag = tmpString.toUpperCase().indexOf("</BODY>");\r
+                  if (endTag > -1)\r
+                  {\r
+                    tmpString = tmpString.substring(0, endTag);\r
+                  }\r
+                  endTag = tmpString.toUpperCase().indexOf("</HTML>");\r
+                  if (endTag > -1)\r
                   {\r
-                    if (sbuffer.length() > 6)\r
-                      sbuffer.append("<br>");\r
+                    tmpString = tmpString.substring(0, endTag);\r
+                  }\r
 \r
-                    sbuffer.append(features[i].getType() + " " +\r
-                                   features[i].begin);\r
-                    if (features[i].begin != features[i].end)\r
-                      sbuffer.append(" " + features[i].end);\r
+                  if (startTag > -1)\r
+                  {\r
+                    tooltipText.append("; " + tmpString);\r
+                  }\r
+                  else\r
+                  {\r
+                    if (tmpString.indexOf("<") > -1 || tmpString.indexOf(">") > -1)\r
+                    {\r
+                      //The description does not specify html is to\r
+                      //be used, so we must remove < > symbols\r
+                      tmpString = tmpString.replaceAll("<", "&lt;");\r
+                      tmpString = tmpString.replaceAll(">", "&gt;");\r
 \r
-                    if (features[i].getDescription() != null\r
-                        && !features[i].description.equals(features[i].getType()))\r
-                      sbuffer.append("; " + features[i].getDescription());\r
+                      tooltipText.append("; ");\r
+                      tooltipText.append(tmpString);\r
 \r
-                    if (features[i].getValue("status") != null)\r
-                    {\r
-                      sbuffer.append("; (" + features[i].getValue("status") + ")");\r
                     }\r
-                    if (features[i].links != null)\r
-                      sbuffer.append(" <img src=\"" + linkImageURL + "\">");\r
-\r
+                    else\r
+                      tooltipText.append("; " + tmpString);\r
                   }\r
                 }\r
-                else if(features[i].begin==0 && features[i].end==0)\r
+                if (features[i].getValue("status") != null)\r
                 {\r
-                  // seqSpecific.append(features[i].featureGroup+": "\r
-                  //                   + features[i].getType()+" "\r
-                   //                   +features[i].getDescription()+"<br>");\r
-\r
+                  String status = features[i].getValue("status").toString();\r
+                  if (status.length() > 0)\r
+                    tooltipText.append("; (" + features[i].getValue("status") + ")");\r
                 }\r
-              }\r
 \r
-              if(seqSpecific.length()>0)\r
-                seqSpecific.setLength(seqSpecific.length()-4);\r
+                if (features[i].links != null)\r
+                  tooltipText.append(" <img src=\"" + linkImageURL + "\">");\r
 \r
-              sbuffer.append(seqSpecific);\r
-              sbuffer.append("</html>");\r
-              if(sbuffer.length()==13) // <html></html>\r
-                setToolTipText("");\r
-              else\r
-               setToolTipText(sbuffer.toString());\r
+              }\r
             }\r
-            else\r
-              setToolTipText("");\r
+          }\r
+        }\r
+        if (tooltipText.length() == 6) // <html></html>\r
+        {\r
+          setToolTipText(null);\r
         }\r
+        else\r
+        {\r
+          tooltipText.append("</html>");\r
+          setToolTipText(tooltipText.toString());\r
+        }\r
+       // else\r
+        //  setToolTipText(null);\r
     }\r
 \r
     void setStatusMessage(SequenceI sequence, int res, int seq)\r
@@ -731,29 +820,36 @@ public class SeqPanel extends JPanel implements MouseListener,
       boolean fixedColumns = false;\r
       SequenceGroup sg = av.getSelectionGroup();\r
 \r
+      SequenceI seq = av.alignment.getSequenceAt(startseq);\r
 \r
+      //No group, but the sequence may represent a group\r
         if (!groupEditing && av.hasHiddenRows)\r
         {\r
-          if (av.alignment.getSequenceAt(startseq).getHiddenSequences() != null)\r
+          if (av.hiddenRepSequences!=null\r
+              && av.hiddenRepSequences.containsKey(seq))\r
           {\r
+            sg = (SequenceGroup)av.hiddenRepSequences.get(seq);\r
             groupEditing = true;\r
           }\r
         }\r
 \r
-        //No group, but the sequence may represent a group\r
-        if (groupEditing\r
-            && sg == null\r
-            && av.alignment.getSequenceAt(startseq).getHiddenSequences() == null)\r
-        {\r
-          groupEditing = false;\r
-        }\r
 \r
-        SequenceI seq = av.alignment.getSequenceAt(startseq);\r
         StringBuffer message = new StringBuffer();\r
         if (groupEditing)\r
-           message.append("Edit group:");\r
+        {\r
+          message.append("Edit group:");\r
+          if (editCommand == null)\r
+            editCommand = new EditCommand("Edit Group");\r
+        }\r
         else\r
-           message.append("Edit sequence: "+seq.getName());\r
+         {\r
+           message.append("Edit sequence: " + seq.getName());\r
+           String label = seq.getName();\r
+           if(label.length()>10)\r
+             label = label.substring(0,10);\r
+           if(editCommand==null)\r
+             editCommand = new EditCommand("Edit "+label);\r
+         }\r
 \r
        if(insertGap)\r
          message.append(" insert ");\r
@@ -766,7 +862,7 @@ public class SeqPanel extends JPanel implements MouseListener,
 \r
         //Are we editing within a selection group?\r
         if (groupEditing\r
-            || (sg != null && sg.getSequences(true).contains(seq)))\r
+            || (sg != null && sg.getSequences(av.hiddenRepSequences).contains(seq)))\r
         {\r
           fixedColumns = true;\r
 \r
@@ -774,9 +870,7 @@ public class SeqPanel extends JPanel implements MouseListener,
           //but the sequence represents a group\r
           if (sg == null)\r
           {\r
-            sg = new SequenceGroup(null, null, false, false, false, 0,\r
-                                   av.alignment.getWidth()-1);\r
-            sg.addSequence(av.alignment.getSequenceAt(startseq), false);\r
+            sg = (SequenceGroup) av.hiddenRepSequences.get(seq);\r
           }\r
 \r
           fixedLeft = sg.getStartRes();\r
@@ -835,6 +929,12 @@ public class SeqPanel extends JPanel implements MouseListener,
 \r
         if (groupEditing)\r
         {\r
+          Vector vseqs = sg.getSequences(av.hiddenRepSequences);\r
+          int g, groupSize = vseqs.size();\r
+          SequenceI[] groupSeqs = new SequenceI[groupSize];\r
+          for (g = 0; g < groupSeqs.length; g++)\r
+            groupSeqs[g] = (SequenceI) vseqs.elementAt(g);\r
+\r
           // drag to right\r
           if (insertGap)\r
           {\r
@@ -858,13 +958,13 @@ public class SeqPanel extends JPanel implements MouseListener,
                  fixedRight--)\r
             {\r
               blank = true;\r
-              for (int s = 0; s < sg.getSize(true); s++)\r
+\r
+              for (g = 0; g < groupSize; g++)\r
               {\r
-                seq = (SequenceI)sg.getSequences(true).elementAt(s);\r
                 for (int j = 0; j < startres - lastres; j++)\r
                 {\r
                   if (!jalview.util.Comparison.isGap(\r
-                      seq.getCharAt(fixedRight - j)))\r
+                      groupSeqs[g].getCharAt(fixedRight - j)))\r
                   {\r
                     blank = false;\r
                     break;\r
@@ -877,7 +977,7 @@ public class SeqPanel extends JPanel implements MouseListener,
 \r
             if (!blank)\r
             {\r
-              if(sg.getSize(false) == av.alignment.getHeight()  )\r
+              if(sg.getSize() == av.alignment.getHeight()  )\r
               {\r
                 if((av.hasHiddenColumns\r
                     && startres<av.getColumnSelection().getHiddenBoundaryRight(startres)))\r
@@ -913,19 +1013,17 @@ public class SeqPanel extends JPanel implements MouseListener,
             /// Are we able to delete?\r
             // ie are all columns blank?\r
 \r
-            for (int s = 0; s < sg.getSize(true); s++)\r
+            for (g = 0; g < groupSize; g++)\r
             {\r
-              seq = (SequenceI)sg.getSequences(true).elementAt(s);\r
-\r
               for (int j = startres; j < lastres; j++)\r
               {\r
-                if (seq.getSequence().length() <= j)\r
+                if (groupSeqs[g].getLength() <= j)\r
                 {\r
                   continue;\r
                 }\r
 \r
                 if (!jalview.util.Comparison.isGap(\r
-                    seq.getSequence().charAt(j)))\r
+                    groupSeqs[g].getCharAt(j)))\r
                 {\r
                   // Not a gap, block edit not valid\r
                   endEditing();\r
@@ -935,68 +1033,100 @@ public class SeqPanel extends JPanel implements MouseListener,
             }\r
           }\r
 \r
-\r
-          for (int i = 0; i < sg.getSize(true); i++)\r
-          {\r
-            seq = (SequenceI) sg.getSequences(true).elementAt(i);\r
-\r
             if (insertGap)\r
             {\r
               // dragging to the right\r
-              for (int j = lastres; j < startres; j++)\r
+              if (fixedColumns && fixedRight != -1)\r
               {\r
-                if (fixedColumns && fixedRight != -1)\r
+                for (int j = lastres; j < startres; j++)\r
                 {\r
-                  insertChar(j, seq, fixedRight);\r
+                    insertChar(j, groupSeqs, fixedRight);\r
                 }\r
-                else\r
-                  insertChar(j, seq);\r
+              }\r
+              else\r
+              {\r
+                editCommand.appendEdit(EditCommand.INSERT_GAP,\r
+                                       groupSeqs,\r
+                                       startres, startres-lastres,\r
+                                       av.alignment,\r
+                                       true);\r
               }\r
             }\r
             else\r
             {\r
               // dragging to the left\r
-              for (int j = lastres; j > startres; j--)\r
+              if (fixedColumns && fixedRight != -1)\r
               {\r
-                if (fixedColumns && fixedRight != -1)\r
-                {\r
-                  deleteChar(startres, seq, fixedRight);\r
-                }\r
-                else\r
+                for (int j = lastres; j > startres; j--)\r
                 {\r
-                  deleteChar(startres, seq);\r
+                  deleteChar(startres, groupSeqs, fixedRight);\r
                 }\r
               }\r
+              else\r
+                editCommand.appendEdit(EditCommand.DELETE_GAP,\r
+                                       groupSeqs,\r
+                                       startres, lastres - startres,\r
+                                       av.alignment,\r
+                                       true);\r
+\r
             }\r
-          }\r
         }\r
         else /////Editing a single sequence///////////\r
         {\r
           if (insertGap)\r
           {\r
             // dragging to the right\r
-            for (int j = lastres; j < startres; j++)\r
+            if (fixedColumns && fixedRight != -1)\r
             {\r
-              if (fixedColumns && fixedRight != -1)\r
+              for (int j = lastres; j < startres; j++)\r
               {\r
-                  insertChar(j, seq, fixedRight);\r
+                insertChar(j, new SequenceI[]{seq}, fixedRight);\r
               }\r
-              else\r
-                insertChar(j, seq);\r
+            }\r
+            else\r
+            {\r
+              editCommand.appendEdit(EditCommand.INSERT_GAP,\r
+                                     new SequenceI[]\r
+                                     {seq},\r
+                                     lastres, startres-lastres,\r
+                                     av.alignment,\r
+                                     true);\r
             }\r
           }\r
           else\r
           {\r
             // dragging to the left\r
-            for (int j = lastres; j > startres; j--)\r
+            if (fixedColumns && fixedRight != -1)\r
             {\r
-              if (fixedColumns && fixedRight != -1)\r
+              for (int j = lastres; j > startres; j--)\r
               {\r
-                deleteChar(startres, seq, fixedRight);\r
+                if (!jalview.util.Comparison.isGap(seq.getCharAt(startres)))\r
+                {\r
+                  endEditing();\r
+                  break;\r
+                }\r
+                deleteChar(startres, new SequenceI[]{seq}, fixedRight);\r
               }\r
-              else\r
+            }\r
+            else\r
+            {\r
+              //could be a keyboard edit trying to delete none gaps\r
+              int max=0;\r
+              for(int m = startres; m<lastres; m++)\r
+              {\r
+                if(!jalview.util.Comparison.isGap(seq.getCharAt(m)))\r
+                  break;\r
+                max++;\r
+              }\r
+\r
+              if (max>0)\r
               {\r
-                deleteChar(startres, seq);\r
+                editCommand.appendEdit(EditCommand.DELETE_GAP,\r
+                                       new SequenceI[]\r
+                                       {seq},\r
+                                       startres, max,\r
+                                       av.alignment,\r
+                                       true);\r
               }\r
             }\r
           }\r
@@ -1007,84 +1137,55 @@ public class SeqPanel extends JPanel implements MouseListener,
     }\r
 \r
 \r
-    /**\r
-     * DOCUMENT ME!\r
-     *\r
-     * @param j DOCUMENT ME!\r
-     * @param seq DOCUMENT ME!\r
-     */\r
-    void insertChar(int j, SequenceI seq)\r
-    {\r
-        seq.insertCharAt(j, av.getGapCharacter());\r
-        seqEditOccurred = true;\r
-    }\r
 \r
-    void insertChar(int j, SequenceI seq, int fixedColumn)\r
+    void insertChar(int j, SequenceI [] seq, int fixedColumn)\r
     {\r
-      //Find the next gap before the end of the visible region boundary\r
-      //If lastCol > j, theres a boundary after the gap insertion\r
       int blankColumn = fixedColumn;\r
-      for (blankColumn = fixedColumn; blankColumn > j; blankColumn--)\r
+      for(int s=0; s<seq.length; s++)\r
       {\r
-        if (jalview.util.Comparison.isGap(seq.getCharAt(blankColumn)))\r
+        //Find the next gap before the end of the visible region boundary\r
+        //If lastCol > j, theres a boundary after the gap insertion\r
+\r
+        for (blankColumn = fixedColumn; blankColumn > j; blankColumn--)\r
         {\r
-          //Theres a space, so break and insert the gap\r
-          break;\r
+          if (jalview.util.Comparison.isGap(seq[s].getCharAt(blankColumn)))\r
+          {\r
+            //Theres a space, so break and insert the gap\r
+            break;\r
+          }\r
         }\r
-      }\r
 \r
-      if (blankColumn <= j)\r
-      {\r
-        blankColumn = fixedColumn;\r
-        endEditing();\r
-        return;\r
+        if (blankColumn <= j)\r
+        {\r
+          blankColumn = fixedColumn;\r
+          endEditing();\r
+          return;\r
+        }\r
       }\r
 \r
-      if (!jalview.util.Comparison.isGap(seq.getCharAt(blankColumn)))\r
-      {\r
-        //Just Checking\r
-        System.out.println("Tried removing residue (INSERT)"+seq.getCharAt(fixedColumn));\r
-        return;\r
-      }\r
+      editCommand.appendEdit(EditCommand.DELETE_GAP,\r
+                             seq,\r
+                             blankColumn, 1, av.alignment, true);\r
 \r
-      seq.deleteCharAt(blankColumn);\r
-      seq.insertCharAt(j, av.getGapCharacter());\r
-      seqEditOccurred = true;\r
-    }\r
+      editCommand.appendEdit(EditCommand.INSERT_GAP,\r
+                             seq,\r
+                             j, 1, av.alignment,\r
+                             true);\r
 \r
-    void deleteChar(int j, SequenceI seq, int fixedColumn)\r
-    {\r
-      if (!jalview.util.Comparison.isGap(seq.getCharAt(j)))\r
-      {\r
-        ap.alignFrame.statusBar.setText(\r
-            "End editing: Tried removing residue " + seq.getCharAt(j));\r
-        return;\r
-      }\r
-\r
-      seq.deleteCharAt(j);\r
-      seq.insertCharAt(fixedColumn, av.getGapCharacter());\r
-      seqEditOccurred = true;\r
     }\r
 \r
-    /**\r
-     * DOCUMENT ME!\r
-     *\r
-     * @param j DOCUMENT ME!\r
-     * @param seq DOCUMENT ME!\r
-     */\r
-    void deleteChar(int j, SequenceI seq)\r
+    void deleteChar(int j, SequenceI [] seq, int fixedColumn)\r
     {\r
-      if (!jalview.util.Comparison.isGap(seq.getCharAt(j)))\r
-      {\r
-        ap.alignFrame.statusBar.setText(\r
-            "End editing: Tried removing residue " + seq.getCharAt(j));\r
-        return;\r
-      }\r
 \r
-        seq.deleteCharAt(j);\r
-        seqEditOccurred = true;\r
-        seqCanvas.repaint();\r
+      editCommand.appendEdit(EditCommand.DELETE_GAP,\r
+                             seq,\r
+                             j, 1, av.alignment, true);\r
+\r
+      editCommand.appendEdit(EditCommand.INSERT_GAP,\r
+                             seq,\r
+                             fixedColumn, 1, av.alignment, true);\r
     }\r
+\r
     /**\r
      * DOCUMENT ME!\r
      *\r
@@ -1121,7 +1222,38 @@ public class SeqPanel extends JPanel implements MouseListener,
     }\r
 \r
     public void mouseClicked(MouseEvent evt)\r
-    {}\r
+    {\r
+      SequenceI sequence = av.alignment.getSequenceAt( findSeq(evt) );\r
+      if(evt.getClickCount()>1)\r
+      {\r
+        if (av.getSelectionGroup().getSize() == 1\r
+            && av.getSelectionGroup().getEndRes()\r
+            - av.getSelectionGroup().getStartRes() < 2)\r
+          av.setSelectionGroup(null);\r
+\r
+        SequenceFeature[] features = findFeaturesAtRes(\r
+            sequence.getDatasetSequence(),\r
+            sequence.findPosition( findRes(evt) )\r
+            );\r
+\r
+        if(features!=null && features.length>0)\r
+        {\r
+          SearchResults highlight = new SearchResults();\r
+          highlight.addResult(sequence,\r
+                              features[0].getBegin(),\r
+                              features[0].getEnd());\r
+          seqCanvas.highlightSearchResults(highlight);\r
+        }\r
+        if(features!=null && features.length>0)\r
+        {\r
+          seqCanvas.getFeatureRenderer().amendFeatures(\r
+              new SequenceI[]\r
+              {sequence}, features, false, ap);\r
+\r
+          seqCanvas.highlightSearchResults(null);\r
+        }\r
+      }\r
+    }\r
 \r
     public void mouseWheelMoved(MouseWheelEvent e)\r
     {\r
@@ -1152,26 +1284,6 @@ public class SeqPanel extends JPanel implements MouseListener,
     }\r
 \r
 \r
-\r
-    /**\r
-     * DOCUMENT ME!\r
-     *\r
-     * @param i DOCUMENT ME!\r
-     */\r
-    void editOccurred()\r
-    {\r
-      if (!seqEditOccurred)\r
-      {\r
-        ap.alignFrame.historyList.pop();\r
-        ap.alignFrame.updateEditMenuBar();\r
-      }\r
-\r
-      endEditing();\r
-\r
-      av.firePropertyChange("alignment", null,av.getAlignment().getSequences());\r
-\r
-    }\r
-\r
     /**\r
      * DOCUMENT ME!\r
      *\r
@@ -1220,7 +1332,7 @@ public class SeqPanel extends JPanel implements MouseListener,
                 stretchGroup = null;\r
             }\r
         }\r
-        else if (!stretchGroup.getSequences(false).contains(sequence) ||\r
+        else if (!stretchGroup.getSequences(null).contains(sequence) ||\r
                 (stretchGroup.getStartRes() > res) ||\r
                 (stretchGroup.getEndRes() < res))\r
         {\r
@@ -1335,15 +1447,20 @@ public class SeqPanel extends JPanel implements MouseListener,
           if (stretchGroup.cs instanceof ClustalxColourScheme)\r
           {\r
             ( (ClustalxColourScheme) stretchGroup.cs).resetClustalX(\r
-                stretchGroup.getSequences(true),\r
+                stretchGroup.getSequences(av.hiddenRepSequences),\r
                 stretchGroup.getWidth());\r
           }\r
 \r
+          if (stretchGroup.cs instanceof Blosum62ColourScheme\r
+              || stretchGroup.cs instanceof PIDColourScheme\r
+              || stretchGroup.cs.conservationApplied()\r
+              || stretchGroup.cs.getThreshold()>0)\r
+            stretchGroup.recalcConservation();\r
+\r
           if (stretchGroup.cs.conservationApplied())\r
           {\r
             SliderPanel.setConservationSlider(ap, stretchGroup.cs,\r
                                               stretchGroup.getName());\r
-            stretchGroup.recalcConservation();\r
           }\r
           else\r
           {\r
@@ -1376,7 +1493,7 @@ public class SeqPanel extends JPanel implements MouseListener,
             return;\r
        }\r
 \r
-        if(res> av.alignment.getWidth())\r
+        if(res>= av.alignment.getWidth())\r
         {\r
           res = av.alignment.getWidth()-1;\r
         }\r
@@ -1437,7 +1554,7 @@ public class SeqPanel extends JPanel implements MouseListener,
 \r
             Sequence nextSeq = (Sequence) av.getAlignment().getSequenceAt(oldSeq);\r
 \r
-            if (stretchGroup.getSequences(false).contains(nextSeq))\r
+            if (stretchGroup.getSequences(null).contains(nextSeq))\r
             {\r
                 stretchGroup.deleteSequence(seq, false);\r
             }\r