From: gmungoc <g.m.carstairs@dundee.ac.uk>
Date: Tue, 6 Nov 2018 17:05:23 +0000 (+0000)
Subject: JAL-3148 further bug fix to state of 'colourBy'
X-Git-Url: http://source.jalview.org/gitweb/?a=commitdiff_plain;h=3ed1950a0283cbf7f312f58d9aabbbd21abfa4a8;p=jalview.git

JAL-3148 further bug fix to state of 'colourBy'
---

diff --git a/src/jalview/structures/models/AAStructureBindingModel.java b/src/jalview/structures/models/AAStructureBindingModel.java
index a53674f..f172490 100644
--- a/src/jalview/structures/models/AAStructureBindingModel.java
+++ b/src/jalview/structures/models/AAStructureBindingModel.java
@@ -844,8 +844,15 @@ public abstract class AAStructureBindingModel
     colourBy = ColourBy.ChargeAndCysteine;
   }
 
+  /**
+   * Colour the structure as per the sequence alignment
+   * 
+   * @param alignmentv
+   * @param showFeatures
+   */
   public void colourBySequence(AlignmentViewPanel alignmentv, boolean showFeatures)
   {
+    colourBy = ColourBy.Sequence;
     colourBySequence(alignmentv, alignmentv.getSequenceRenderer(), showFeatures);
   }
 
@@ -856,7 +863,6 @@ public abstract class AAStructureBindingModel
   public void colourBySequence(AlignmentViewPanel alignmentv, SequenceRendererI sr, boolean showFeats)
   {
     showFeatures = showFeats;
-    colourBy = ColourBy.Sequence;
     
     if (!isLoadingFinished())
     {