X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2FMCview%2FPDBCanvas.java;h=b2f250337f1a5f50dce6f0b9904268bf454c8101;hb=cfa72c2cceb390460b4f08c320146bd6910d8484;hp=7b4e0f0c066e07454233fd18c065009ce2f786dd;hpb=230fac58c5afe1843bf7177834fe1bd1d7e6f9d5;p=jalview.git

diff --git a/src/MCview/PDBCanvas.java b/src/MCview/PDBCanvas.java
index 7b4e0f0..b2f2503 100644
--- a/src/MCview/PDBCanvas.java
+++ b/src/MCview/PDBCanvas.java
@@ -28,6 +28,7 @@ import jalview.gui.FeatureRenderer;
 import jalview.gui.SequenceRenderer;
 import jalview.io.DataSourceType;
 import jalview.io.StructureFile;
+import jalview.renderer.seqfeatures.FeatureColourFinder;
 import jalview.structure.AtomSpec;
 import jalview.structure.StructureListener;
 import jalview.structure.StructureMapping;
@@ -54,8 +55,8 @@ import java.util.Vector;
 import javax.swing.JPanel;
 import javax.swing.ToolTipManager;
 
-public class PDBCanvas extends JPanel implements MouseListener,
-        MouseMotionListener, StructureListener
+public class PDBCanvas extends JPanel
+        implements MouseListener, MouseMotionListener, StructureListener
 {
   boolean redrawneeded = true;
 
@@ -176,7 +177,7 @@ public class PDBCanvas extends JPanel implements MouseListener,
 
     colourBySequence();
 
-    int max = -10;
+    float max = -10;
     int maxchain = -1;
     int pdbstart = 0;
     int pdbend = 0;
@@ -190,7 +191,8 @@ public class PDBCanvas extends JPanel implements MouseListener,
     {
 
       mappingDetails.append("\n\nPDB Sequence is :\nSequence = "
-              + pdb.getChains().elementAt(i).sequence.getSequenceAsString());
+              + pdb.getChains().elementAt(i).sequence
+                      .getSequenceAsString());
       mappingDetails.append("\nNo of residues = "
               + pdb.getChains().elementAt(i).residues.size() + "\n\n");
 
@@ -544,6 +546,7 @@ public class PDBCanvas extends JPanel implements MouseListener,
       showFeatures = true;
     }
 
+    FeatureColourFinder finder = new FeatureColourFinder(fr);
     PDBChain chain;
     if (bysequence && pdb != null)
     {
@@ -571,23 +574,15 @@ public class PDBCanvas extends JPanel implements MouseListener,
                 if (pos > 0)
                 {
                   pos = sequence[s].findIndex(pos);
-                  tmp.startCol = sr.getResidueBoxColour(sequence[s], pos);
-                  if (showFeatures)
-                  {
-                    tmp.startCol = fr.findFeatureColour(tmp.startCol,
-                            sequence[s], pos);
-                  }
+                  tmp.startCol = sr.getResidueColour(sequence[s], pos,
+                          finder);
                 }
                 pos = mapping[m].getSeqPos(tmp.at2.resNumber) - 1;
                 if (pos > 0)
                 {
                   pos = sequence[s].findIndex(pos);
-                  tmp.endCol = sr.getResidueBoxColour(sequence[s], pos);
-                  if (showFeatures)
-                  {
-                    tmp.endCol = fr.findFeatureColour(tmp.endCol,
-                            sequence[s], pos);
-                  }
+                  tmp.endCol = sr.getResidueColour(sequence[s], pos,
+                          finder);
                 }
 
               }
