X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fgui%2FSequenceRenderer.java;h=43fef152578c8659c36f4bdb8ba741992445e5eb;hb=e6dcbe140f44b9484be5641d10c1c09bf92a5b61;hp=4498f881dd3689751d9132be8dcfaf36a5e6791c;hpb=3d0101179759ef157b088ea135423cd909512d9f;p=jalview.git diff --git a/src/jalview/gui/SequenceRenderer.java b/src/jalview/gui/SequenceRenderer.java index 4498f88..43fef15 100755 --- a/src/jalview/gui/SequenceRenderer.java +++ b/src/jalview/gui/SequenceRenderer.java @@ -23,14 +23,16 @@ package jalview.gui; import jalview.api.AlignViewportI; import jalview.datamodel.SequenceGroup; import jalview.datamodel.SequenceI; -import jalview.renderer.ResidueShaderI; +import jalview.renderer.ResidueColourFinder; import jalview.renderer.seqfeatures.FeatureColourFinder; -import jalview.util.Comparison; import java.awt.Color; import java.awt.FontMetrics; import java.awt.Graphics; +import org.jfree.graphics2d.svg.SVGGraphics2D; +import org.jibble.epsgraphics.EpsGraphics2D; + public class SequenceRenderer implements jalview.api.SequenceRenderer { final static int CHAR_TO_UPPER = 'A' - 'a'; @@ -41,17 +43,15 @@ public class SequenceRenderer implements jalview.api.SequenceRenderer boolean renderGaps = true; - SequenceGroup currentSequenceGroup = null; - SequenceGroup[] allGroups = null; - Color resBoxColour; + // Color resBoxColour; Graphics graphics; boolean monospacedFont; - boolean forOverview = false; + ResidueColourFinder resColourFinder; /** * Creates a new SequenceRenderer object @@ -61,6 +61,7 @@ public class SequenceRenderer implements jalview.api.SequenceRenderer public SequenceRenderer(AlignViewportI viewport) { this.av = viewport; + resColourFinder = new ResidueColourFinder(); } /** @@ -83,26 +84,6 @@ public class SequenceRenderer implements jalview.api.SequenceRenderer this.renderGaps = renderGaps; } - protected Color getResidueBoxColour(SequenceI seq, int i) - { - // rate limiting step when rendering overview for lots of groups - allGroups = av.getAlignment().findAllGroups(seq); - - if (inCurrentSequenceGroup(i)) - { - if (currentSequenceGroup.getDisplayBoxes()) - { - getBoxColour(currentSequenceGroup.getGroupColourScheme(), seq, i); - } - } - else if (av.getShowBoxes()) - { - getBoxColour(av.getResidueShading(), seq, i); - } - - return resBoxColour; - } - /** * Get the residue colour at the given sequence position - as determined by * the sequence group colour (if any), else the colour scheme, possibly @@ -117,39 +98,9 @@ public class SequenceRenderer implements jalview.api.SequenceRenderer public Color getResidueColour(final SequenceI seq, int position, FeatureColourFinder finder) { - Color col = getResidueBoxColour(seq, position); - - if (finder != null) - { - col = finder.findFeatureColour(col, seq, position); - } - return col; - } - - /** - * DOCUMENT ME! - * - * @param shader - * DOCUMENT ME! - * @param seq - * DOCUMENT ME! - * @param i - * DOCUMENT ME! - */ - void getBoxColour(ResidueShaderI shader, SequenceI seq, int i) - { - if (shader.getColourScheme() != null) - { - resBoxColour = shader.findColour(seq.getCharAt(i), i, seq); - } - else if (forOverview && !Comparison.isGap(seq.getCharAt(i))) - { - resBoxColour = Color.lightGray; - } - else - { - resBoxColour = Color.white; - } + allGroups = av.getAlignment().findAllGroups(seq); + return resColourFinder.getResidueColour(av.getShowBoxes(), + av.getResidueShading(), allGroups, seq, position, finder); } /** @@ -208,6 +159,8 @@ public class SequenceRenderer implements jalview.api.SequenceRenderer public synchronized void drawBoxes(SequenceI seq, int start, int end, int y1) { + Color resBoxColour = Color.white; + if (seq == null) { return; // fix for racecondition @@ -227,17 +180,20 @@ public class SequenceRenderer implements jalview.api.SequenceRenderer if (i < length) { - if (inCurrentSequenceGroup(i)) + SequenceGroup currentSequenceGroup = resColourFinder + .getCurrentSequenceGroup(allGroups, i); + if (currentSequenceGroup != null) { if (currentSequenceGroup.getDisplayBoxes()) { - getBoxColour(currentSequenceGroup.getGroupColourScheme(), seq, - i); + resBoxColour = resColourFinder.getBoxColour( + currentSequenceGroup.getGroupColourScheme(), seq, i); } } else if (av.getShowBoxes()) { - getBoxColour(av.getResidueShading(), seq, i); + resBoxColour = resColourFinder + .getBoxColour(av.getResidueShading(), seq, i); } } @@ -297,8 +253,20 @@ public class SequenceRenderer implements jalview.api.SequenceRenderer } graphics.setColor(av.getTextColour()); - if (monospacedFont && av.getShowText() && allGroups.length == 0 - && !av.getColourText() && av.getThresholdTextColour() == 0) + boolean drawAllText = monospacedFont && av.getShowText() + && allGroups.length == 0 && !av.getColourText() + && av.getThresholdTextColour() == 0; + + /* + * EPS or SVG misaligns monospaced strings (JAL-3239) + * so always draw these one character at a time + */ + if (graphics instanceof EpsGraphics2D + || graphics instanceof SVGGraphics2D) + { + drawAllText = false; + } + if (drawAllText) { if (av.isRenderGaps()) { @@ -317,10 +285,8 @@ public class SequenceRenderer implements jalview.api.SequenceRenderer boolean srep = av.isDisplayReferenceSeq(); boolean getboxColour = false; boolean isarep = av.getAlignment().getSeqrep() == seq; - boolean isgrep = currentSequenceGroup != null - ? currentSequenceGroup.getSeqrep() == seq - : false; - char sr_c; + Color resBoxColour = Color.white; + for (int i = start; i <= end; i++) { @@ -333,7 +299,9 @@ public class SequenceRenderer implements jalview.api.SequenceRenderer continue; } - if (inCurrentSequenceGroup(i)) + SequenceGroup currentSequenceGroup = resColourFinder + .getCurrentSequenceGroup(allGroups, i); + if (currentSequenceGroup != null) { if (!currentSequenceGroup.getDisplayText()) { @@ -344,8 +312,8 @@ public class SequenceRenderer implements jalview.api.SequenceRenderer || currentSequenceGroup.getColourText()) { getboxColour = true; - getBoxColour(currentSequenceGroup.getGroupColourScheme(), seq, - i); + resBoxColour = resColourFinder.getBoxColour( + currentSequenceGroup.getGroupColourScheme(), seq, i); if (currentSequenceGroup.getColourText()) { @@ -366,6 +334,9 @@ public class SequenceRenderer implements jalview.api.SequenceRenderer { graphics.setColor(currentSequenceGroup.textColour); } + boolean isgrep = currentSequenceGroup != null + ? currentSequenceGroup.getSeqrep() == seq + : false; if (!isarep && !isgrep && currentSequenceGroup.getShowNonconserved()) // todo // optimize @@ -386,7 +357,8 @@ public class SequenceRenderer implements jalview.api.SequenceRenderer if (av.getColourText()) { getboxColour = true; - getBoxColour(av.getResidueShading(), seq, i); + resBoxColour = resColourFinder + .getBoxColour(av.getResidueShading(), seq, i); if (av.getShowBoxes()) { @@ -402,7 +374,8 @@ public class SequenceRenderer implements jalview.api.SequenceRenderer { if (!getboxColour) { - getBoxColour(av.getResidueShading(), seq, i); + resBoxColour = resColourFinder + .getBoxColour(av.getResidueShading(), seq, i); } if (resBoxColour.getRed() + resBoxColour.getBlue() @@ -473,35 +446,6 @@ public class SequenceRenderer implements jalview.api.SequenceRenderer /** * DOCUMENT ME! * - * @param res - * DOCUMENT ME! - * - * @return DOCUMENT ME! - */ - boolean inCurrentSequenceGroup(int res) - { - if (allGroups == null) - { - return false; - } - - for (int i = 0; i < allGroups.length; i++) - { - if ((allGroups[i].getStartRes() <= res) - && (allGroups[i].getEndRes() >= res)) - { - currentSequenceGroup = allGroups[i]; - - return true; - } - } - - return false; - } - - /** - * DOCUMENT ME! - * * @param seq * DOCUMENT ME! * @param start @@ -547,21 +491,30 @@ public class SequenceRenderer implements jalview.api.SequenceRenderer } } - public void drawCursor(SequenceI seq, int res, int x1, int y1) + /** + * Draw a sequence canvas cursor + * + * @param g + * graphics context to draw on + * @param s + * character to draw at cursor + * @param x1 + * x position of cursor in graphics context + * @param y1 + * y position of cursor in graphics context + */ + public void drawCursor(Graphics g, char s, int x1, int y1) { int pady = av.getCharHeight() / 5; int charOffset = 0; - graphics.setColor(Color.black); - graphics.fillRect(x1, y1, av.getCharWidth(), av.getCharHeight()); + g.setColor(Color.black); + g.fillRect(x1, y1, av.getCharWidth(), av.getCharHeight()); if (av.isValidCharWidth()) { - graphics.setColor(Color.white); - - char s = seq.getCharAt(res); - + g.setColor(Color.white); charOffset = (av.getCharWidth() - fm.charWidth(s)) / 2; - graphics.drawString(String.valueOf(s), charOffset + x1, + g.drawString(String.valueOf(s), charOffset + x1, (y1 + av.getCharHeight()) - pady); }