X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fgui%2FSequenceRenderer.java;h=43fef152578c8659c36f4bdb8ba741992445e5eb;hb=57738a1f3c19b1c3a00bd3ac5108f8cd0af32f99;hp=81b394bd3c3ecd8ed26d73ddac2074221efd3f4a;hpb=875193c17720b95fa13a38703b1dd63949e12ca9;p=jalview.git diff --git a/src/jalview/gui/SequenceRenderer.java b/src/jalview/gui/SequenceRenderer.java index 81b394b..43fef15 100755 --- a/src/jalview/gui/SequenceRenderer.java +++ b/src/jalview/gui/SequenceRenderer.java @@ -30,6 +30,9 @@ 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'; @@ -97,9 +100,7 @@ public class SequenceRenderer implements jalview.api.SequenceRenderer { allGroups = av.getAlignment().findAllGroups(seq); return resColourFinder.getResidueColour(av.getShowBoxes(), - av.getResidueShading(), - allGroups, seq, position, - finder); + av.getResidueShading(), allGroups, seq, position, finder); } /** @@ -180,15 +181,13 @@ public class SequenceRenderer implements jalview.api.SequenceRenderer if (i < length) { SequenceGroup currentSequenceGroup = resColourFinder - .getCurrentSequenceGroup( - allGroups, i); + .getCurrentSequenceGroup(allGroups, i); if (currentSequenceGroup != null) { if (currentSequenceGroup.getDisplayBoxes()) { resBoxColour = resColourFinder.getBoxColour( - currentSequenceGroup.getGroupColourScheme(), seq, - i); + currentSequenceGroup.getGroupColourScheme(), seq, i); } } else if (av.getShowBoxes()) @@ -254,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()) { @@ -289,8 +300,7 @@ public class SequenceRenderer implements jalview.api.SequenceRenderer } SequenceGroup currentSequenceGroup = resColourFinder - .getCurrentSequenceGroup( - allGroups, i); + .getCurrentSequenceGroup(allGroups, i); if (currentSequenceGroup != null) { if (!currentSequenceGroup.getDisplayText()) @@ -303,8 +313,7 @@ public class SequenceRenderer implements jalview.api.SequenceRenderer { getboxColour = true; resBoxColour = resColourFinder.getBoxColour( - currentSequenceGroup.getGroupColourScheme(), seq, - i); + currentSequenceGroup.getGroupColourScheme(), seq, i); if (currentSequenceGroup.getColourText()) { @@ -326,7 +335,8 @@ public class SequenceRenderer implements jalview.api.SequenceRenderer graphics.setColor(currentSequenceGroup.textColour); } boolean isgrep = currentSequenceGroup != null - ? currentSequenceGroup.getSeqrep() == seq : false; + ? currentSequenceGroup.getSeqrep() == seq + : false; if (!isarep && !isgrep && currentSequenceGroup.getShowNonconserved()) // todo // optimize