From: Jim Procter <jprocter@issues.jalview.org>
Date: Thu, 3 Sep 2015 21:00:18 +0000 (+0100)
Subject: JAL-961 efficiency TODO: decouple overview updates from sequence colour broadcasts
X-Git-Tag: Release_2_10_0~497
X-Git-Url: http://source.jalview.org/gitweb/?a=commitdiff_plain;h=f6f774a4fa6885275b9d1902a4c8d470607e3447;p=jalview.git

JAL-961 efficiency TODO: decouple overview updates from sequence colour broadcasts
---

diff --git a/src/jalview/appletgui/AlignmentPanel.java b/src/jalview/appletgui/AlignmentPanel.java
index 463c6c3..fd6652b 100644
--- a/src/jalview/appletgui/AlignmentPanel.java
+++ b/src/jalview/appletgui/AlignmentPanel.java
@@ -953,6 +953,7 @@ public class AlignmentPanel extends Panel implements AdjustmentListener,
 
     if (updateOverview)
     {
+      // TODO: determine if this paintAlignment changed structure colours
       jalview.structure.StructureSelectionManager
               .getStructureSelectionManager(av.applet)
               .sequenceColoursChanged(this);
diff --git a/src/jalview/gui/AlignmentPanel.java b/src/jalview/gui/AlignmentPanel.java
index e29a0e2..1b93f10 100644
--- a/src/jalview/gui/AlignmentPanel.java
+++ b/src/jalview/gui/AlignmentPanel.java
@@ -851,6 +851,7 @@ public class AlignmentPanel extends GAlignmentPanel implements
 
     if (updateOverview)
     {
+      // TODO: determine if this paintAlignment changed structure colours
       av.getStructureSelectionManager().sequenceColoursChanged(this);
 
       if (overviewPanel != null)