JAL-2524 don't render up to a hidden marker beyond the visible region

[jalview.git] / src / jalview / gui / OverviewCanvas.java

diff --git a/src/jalview/gui/OverviewCanvas.java b/src/jalview/gui/OverviewCanvas.java
index e4c2056..6f9fbbf 100644 (file)
--- a/src/jalview/gui/OverviewCanvas.java
+++ b/src/jalview/gui/OverviewCanvas.java
@@ -21,7 +21,7 @@
 package jalview.gui;
 
 import jalview.api.AlignViewportI;
-import jalview.renderer.seqfeatures.FeatureColourFinder;
+import jalview.renderer.OverviewRenderer;
 import jalview.viewmodel.OverviewDimensions;
 
 import java.awt.Color;
@@ -31,7 +31,6 @@ import java.awt.image.BufferedImage;
 
 import javax.swing.JComponent;
 
-
 public class OverviewCanvas extends JComponent
 {
   private static final Color TRANS_GREY = new Color(100, 100, 100, 25);
@@ -52,9 +51,9 @@ public class OverviewCanvas extends JComponent
 
   private jalview.renderer.seqfeatures.FeatureRenderer fr;
 
-  OverviewDimensions od;
+  private OverviewDimensions od;
 
-  AlignViewportI av;
+  private AlignViewportI av;
 
   public OverviewCanvas(OverviewDimensions overviewDims,
           AlignViewportI alignvp)
@@ -68,14 +67,20 @@ public class OverviewCanvas extends JComponent
     fr = new jalview.renderer.seqfeatures.FeatureRenderer(av);
   }
 
+  /**
+   * Update the overview dimensions object used by the canvas (e.g. if we change
+   * from showing hidden columns to hiding them or vice versa)
+   * 
+   * @param overviewDims
+   */
   public void resetOviewDims(OverviewDimensions overviewDims)
   {
     od = overviewDims;
   }
 
-  /*
-   * Signals to drawing code that the associated alignment viewport
-   * has changed and a redraw will be required
+  /**
+   * Signals to drawing code that the associated alignment viewport has changed
+   * and a redraw will be required
    */
   public boolean restartDraw()
   {
@@ -93,6 +98,16 @@ public class OverviewCanvas extends JComponent
     }
   }
 
+  /**
+   * Draw the overview sequences
+   * 
+   * @param showSequenceFeatures
+   *          true if sequence features are to be shown
+   * @param showAnnotation
+   *          true if the annotation is to be shown
+   * @param transferRenderer
+   *          the renderer to transfer feature colouring from
+   */
   public void draw(boolean showSequenceFeatures, boolean showAnnotation,
           FeatureRenderer transferRenderer)
   {
@@ -102,15 +117,12 @@ public class OverviewCanvas extends JComponent
     {
       fr.transferSettings(transferRenderer);
     }
-    FeatureColourFinder finder = new FeatureColourFinder(fr);
 
-    // why do we need to set preferred size again? was set in
-    // updateOverviewImage
     setPreferredSize(new Dimension(od.getWidth(), od.getHeight()));
 
-    OverviewRenderer or = new OverviewRenderer(sr, finder, od);
-    miniMe = or.draw(od.getRows(av.getRanges(), av.getAlignment()),
-            od.getColumns(av.getRanges(), av.getColumnSelection()));
+    OverviewRenderer or = new OverviewRenderer(sr, fr, od);
+    miniMe = or.draw(od.getRows(av.getAlignment()),
+            od.getColumns(av.getAlignment()));
 
     Graphics mg = miniMe.getGraphics();
 
@@ -119,7 +131,7 @@ public class OverviewCanvas extends JComponent
       mg.translate(0, od.getSequencesHeight());
       or.drawGraph(mg, av.getAlignmentConservationAnnotation(),
               av.getCharWidth(), od.getGraphHeight(),
-              od.getColumns(av.getRanges(), av.getColumnSelection()));
+              od.getColumns(av.getAlignment()));
       mg.translate(0, -od.getSequencesHeight());
     }
     System.gc();