X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fgui%2FAlignmentPanel.java;h=65546556e218a42f45c4c4bb2f7bc80d46cf1294;hb=608282e79f4dd02c92565235b7946265bdca847d;hp=4d49bc2c742cc7abf8c31c849b6ce8e665d3a0be;hpb=c8ae8ef3f6365ea5664930f3139fa4871eb188c2;p=jalview.git diff --git a/src/jalview/gui/AlignmentPanel.java b/src/jalview/gui/AlignmentPanel.java index 4d49bc2..6554655 100644 --- a/src/jalview/gui/AlignmentPanel.java +++ b/src/jalview/gui/AlignmentPanel.java @@ -101,6 +101,8 @@ public class AlignmentPanel extends GAlignmentPanel implements */ private boolean dontScrollComplement; + private PropertyChangeListener propertyChangeListener; + /** * Creates a new AlignmentPanel object. * @@ -135,8 +137,9 @@ public class AlignmentPanel extends GAlignmentPanel implements vscroll.addAdjustmentListener(this); final AlignmentPanel ap = this; - av.addPropertyChangeListener(new PropertyChangeListener() + propertyChangeListener = new PropertyChangeListener() { + @Override public void propertyChange(PropertyChangeEvent evt) { if (evt.getPropertyName().equals("alignment")) @@ -145,7 +148,8 @@ public class AlignmentPanel extends GAlignmentPanel implements alignmentChanged(); } } - }); + }; + av.addPropertyChangeListener(propertyChangeListener); fontChanged(); adjustAnnotationHeight(); updateLayout(); @@ -509,6 +513,7 @@ public class AlignmentPanel extends GAlignmentPanel implements * automatically adjust annotation panel height for new annotation whilst * ensuring the alignment is still visible. */ + @Override public void adjustAnnotationHeight() { // TODO: display vertical annotation scrollbar if necessary @@ -749,6 +754,7 @@ public class AlignmentPanel extends GAlignmentPanel implements * @param evt * DOCUMENT ME! */ + @Override public void adjustmentValueChanged(AdjustmentEvent evt) { int oldX = av.getStartRes(); @@ -782,6 +788,7 @@ public class AlignmentPanel extends GAlignmentPanel implements // as preference setting SwingUtilities.invokeLater(new Runnable() { + @Override public void run() { setScrollValues(av.getStartRes(), av.getStartSeq()); @@ -852,6 +859,7 @@ public class AlignmentPanel extends GAlignmentPanel implements * Repaint the alignment including the annotations and overview panels (if * shown). */ + @Override public void paintAlignment(boolean updateOverview) { final AnnotationSorter sorter = new AnnotationSorter(getAlignment(), @@ -878,6 +886,7 @@ public class AlignmentPanel extends GAlignmentPanel implements * @param g * DOCUMENT ME! */ + @Override public void paintComponent(Graphics g) { invalidate(); @@ -936,6 +945,7 @@ public class AlignmentPanel extends GAlignmentPanel implements * @throws PrinterException * DOCUMENT ME! */ + @Override public int print(Graphics pg, PageFormat pf, int pi) throws PrinterException { @@ -946,11 +956,11 @@ public class AlignmentPanel extends GAlignmentPanel implements if (av.getWrapAlignment()) { - return printWrappedAlignment(pg, pwidth, pheight, pi); + return printWrappedAlignment(pwidth, pheight, pi, pg); } else { - return printUnwrapped(pg, pwidth, pheight, pi); + return printUnwrapped(pwidth, pheight, pi, pg, pg); } } @@ -971,84 +981,103 @@ public class AlignmentPanel extends GAlignmentPanel implements * @throws PrinterException * DOCUMENT ME! */ - public int printUnwrapped(Graphics pg, int pwidth, int pheight, int pi) + /** + * Draws the alignment image, including sequence ids, sequences, and + * annotation labels and annotations if shown, on either one or two Graphics + * context. + * + * @param pageWidth + * @param pageHeight + * @param pi + * @param idGraphics + * the graphics context for sequence ids and annotation labels + * @param alignmentGraphics + * the graphics context for sequences and annotations (may or may not + * be the same context as idGraphics) + * @return + * @throws PrinterException + */ + public int printUnwrapped(int pageWidth, int pageHeight, int pi, + Graphics idGraphics, Graphics alignmentGraphics) throws PrinterException { - int idWidth = getVisibleIdWidth(false); - FontMetrics fm = getFontMetrics(av.getFont()); - int scaleHeight = av.getCharHeight() + fm.getDescent(); + final int idWidth = getVisibleIdWidth(false); - pg.setColor(Color.white); - pg.fillRect(0, 0, pwidth, pheight); - pg.setFont(av.getFont()); - - // ////////////////////////////////// - // / How many sequences and residues can we fit on a printable page? - int totalRes = (pwidth - idWidth) / av.getCharWidth(); + /* + * Get the horizontal offset to where we draw the sequences. + * This is idWidth if using a single Graphics context, else zero. + */ + final int alignmentGraphicsOffset = idGraphics != alignmentGraphics ? 0 : idWidth; - int totalSeq = (pheight - scaleHeight) / av.getCharHeight() - 1; + FontMetrics fm = getFontMetrics(av.getFont()); + int charHeight = av.getCharHeight(); + int scaleHeight = charHeight + fm.getDescent(); - int pagesWide = (av.getAlignment().getWidth() / totalRes) + 1; + idGraphics.setColor(Color.white); + idGraphics.fillRect(0, 0, pageWidth, pageHeight); + idGraphics.setFont(av.getFont()); - // /////////////////////////// - // / Only print these sequences and residues on this page - int startRes; + /* + * How many sequences and residues can we fit on a printable page? + */ + int totalRes = (pageWidth - idWidth) / av.getCharWidth(); - // /////////////////////////// - // / Only print these sequences and residues on this page - int endRes; + int totalSeq = (pageHeight - scaleHeight) / charHeight - 1; - // /////////////////////////// - // / Only print these sequences and residues on this page - int startSeq; + int alignmentWidth = av.getAlignment().getWidth(); + int pagesWide = (alignmentWidth / totalRes) + 1; - // /////////////////////////// - // / Only print these sequences and residues on this page - int endSeq; - startRes = (pi % pagesWide) * totalRes; - endRes = (startRes + totalRes) - 1; + final int startRes = (pi % pagesWide) * totalRes; + int endRes = (startRes + totalRes) - 1; - if (endRes > (av.getAlignment().getWidth() - 1)) + if (endRes > (alignmentWidth - 1)) { - endRes = av.getAlignment().getWidth() - 1; + endRes = alignmentWidth - 1; } - startSeq = (pi / pagesWide) * totalSeq; - endSeq = startSeq + totalSeq; + final int startSeq = (pi / pagesWide) * totalSeq; + int endSeq = startSeq + totalSeq; - if (endSeq > av.getAlignment().getHeight()) + int alignmentHeight = av.getAlignment().getHeight(); + if (endSeq > alignmentHeight) { - endSeq = av.getAlignment().getHeight(); + endSeq = alignmentHeight; } - int pagesHigh = ((av.getAlignment().getHeight() / totalSeq) + 1) - * pheight; + int pagesHigh = ((alignmentHeight / totalSeq) + 1) + * pageHeight; if (av.isShowAnnotation()) { pagesHigh += getAnnotationPanel().adjustPanelHeight() + 3; } - pagesHigh /= pheight; + pagesHigh /= pageHeight; if (pi >= (pagesWide * pagesHigh)) { return Printable.NO_SUCH_PAGE; } + final int alignmentDrawnHeight = (endSeq - startSeq) * charHeight + + 3; - // draw Scale - pg.translate(idWidth, 0); - getScalePanel().drawScale(pg, startRes, endRes, pwidth - idWidth, - scaleHeight); - pg.translate(-idWidth, scaleHeight); + /* + * draw the Scale at horizontal offset, then reset to origin + */ + alignmentGraphics.translate(alignmentGraphicsOffset, 0); + getScalePanel().drawScale(alignmentGraphics, startRes, endRes, + pageWidth - idWidth, scaleHeight); + alignmentGraphics.translate(-alignmentGraphicsOffset, 0); - // ////////////// - // Draw the ids + /* + * Draw the sequence ids, offset for scale height, + * then reset to origin + */ + idGraphics.translate(0, scaleHeight); + idGraphics.setFont(getIdPanel().getIdCanvas().getIdfont()); Color currentColor = null; Color currentTextColor = null; - pg.setFont(getIdPanel().getIdCanvas().getIdfont()); - SequenceI seq; for (int i = startSeq; i < endSeq; i++) { @@ -1056,6 +1085,9 @@ public class AlignmentPanel extends GAlignmentPanel implements if ((av.getSelectionGroup() != null) && av.getSelectionGroup().getSequences(null).contains(seq)) { + /* + * gray out ids of sequences in selection group (if any) + */ currentColor = Color.gray; currentTextColor = Color.black; } @@ -1065,45 +1097,55 @@ public class AlignmentPanel extends GAlignmentPanel implements currentTextColor = Color.black; } - pg.setColor(currentColor); - pg.fillRect(0, (i - startSeq) * av.getCharHeight(), idWidth, - av.getCharHeight()); + idGraphics.setColor(currentColor); + idGraphics.fillRect(0, (i - startSeq) * charHeight, idWidth, + charHeight); - pg.setColor(currentTextColor); + idGraphics.setColor(currentTextColor); int xPos = 0; + String displayId = seq.getDisplayId(av.getShowJVSuffix()); if (av.isRightAlignIds()) { - fm = pg.getFontMetrics(); + fm = idGraphics.getFontMetrics(); xPos = idWidth - - fm.stringWidth(seq.getDisplayId(av.getShowJVSuffix())) + - fm.stringWidth(displayId) - 4; } - pg.drawString(seq.getDisplayId(av.getShowJVSuffix()), xPos, - (((i - startSeq) * av.getCharHeight()) + av.getCharHeight()) - - (av.getCharHeight() / 5)); + idGraphics.drawString(displayId, xPos, + (((i - startSeq) * charHeight) + charHeight) + - (charHeight / 5)); } + idGraphics.setFont(av.getFont()); + idGraphics.translate(0, -scaleHeight); - pg.setFont(av.getFont()); - - // draw main sequence panel - pg.translate(idWidth, 0); - getSeqPanel().seqCanvas.drawPanel(pg, startRes, endRes, startSeq, - endSeq, 0); + /* + * draw the sequences, offset for scale height, and id width (if using a + * single graphics context), then reset to origin + scale height + */ + alignmentGraphics.translate(alignmentGraphicsOffset, scaleHeight); + getSeqPanel().seqCanvas.drawPanel(alignmentGraphics, startRes, endRes, + startSeq, endSeq, 0); + alignmentGraphics.translate(-alignmentGraphicsOffset, 0); - if (av.isShowAnnotation() && (endSeq == av.getAlignment().getHeight())) + if (av.isShowAnnotation() && (endSeq == alignmentHeight)) { - // draw annotation - need to offset for current scroll position + /* + * draw annotation labels, offset for current scroll position + * then reset to origin + scale height + */ int offset = -getAlabels().getScrollOffset(); - pg.translate(0, offset); - pg.translate(-idWidth - 3, (endSeq - startSeq) * av.getCharHeight() - + 3); - getAlabels().drawComponent(pg, idWidth); - pg.translate(idWidth + 3, 0); + idGraphics.translate(0, offset + alignmentDrawnHeight); + getAlabels().drawComponent(idGraphics, idWidth); + idGraphics.translate(0, -offset - alignmentDrawnHeight); + + /* + * draw the annotations + */ + alignmentGraphics.translate(alignmentGraphicsOffset, alignmentDrawnHeight); getAnnotationPanel().renderer.drawComponent(getAnnotationPanel(), av, - pg, -1, startRes, endRes + 1); - pg.translate(0, -offset); + alignmentGraphics, -1, startRes, endRes + 1); } return Printable.PAGE_EXISTS; @@ -1126,8 +1168,8 @@ public class AlignmentPanel extends GAlignmentPanel implements * @throws PrinterException * DOCUMENT ME! */ - public int printWrappedAlignment(Graphics pg, int pwidth, int pheight, - int pi) throws PrinterException + public int printWrappedAlignment(int pwidth, int pheight, int pi, + Graphics pg) throws PrinterException { int annotationHeight = 0; AnnotationLabels labels = null; @@ -1262,14 +1304,18 @@ public class AlignmentPanel extends GAlignmentPanel implements void makeAlignmentImage(jalview.util.ImageMaker.TYPE type, File file) { - long progress = System.currentTimeMillis(); + int boarderBottomOffset = 5; + long pSessionId = System.currentTimeMillis(); headless = (System.getProperty("java.awt.headless") != null && System .getProperty("java.awt.headless").equals("true")); if (alignFrame != null && !headless) { - alignFrame.setProgressBar(MessageManager.formatMessage( - "status.saving_file", new Object[] { type.getLabel() }), - progress); + if (file != null) + { + alignFrame.setProgressBar(MessageManager.formatMessage( + "status.saving_file", new Object[] { type.getLabel() }), + pSessionId); + } } try { @@ -1295,26 +1341,30 @@ public class AlignmentPanel extends GAlignmentPanel implements } im = new jalview.util.ImageMaker(this, type, imageAction, - aDimension.getWidth(), aDimension.getHeight(), file, - imageTitle); + aDimension.getWidth(), aDimension.getHeight() + + boarderBottomOffset, file, imageTitle, + alignFrame, pSessionId, headless); + Graphics graphics = im.getGraphics(); if (av.getWrapAlignment()) { - if (im.getGraphics() != null) + if (graphics != null) { - printWrappedAlignment(im.getGraphics(), aDimension.getWidth(), - aDimension.getHeight(), 0); + printWrappedAlignment(aDimension.getWidth(), + aDimension.getHeight() + boarderBottomOffset, 0, + graphics); im.writeImage(); } } else { - if (im.getGraphics() != null) + if (graphics != null) { - printUnwrapped(im.getGraphics(), aDimension.getWidth(), - aDimension.getHeight(), 0); + printUnwrapped(aDimension.getWidth(), aDimension.getHeight(), + 0, graphics, graphics); im.writeImage(); } } + } catch (OutOfMemoryError err) { // Be noisy here. @@ -1328,12 +1378,7 @@ public class AlignmentPanel extends GAlignmentPanel implements } } finally { - if (alignFrame != null && !headless) - { - alignFrame.setProgressBar( - MessageManager.getString("status.export_complete"), - progress); - } + } } @@ -1587,8 +1632,18 @@ public class AlignmentPanel extends GAlignmentPanel implements PaintRefresher.RemoveComponent(getSeqPanel().seqCanvas); PaintRefresher.RemoveComponent(getIdPanel().getIdCanvas()); PaintRefresher.RemoveComponent(this); + + /* + * try to ensure references are nulled + */ + if (annotationPanel != null) + { + annotationPanel.dispose(); + } + if (av != null) { + av.removePropertyChangeListener(propertyChangeListener); jalview.structure.StructureSelectionManager ssm = av .getStructureSelectionManager(); ssm.removeStructureViewerListener(getSeqPanel(), null); @@ -1596,7 +1651,7 @@ public class AlignmentPanel extends GAlignmentPanel implements ssm.removeCommandListener(av); ssm.removeStructureViewerListener(getSeqPanel(), null); ssm.removeSelectionListener(getSeqPanel()); - av.setAlignment(null); + av.dispose(); av = null; } else