From 2014b8df8e4cb0af7d5240b1839c70f1fdcda3fd Mon Sep 17 00:00:00 2001 From: Jim Procter Date: Mon, 24 Jun 2019 18:36:04 +0100 Subject: [PATCH] JAL-3330 merge styles onto originating views when show cross-refs performed (should really do this for all views involving dataset). --- src/jalview/gui/CrossRefAction.java | 15 ++++++++++++--- 1 file changed, 12 insertions(+), 3 deletions(-) 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 -- 1.7.10.2