JAL-3187 clunky sort by renderorder in findComplementFeaturesAtResidue

[jalview.git] / src / jalview / renderer / seqfeatures / FeatureRenderer.java

diff --git a/src/jalview/renderer/seqfeatures/FeatureRenderer.java b/src/jalview/renderer/seqfeatures/FeatureRenderer.java
index ac8f6d9..3989b0c 100644 (file)
--- a/src/jalview/renderer/seqfeatures/FeatureRenderer.java
+++ b/src/jalview/renderer/seqfeatures/FeatureRenderer.java
@@ -28,6 +28,7 @@ import jalview.datamodel.SequenceI;
 import jalview.gui.AlignFrame;
 import jalview.gui.Desktop;
 import jalview.util.Comparison;
+import jalview.util.ReverseListIterator;
 import jalview.viewmodel.seqfeatures.FeatureRendererModel;
 
 import java.awt.AlphaComposite;
@@ -408,7 +409,6 @@ public class FeatureRenderer extends FeatureRendererModel
       for (int pos = visiblePositions.start; pos <= visiblePositions.end; pos++)
       {
         int column = seq.findIndex(pos);
-        // TODO ensure these are in complement's render order (last on top)
         List<SequenceFeature> features = fr2
                 .findComplementFeaturesAtResidue(seq, pos);
         for (SequenceFeature sf : features)
@@ -478,8 +478,11 @@ public class FeatureRenderer extends FeatureRendererModel
       List<SequenceFeature> features = fr2.findComplementFeaturesAtResidue(
               seq, seq.findPosition(column));
       // todo: ensure ordered by feature render order
-      for (SequenceFeature sf : features)
+      ReverseListIterator<SequenceFeature> it = new ReverseListIterator<>(
+              features);
+      while (it.hasNext())
       {
+        SequenceFeature sf = it.next();
         if (!fr2.featureGroupNotShown(sf))
         {
           Color col = fr2.getColour(sf);