From: gmungoc <g.m.carstairs@dundee.ac.uk>
Date: Wed, 25 Feb 2015 10:42:13 +0000 (+0000)
Subject: JAL-1677 commenting use of AlignViewport.gatherViewsHere
X-Git-Url: http://source.jalview.org/gitweb/?a=commitdiff_plain;h=c94c1390298db2e80abc04b371d3e7767f5e5968;p=jalview.git

JAL-1677 commenting use of AlignViewport.gatherViewsHere
---

diff --git a/src/jalview/gui/AlignViewport.java b/src/jalview/gui/AlignViewport.java
index 2129db1..4ba2113 100644
--- a/src/jalview/gui/AlignViewport.java
+++ b/src/jalview/gui/AlignViewport.java
@@ -107,6 +107,13 @@ public class AlignViewport extends AlignmentViewport implements
 
   String viewName;
 
+  /*
+   * Flag set true on the view that should 'gather' multiple views of the same
+   * sequence set id when a project is reloaded. Set false on all views when
+   * they are 'exploded' into separate windows. Set true on the current view
+   * when 'Gather' is performed, and also on the first tab when the first new
+   * view is created.
+   */
   private boolean gatherViewsHere = false;
 
   private AnnotationColumnChooser annotationColumnSelectionState;
diff --git a/src/jalview/gui/Jalview2XML.java b/src/jalview/gui/Jalview2XML.java
index 36b48ab..b30b28f 100644
--- a/src/jalview/gui/Jalview2XML.java
+++ b/src/jalview/gui/Jalview2XML.java
@@ -2155,6 +2155,13 @@ public class Jalview2XML
       Desktop.instance.stopLoading();
     }
 
+    /*
+     * Regather multiple views (with the same sequence set id) to the frame (if
+     * any) that is flagged as the one to gather to, i.e. convert them to tabbed
+     * views instead of separate frames. Note this doesn't restore a state where
+     * some expanded views in turn have tabbed views - the last "first tab" read
+     * in will play the role of gatherer for all.
+     */
     for (AlignFrame fr : gatherToThisFrame.values())
     {
       Desktop.instance.gatherViews(fr);