X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fgui%2FSeqPanel.java;h=3595fe626c6bcf948b2ba0355f499edc15f65c41;hb=bed54fafd288a7cf39f15262522b0f2503883479;hp=55bec56139156b54db198696e1010d9c89542289;hpb=99cb058c4b312055ca758890b10d41428105dabe;p=jalview.git

diff --git a/src/jalview/gui/SeqPanel.java b/src/jalview/gui/SeqPanel.java
index 55bec56..3595fe6 100755
--- a/src/jalview/gui/SeqPanel.java
+++ b/src/jalview/gui/SeqPanel.java
@@ -250,7 +250,9 @@ public class SeqPanel extends JPanel
 	// Group editing
 	if (evt.isAltDown() || evt.isControlDown())
         {
-	  SequenceGroup sg = av.getAlignment().findGroup(startseq);
+	  SequenceGroup sg = av.getSelectionGroup();
+          if(sg==null)
+            av.getAlignment().findGroup(startseq);
           if (sg != null)
           {
             boolean deleteAllowed = false;
@@ -360,21 +362,8 @@ public class SeqPanel extends JPanel
 
   void updateConservation(int i)
   {
-  /*  Alignment al = (Alignment) av.getAlignment();
-    SequenceGroup sg = av.alignment.findGroup( al.getSequenceAt(i));
-    if(sg==null || !(sg.cs instanceof ConservationColourScheme))
-      return;
-
-    Conservation c = sg.getConservation();
-
-    c = new Conservation("All", al.cons,
-                         ResidueProperties.propHash, 3, sg.sequences, 0,
-                         al.getWidth());
-    c.calculate();
-    c.verdict(false, 100);
-    sg.setConservation(c);
-    ConservationColourScheme ccs = (ConservationColourScheme)sg.cs;
-    ccs.conserve = c;*/
+    av.updateConservation();
+    av.updateConsensus();
   }
 
 //////////////////////////////////////////
@@ -397,7 +386,10 @@ public class SeqPanel extends JPanel
     if(stretchGroup == null)
      {
        stretchGroup = av.alignment.findGroup( sequence );
-       av.setSelectionGroup( stretchGroup );
+       if(stretchGroup!=null && res>stretchGroup.getStartRes() && res<stretchGroup.getEndRes())
+         av.setSelectionGroup(stretchGroup);
+       else
+         stretchGroup = null;
      }
 
     else if(!stretchGroup.sequences.contains(sequence)
@@ -457,7 +449,6 @@ public class SeqPanel extends JPanel
       changeStartRes = true;
 
 
-    seqCanvas.paintFlag = true;
     repaint();
 
   }
@@ -478,7 +469,6 @@ public class SeqPanel extends JPanel
     if(stretchGroup.cs instanceof ClustalxColourScheme)
     {
       stretchGroup.cs = new ClustalxColourScheme(stretchGroup.sequences, av.alignment.getWidth());
-      seqCanvas.paintFlag = true;
       repaint();
     }
 
@@ -486,20 +476,12 @@ public class SeqPanel extends JPanel
     {
       ConservationColourScheme ccs = (ConservationColourScheme)stretchGroup.cs;
 
-      Conservation c = new Conservation("Group",
-                                        ResidueProperties.propHash, 3, stretchGroup.sequences, 0,
-                                        av.alignment.getWidth() );
-
-       c.calculate();
-       c.verdict(false, 100);
-       ccs = new ConservationColourScheme(c, ccs.cs);
 
        stretchGroup.cs = ccs;
 
 
        Desktop.setConservationSliderSource(ap, stretchGroup.cs, stretchGroup.getName()) ;
 
-       seqCanvas.paintFlag = true;
        repaint();
     }
     else
@@ -562,7 +544,7 @@ public class SeqPanel extends JPanel
     else if (y < oldSeq)
       dragDirection = -1;
 
-    while (y != oldSeq && oldSeq>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.
@@ -587,7 +569,6 @@ public class SeqPanel extends JPanel
     if(scrollThread!=null)
       scrollThread.setEvent(evt);
 
-    seqCanvas.paintFlag = true;
     repaint();
   }
 
@@ -648,7 +629,6 @@ public class SeqPanel extends JPanel
             running = ap.scrollRight(false);
         }
 
-        ap.repaint();
         try
         {
           Thread.sleep(75);