JAL-3210 Barebones gradle/buildship/eclipse. See README
[jalview.git] / src / jalview / gui / OverviewCanvas.java
index de55996..ca27a9c 100644 (file)
@@ -22,7 +22,6 @@ package jalview.gui;
 
 import jalview.api.AlignViewportI;
 import jalview.bin.Cache;
-import jalview.datamodel.AlignmentI;
 import jalview.renderer.OverviewRenderer;
 import jalview.renderer.OverviewResColourFinder;
 import jalview.viewmodel.OverviewDimensions;
@@ -34,7 +33,6 @@ import java.awt.image.BufferedImage;
 
 import javax.swing.JPanel;
 
-@SuppressWarnings("serial")
 public class OverviewCanvas extends JPanel
 {
   private static final Color TRANS_GREY = new Color(100, 100, 100, 25);
@@ -45,9 +43,11 @@ public class OverviewCanvas extends JPanel
 
   private volatile boolean updaterunning = false;
 
-  private boolean disposed = false;
+  private boolean dispose = false;
 
-  BufferedImage lastMiniMe = null;
+  private BufferedImage miniMe;
+
+  private BufferedImage lastMiniMe = null;
 
   // Can set different properties in this seqCanvas than
   // main visible SeqCanvas
@@ -65,26 +65,13 @@ public class OverviewCanvas extends JPanel
 
   private ProgressPanel progressPanel;
 
-  private boolean showSequenceFeatures;
-
-  private boolean showAnnotation;
-
-  private jalview.api.FeatureRenderer featureRenderer;
-
-  private OverviewPanel panel;
-
-  private boolean showProgress;
-
-  public OverviewCanvas(OverviewPanel panel,
-          OverviewDimensions overviewDims,
+  public OverviewCanvas(OverviewDimensions overviewDims,
           AlignViewportI alignvp, ProgressPanel pp)
   {
-    this.panel = panel;
     od = overviewDims;
-    lastMiniMe = null;
     av = alignvp;
     progressPanel = pp;
-    showProgress = (pp != null);
+
     sr = new SequenceRenderer(av);
     sr.renderGaps = false;
     fr = new jalview.renderer.seqfeatures.FeatureRenderer(av);
@@ -97,7 +84,6 @@ public class OverviewCanvas extends JPanel
     cf = new OverviewResColourFinder(useLegacy, gapCol, hiddenCol);
 
     setSize(od.getWidth(), od.getHeight());
-    setPreferredSize(getSize()); // BH 2019.07.29 added
   }
 
   /**
@@ -109,7 +95,6 @@ public class OverviewCanvas extends JPanel
   public void resetOviewDims(OverviewDimensions overviewDims)
   {
     od = overviewDims;
-    lastMiniMe = null;
   }
 
   /**
@@ -122,27 +107,20 @@ public class OverviewCanvas extends JPanel
     {
       if (updaterunning)
       {
-        setRestart("restartDraw");
+        restart = true;
+        if (or != null)
+        {
+          or.setRedraw(true);
+        }
       }
       else
       {
         updaterunning = true;
-        restart = false;
       }
       return restart;
     }
   }
 
-  private void setRestart(String why)
-  {
-    // System.out.println("OC restart true " + why);
-    restart = true;
-    if (or != null)
-    {
-      or.setRedraw(true);
-    }
-  }
-
   /**
    * Draw the overview sequences
    * 
@@ -150,173 +128,145 @@ public class OverviewCanvas extends JPanel
    *          true if sequence features are to be shown
    * @param showAnnotation
    *          true if the annotation is to be shown
-   * @param featureRenderer
+   * @param transferRenderer
    *          the renderer to transfer feature colouring from
    */
   public void draw(boolean showSequenceFeatures, boolean showAnnotation,
-          jalview.api.FeatureRenderer featureRenderer)
+          FeatureRenderer transferRenderer)
   {
-    this.showSequenceFeatures = showSequenceFeatures;
-    this.showAnnotation = showAnnotation;
-    this.featureRenderer = featureRenderer;
+    miniMe = null;
+
     if (showSequenceFeatures)
     {
-      fr.transferSettings(featureRenderer);
+      fr.transferSettings(transferRenderer);
     }
+
     setPreferredSize(new Dimension(od.getWidth(), od.getHeight()));
-    AlignmentI al = av.getAlignment();
-    or = new OverviewRenderer(panel.ap, fr, od, al, av.getResidueShading(),
-            cf, showProgress);
-    if (showProgress)
-    {
-      or.addPropertyChangeListener(progressPanel);
-    }
-    or.drawMiniMe();
-  }
 
-  synchronized void finalizeDraw(BufferedImage miniMe)
-  {
+    or = new OverviewRenderer(fr, od, av.getAlignment(),
+            av.getResidueShading(), cf);
 
-    if (showProgress && or != null)
+    or.addPropertyChangeListener(progressPanel);
+
+    miniMe = or.draw(od.getRows(av.getAlignment()),
+            od.getColumns(av.getAlignment()));
+
+    Graphics mg = miniMe.getGraphics();
+
+    if (showAnnotation)
     {
-      or.removePropertyChangeListener(progressPanel);
+      mg.translate(0, od.getSequencesHeight());
+      or.drawGraph(mg, av.getAlignmentConservationAnnotation(),
+              od.getGraphHeight(), od.getColumns(av.getAlignment()));
+      mg.translate(0, -od.getSequencesHeight());
     }
+
+    or.removePropertyChangeListener(progressPanel);
+    or = null;
     if (restart)
     {
-      or = null;
       restart = false;
-      if (!disposed)
+      if (!dispose)
       {
-        draw(showSequenceFeatures, showAnnotation, featureRenderer);
+        draw(showSequenceFeatures, showAnnotation, transferRenderer);
       }
     }
     else
     {
-      if (showAnnotation && or != null)
-      {
-        or.drawGraph(av.getAlignmentConservationAnnotation());
-      }
-      or = null;
       updaterunning = false;
       lastMiniMe = miniMe;
-      repaint();
     }
   }
 
   @Override
   public void paintComponent(Graphics g)
   {
+    //super.paintComponent(g);
 
-    int w = getWidth();
-    int h = getHeight();
-    if (w == 0 || od.getBoxWidth() <= 0)
-    {
-      // BH 2019.07.27 removes two unnecessary paints, since boxwidth can be -1
-      // or 0 during early-stage painting
-      return;
-    }
-
-    boolean drawMe = (lastMiniMe != null);
     if (restart)
     {
-      if (drawMe)
+      if (lastMiniMe == null)
       {
-        g.drawImage(lastMiniMe, 0, 0, w, h, this);
+        g.setColor(Color.white);
+        g.fillRect(0, 0, getWidth(), getHeight());
       }
       else
       {
-        g.setColor(Color.white);
-        g.fillRect(0, 0, w, h);
+        g.drawImage(lastMiniMe, 0, 0, getWidth(), getHeight(), this);
       }
       g.setColor(TRANS_GREY);
-      g.fillRect(0, 0, w, h);
-      drawMe = false;
+      g.fillRect(0, 0, getWidth(), getHeight());
     }
-    else if (drawMe)
+    else if (lastMiniMe != null)
     {
       // is this a resize?
-      if (w != od.getWidth() || h != od.getHeight())
+      if ((getWidth() > 0) && (getHeight() > 0)
+              && ((getWidth() != od.getWidth())
+                      || (getHeight() != od.getHeight())))
       {
-
-        lastMiniMe = null;
-        return;
-        // // if there is annotation, scale the alignment and annotation
-        // // separately
-        // if (od.getGraphHeight() <= 0 && od.getSequencesHeight() <= 0)
-        // {
-        // od.setWidth(w);
-        // od.setHeight(h);
-        // return;
-        // }
-        // try
-        // {
-        // BufferedImage topImage = lastMiniMe.getSubimage(0, 0,
-        // od.getWidth(), od.getSequencesHeight());
-        //
-        // BufferedImage bottomImage = lastMiniMe.getSubimage(0,
-        // od.getSequencesHeight(), od.getWidth(),
-        // od.getGraphHeight());
-        //
-        // // must be done at this point as we rely on using old width/height
-        // // above, and new width/height below
-        // od.setWidth(w);
-        // od.setHeight(h);
-        //
-        // // stick the images back together so lastMiniMe is consistent in the
-        // // event of a repaint - BUT probably not thread safe
-        //
-        // // right -- this fails with fast user action.
-        //
-        // lastMiniMe = new BufferedImage(w, h, BufferedImage.TYPE_INT_RGB);
-        // Graphics lg = lastMiniMe.getGraphics();
-        // lg.drawImage(topImage, 0, 0, w, od.getSequencesHeight(), null);
-        // lg.drawImage(bottomImage, 0, od.getSequencesHeight(), w,
-        // od.getGraphHeight(), this);
-        // lg.dispose();
-        //
-        // } catch (RasterFormatException e)
-        // {
-        // System.out.println("OC Raster Exception " + lastMiniMe.getWidth()
-        // + "/" + w + "," + lastMiniMe.getHeight() + "/" + h + " "
-        // + od.getSequencesHeight() + " " + od.getGraphHeight());
-        // }
-        // BH 2019: removed -- this is now taken care of using vpbox in
-        // OverviewDimension
-        // // make sure the box is in the right place
-        // od.setBoxPosition(av.getAlignment().getHiddenSequences(),
-        // av.getAlignment().getHiddenColumns());
+        // if there is annotation, scale the alignment and annotation
+        // separately
+        if (od.getGraphHeight() > 0 
+                       && od.getSequencesHeight() > 0  // BH 2019
+                       )
+        {
+          BufferedImage topImage = lastMiniMe.getSubimage(0, 0,
+                  od.getWidth(), od.getSequencesHeight());
+          BufferedImage bottomImage = lastMiniMe.getSubimage(0,
+                  od.getSequencesHeight(), od.getWidth(),
+                  od.getGraphHeight());
+
+          // must be done at this point as we rely on using old width/height
+          // above, and new width/height below
+          od.setWidth(getWidth());
+          od.setHeight(getHeight());
+
+          // stick the images back together so lastMiniMe is consistent in the
+          // event of a repaint - BUT probably not thread safe
+          lastMiniMe = new BufferedImage(od.getWidth(), od.getHeight(),
+                  BufferedImage.TYPE_INT_RGB);
+          Graphics lg = lastMiniMe.getGraphics();
+          lg.drawImage(topImage, 0, 0, od.getWidth(),
+                  od.getSequencesHeight(), null);
+          lg.drawImage(bottomImage, 0, od.getSequencesHeight(),
+                  od.getWidth(), od.getGraphHeight(), this);
+          lg.dispose();
+        }
+        else
+        {
+          od.setWidth(getWidth());
+          od.setHeight(getHeight());
+        }
+
+        // make sure the box is in the right place
+        od.setBoxPosition(av.getAlignment().getHiddenSequences(),
+                av.getAlignment().getHiddenColumns());
       }
+      // fall back to normal behaviour
+      g.drawImage(lastMiniMe, 0, 0, getWidth(), getHeight(), this);
     }
-
-    if (drawMe)
+    else
     {
-      g.drawImage(lastMiniMe, 0, 0, w, h, this);
+      g.drawImage(lastMiniMe, 0, 0, getWidth(), getHeight(), this);
     }
+    
     // draw the box
     g.setColor(Color.red);
-    // System.out.println("OC paintComponent nd=" + ndraw + " nr=" + nrepaint
-    // + " np=" + ++npaint);
     od.drawBox(g);
   }
 
-  // private int ndraw, npaint, nrepaint;
-
-  // @Override
-  // public void repaint()
-  // {
-  // System.out.println("OC repaint " + (++nrepaint));
-  // super.repaint();
-  // }
 
   public void dispose()
   {
-    disposed = true;
+    dispose = true;
     od = null;
-    lastMiniMe = null;
     synchronized (this)
     {
-      setRestart("dispose");
+      restart = true;
+      if (or != null)
+      {
+        or.setRedraw(true);
+      }
     }
   }
-
 }