JAL-3397 impl.IntervalStore and nonc.IntervalStore unified api

[jalview.git] / src / jalview / renderer / OverviewResColourFinder.java

diff --git a/src/jalview/renderer/OverviewResColourFinder.java b/src/jalview/renderer/OverviewResColourFinder.java
index a98f3b3..e0d6696 100644 (file)
--- a/src/jalview/renderer/OverviewResColourFinder.java
+++ b/src/jalview/renderer/OverviewResColourFinder.java
@@ -29,6 +29,13 @@ import java.awt.Color;
 
 public class OverviewResColourFinder extends ResidueColourFinder
 {
+  public static final Color OVERVIEW_DEFAULT_GAP = Color.lightGray;
+
+  public static final Color OVERVIEW_DEFAULT_LEGACY_GAP = Color.white;
+
+  public static final Color OVERVIEW_DEFAULT_HIDDEN = Color.darkGray
+          .darker();
+
   final int GAP_COLOUR; // default colour to use at gaps
 
   final int RESIDUE_COLOUR; // default colour to use at residues
@@ -37,13 +44,6 @@ public class OverviewResColourFinder extends ResidueColourFinder
 
   boolean useLegacy = false;
 
-  public static final Color OVERVIEW_DEFAULT_GAP = Color.lightGray;
-
-  public static final Color OVERVIEW_DEFAULT_LEGACY_GAP = Color.white;
-
-  public static final Color OVERVIEW_DEFAULT_HIDDEN = Color.darkGray
-          .darker();
-
   /**
    * Constructor without colour settings (used by applet)
    */
@@ -130,9 +130,9 @@ public class OverviewResColourFinder extends ResidueColourFinder
 
     // if there's a FeatureColourFinder we might override the residue colour
     // here with feature colouring
-    return seq.setColor(i,
-            finder == null || finder.noFeaturesDisplayed() ? col
-                    : finder.findFeatureColourInt(col, seq, i));
+    col = finder == null || finder.noFeaturesDisplayed() ? col
+            : finder.findFeatureColourInt(col, seq, i);
+    return seq.setColor(i, col);
   }
 
   /**