X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fgui%2FSequenceRenderer.java;h=fedcc37b55b0452b4f4a905053138521b32ddb84;hb=fc7dac25fd7a4eac42083e9aa94d1e752cabbe8b;hp=81b394bd3c3ecd8ed26d73ddac2074221efd3f4a;hpb=9e926ac4305fd9dff38b6e079e55b4f50664d544;p=jalview.git

diff --git a/src/jalview/gui/SequenceRenderer.java b/src/jalview/gui/SequenceRenderer.java
index 81b394b..fedcc37 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';
@@ -67,6 +70,7 @@ public class SequenceRenderer implements jalview.api.SequenceRenderer
    * @param b
    *          DOCUMENT ME!
    */
+  @Override
   public void prepare(Graphics g, boolean renderGaps)
   {
     graphics = g;
@@ -97,9 +101,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 +182,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 +254,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 +301,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 +314,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 +336,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