From 4eead9c2f173d090d67d4ee180c5706242face13 Mon Sep 17 00:00:00 2001 From: Jim Procter Date: Tue, 10 Feb 2015 15:57:41 +0000 Subject: [PATCH] added comment to @gmungoc todo about referencing PDB entities in Javascript - also verified JAL-592 is now a bug again and can be reproduced :( --- src/jalview/javascript/MouseOverStructureListener.java | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/jalview/javascript/MouseOverStructureListener.java b/src/jalview/javascript/MouseOverStructureListener.java index 3cc7c98..cd70291 100644 --- a/src/jalview/javascript/MouseOverStructureListener.java +++ b/src/jalview/javascript/MouseOverStructureListener.java @@ -152,6 +152,8 @@ public class MouseOverStructureListener extends JSFunctionExec implements { // TODO is this right? StructureSelectionManager passes pdbFile as the // field that is interpreted (in 2.8.2) as pdbId? + // JBPComment: yep - this is right! the Javascript harness uses the + // absolute pdbFile URI to locate the PDB file in the external viewer executeJavascriptFunction(_listenerfn, new String[] { "mouseover", "" + atom.getPdbFile(), "" + atom.getChain(), -- 1.7.10.2