From: Jim Procter <jprocter@issues.jalview.org>
Date: Mon, 24 Jun 2019 17:36:04 +0000 (+0100)
Subject: JAL-3330 merge styles onto originating views when show cross-refs performed (should... 
X-Git-Tag: Release_2_11_0~4^2~1^2~8
X-Git-Url: http://source.jalview.org/gitweb/?a=commitdiff_plain;h=2014b8df8e4cb0af7d5240b1839c70f1fdcda3fd;p=jalview.git

JAL-3330 merge styles onto originating views when show cross-refs performed (should really do this for all views involving dataset).
---

diff --git a/src/jalview/gui/CrossRefAction.java b/src/jalview/gui/CrossRefAction.java
index 51ac2ee..673bcc0 100644
--- a/src/jalview/gui/CrossRefAction.java
+++ b/src/jalview/gui/CrossRefAction.java
@@ -187,15 +187,24 @@ public class CrossRefAction implements Runnable
 
       /*
        * apply 'database source' feature configuration
-       * if any was found
+       * if any - first to the new splitframe view about to be displayed
        */
-      // TODO is this the feature colouring for the original
-      // alignment or the fetched xrefs? either could be Ensembl
+
       newFrame.getViewport().applyFeaturesStyle(featureColourScheme);
       copyThis.getViewport().applyFeaturesStyle(featureColourScheme);
 
+      /*
+       * and for JAL-3330 also to original alignFrame view(s)
+       * this currently trashes any original settings.
+       */
+      for (AlignmentViewPanel origpanel: alignFrame.getAlignPanels()) {
+        origpanel.getAlignViewport()
+                .mergeFeaturesStyle(featureColourScheme);
+      }
+
       SplitFrame sf = new SplitFrame(dna ? copyThis : newFrame,
               dna ? newFrame : copyThis);
+
       newFrame.setVisible(true);
       copyThis.setVisible(true);
       String linkedTitle = MessageManager