removed useless flag (was always null for button press) to ensure default view colour...

diff --git a/src/jalview/gui/AppJmol.java b/src/jalview/gui/AppJmol.java
index 9d68c1b..d5148ba 100644 (file)
--- a/src/jalview/gui/AppJmol.java
+++ b/src/jalview/gui/AppJmol.java
@@ -210,30 +210,7 @@ public class AppJmol extends GStructureViewer implements Runnable,
 
       }
     });
-    colourMenu.addMenuListener(new MenuListener()
-    {
-      
-      @Override
-      public void menuSelected(MenuEvent arg0)
-      {
-        
-        handlingGuiEvents=true;
-      }
-      
-      @Override
-      public void menuDeselected(MenuEvent arg0)
-      {
-        handlingGuiEvents=false;
-      }
-      
-      @Override
-      public void menuCanceled(MenuEvent arg0)
-      {
-        handlingGuiEvents=false;
-      }
-    });
   }
-
   IProgressIndicator progressBar = null;
 
   public AppJmol(PDBEntry pdbentry, SequenceI[] seq, String[] chains,
@@ -658,13 +635,6 @@ public class AppJmol extends GStructureViewer implements Runnable,
    */
   private boolean _started = false;
 
-  /**
-   * state flag indicating if the GUI is active when a particular button event
-   * takes place (e.g. when a stateChange occurs on a checkbox - this could be
-   * programmatic or due to the user doing something)
-   */
-  private boolean handlingGuiEvents=false;
-
   public void run()
   {
     _started = true;
@@ -958,7 +928,7 @@ public class AppJmol extends GStructureViewer implements Runnable,
     }
     if (jmb.isColourBySequence())
     {
-      if (!jmb.isLoadingFromArchive() && handlingGuiEvents)
+      if (!jmb.isLoadingFromArchive())
       {
         if (ap!=null) {
                // Make the currently displayed alignment panel the associated view