@@ -617,11 +612,15 @@ public class PDBCanvas extends JPanel implements MouseListener,
     {
       tmpBond = visiblebonds.elementAt(i);
 
-      xstart = (int) (((tmpBond.start[0] - centre[0]) * scale) + (getWidth() / 2));
-      ystart = (int) (((centre[1] - tmpBond.start[1]) * scale) + (getHeight() / 2));
+      xstart = (int) (((tmpBond.start[0] - centre[0]) * scale)
+              + (getWidth() / 2));
+      ystart = (int) (((centre[1] - tmpBond.start[1]) * scale)
+              + (getHeight() / 2));
 
-      xend = (int) (((tmpBond.end[0] - centre[0]) * scale) + (getWidth() / 2));
-      yend = (int) (((centre[1] - tmpBond.end[1]) * scale) + (getHeight() / 2));
+      xend = (int) (((tmpBond.end[0] - centre[0]) * scale)
+              + (getWidth() / 2));
+      yend = (int) (((centre[1] - tmpBond.end[1]) * scale)
+              + (getHeight() / 2));
 
       xmid = (xend + xstart) / 2;
       ymid = (yend + ystart) / 2;
@@ -685,8 +684,8 @@ public class PDBCanvas extends JPanel implements MouseListener,
 
       if (highlightBond1 != null && highlightBond1 == tmpBond)
       {
-        g.setColor(tmpBond.endCol.brighter().brighter().brighter()
-                .brighter());
+        g.setColor(
+                tmpBond.endCol.brighter().brighter().brighter().brighter());
         drawLine(g, xmid, ymid, xend, yend);
       }
 
@@ -822,8 +821,8 @@ public class PDBCanvas extends JPanel implements MouseListener,
 
     if (fatom != null)
     {
-      this.setToolTipText(chain.id + ":" + fatom.resNumber + " "
-              + fatom.resName);
+      this.setToolTipText(
+              chain.id + ":" + fatom.resNumber + " " + fatom.resName);
     }
     else
     {
@@ -933,16 +932,20 @@ public class PDBCanvas extends JPanel implements MouseListener,
     g.setColor(Color.red);
     if (n == 1)
     {
-      int xstart = (int) (((b.start[0] - centre[0]) * scale) + (getWidth() / 2));
-      int ystart = (int) (((centre[1] - b.start[1]) * scale) + (getHeight() / 2));
+      int xstart = (int) (((b.start[0] - centre[0]) * scale)
+              + (getWidth() / 2));
+      int ystart = (int) (((centre[1] - b.start[1]) * scale)
+              + (getHeight() / 2));
 
       g.drawString(b.at1.resName + "-" + b.at1.resNumber, xstart, ystart);
     }
 
     if (n == 2)
     {
-      int xstart = (int) (((b.end[0] - centre[0]) * scale) + (getWidth() / 2));
-      int ystart = (int) (((centre[1] - b.end[1]) * scale) + (getHeight() / 2));
+      int xstart = (int) (((b.end[0] - centre[0]) * scale)
+              + (getWidth() / 2));
+      int ystart = (int) (((centre[1] - b.end[1]) * scale)
+              + (getHeight() / 2));
 
       g.drawString(b.at2.resName + "-" + b.at2.resNumber, xstart, ystart);
     }
@@ -968,11 +971,13 @@ public class PDBCanvas extends JPanel implements MouseListener,
         {
           tmpBond = bond;
 
-          truex = (int) (((tmpBond.start[0] - centre[0]) * scale) + (getWidth() / 2));
+          truex = (int) (((tmpBond.start[0] - centre[0]) * scale)
+                  + (getWidth() / 2));
 
           if (Math.abs(truex - x) <= 2)
           {
-            int truey = (int) (((centre[1] - tmpBond.start[1]) * scale) + (getHeight() / 2));
+            int truey = (int) (((centre[1] - tmpBond.start[1]) * scale)
+                    + (getHeight() / 2));
 
             if (Math.abs(truey - y) <= 2)
             {
@@ -985,11 +990,13 @@ public class PDBCanvas extends JPanel implements MouseListener,
 
         // Still here? Maybe its the last bond
 
-        truex = (int) (((tmpBond.end[0] - centre[0]) * scale) + (getWidth() / 2));
+        truex = (int) (((tmpBond.end[0] - centre[0]) * scale)
+                + (getWidth() / 2));
 
         if (Math.abs(truex - x) <= 2)
         {
-          int truey = (int) (((tmpBond.end[1] - centre[1]) * scale) + (getHeight() / 2));
+          int truey = (int) (((tmpBond.end[1] - centre[1]) * scale)
+                  + (getHeight() / 2));
 
           if (Math.abs(truey - y) <= 2)
           {
@@ -1079,7 +1086,7 @@ public class PDBCanvas extends JPanel implements MouseListener,
   // ////////////////////////////////
   // /StructureListener
   @Override
-  public String[] getPdbFile()
+  public String[] getStructureFiles()
   {
     return new String[] { pdbentry.getFile() };
   }