From: Jim Procter <jprocter@issues.jalview.org>
Date: Mon, 4 Apr 2016 14:49:37 +0000 (+0100)
Subject: JAL-2001 TODO for new ColumnSelection API
X-Git-Tag: Release_2_10_0~270^2^2~2
X-Git-Url: http://source.jalview.org/gitweb/?a=commitdiff_plain;h=ea710b87c966bf443b6526236ae38a38edf06b3b;p=jalview.git

JAL-2001 TODO for new ColumnSelection API
---

diff --git a/src/jalview/appletgui/AnnotationPanel.java b/src/jalview/appletgui/AnnotationPanel.java
index d642c14..77700d0 100755
--- a/src/jalview/appletgui/AnnotationPanel.java
+++ b/src/jalview/appletgui/AnnotationPanel.java
@@ -160,6 +160,9 @@ public class AnnotationPanel extends Panel implements AwtRenderPanelI,
     {
       for (int sel : av.getColumnSelection().getSelected())
       {
+        // TODO: JAL-2001 check if applet has faulty 'REMOVE' selected columns
+        // of
+        // annotation if selection includes hidden columns
         anot[sel] = null;
       }
     }
@@ -179,6 +182,8 @@ public class AnnotationPanel extends Panel implements AwtRenderPanelI,
 
       for (int index : av.getColumnSelection().getSelected())
       {
+        // TODO: JAL-2001 - provide a fast method to list visible selected
+        // columns
         if (!av.getColumnSelection().isVisible(index))
         {
           continue;
diff --git a/src/jalview/appletgui/ScalePanel.java b/src/jalview/appletgui/ScalePanel.java
index 3c6a4f1..9106385 100755
--- a/src/jalview/appletgui/ScalePanel.java
+++ b/src/jalview/appletgui/ScalePanel.java
@@ -410,6 +410,8 @@ public class ScalePanel extends Panel implements MouseMotionListener,
     int avcharWidth = av.getCharWidth(), avcharHeight = av.getCharHeight();
     for (int sel : cs.getSelected())
     {
+      // TODO: JAL-2001 - provide a fast method to list visible selected in a
+      // given range
       if (av.hasHiddenColumns())
       {
         sel = av.getColumnSelection().findColumnPosition(sel);
diff --git a/src/jalview/gui/ScalePanel.java b/src/jalview/gui/ScalePanel.java
index b2c9a12..7e17f46 100755
--- a/src/jalview/gui/ScalePanel.java
+++ b/src/jalview/gui/ScalePanel.java
@@ -467,6 +467,9 @@ public class ScalePanel extends JPanel implements MouseMotionListener,
 
       for (int sel : cs.getSelected())
       {
+        // TODO: JAL-2001 - provide a fast method to list visible selected in a
+        // given range
+
         if (av.hasHiddenColumns())
         {
           if (cs.isVisible(sel))