JAL-3148 add all options to colour menu

[jalview.git] / src / jalview / gui / ColourMenuHelper.java

diff --git a/src/jalview/gui/ColourMenuHelper.java b/src/jalview/gui/ColourMenuHelper.java
index 7c3107e..fd5f080 100644 (file)
--- a/src/jalview/gui/ColourMenuHelper.java
+++ b/src/jalview/gui/ColourMenuHelper.java
@@ -1,3 +1,23 @@
+/*
+ * Jalview - A Sequence Alignment Editor and Viewer ($$Version-Rel$$)
+ * Copyright (C) $$Year-Rel$$ The Jalview Authors
+ * 
+ * This file is part of Jalview.
+ * 
+ * Jalview is free software: you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License 
+ * as published by the Free Software Foundation, either version 3
+ * of the License, or (at your option) any later version.
+ *  
+ * Jalview is distributed in the hope that it will be useful, but 
+ * WITHOUT ANY WARRANTY; without even the implied warranty 
+ * of MERCHANTABILITY or FITNESS FOR A PARTICULAR 
+ * PURPOSE.  See the GNU General Public License for more details.
+ * 
+ * You should have received a copy of the GNU General Public License
+ * along with Jalview.  If not, see <http://www.gnu.org/licenses/>.
+ * The Jalview Authors are detailed in the 'AUTHORS' file.
+ */
 package jalview.gui;
 
 import jalview.bin.Cache;
@@ -52,12 +72,9 @@ public class ColourMenuHelper
    *          a callback to handle menu selection
    * @param coll
    *          the data the menu is being built for
-   * @param simpleOnly
-   *          if true, only simple per-residue colour schemes are included
    */
   public static ButtonGroup addMenuItems(final JMenu colourMenu,
-          final ColourChangeListener client, AnnotatedCollectionI coll,
-          boolean simpleOnly)
+                       final ColourChangeListener client, AnnotatedCollectionI coll)
   {
     /*
      * ButtonGroup groups those items whose 
@@ -65,22 +82,18 @@ public class ColourMenuHelper
      */
     ButtonGroup colours = new ButtonGroup();
 
-    if (!simpleOnly)
-    {
-      JRadioButtonMenuItem noColourmenuItem = new JRadioButtonMenuItem(
-              MessageManager.getString("label.none"));
-      noColourmenuItem.setName(ResidueColourScheme.NONE);
-      noColourmenuItem.addActionListener(new ActionListener()
-      {
-        @Override
-        public void actionPerformed(ActionEvent e)
-        {
-          client.changeColour_actionPerformed(ResidueColourScheme.NONE);
-        }
-      });
-      colourMenu.add(noColourmenuItem);
-      colours.add(noColourmenuItem);
-    }
+       JRadioButtonMenuItem noColourmenuItem = new JRadioButtonMenuItem(MessageManager.getString("label.none"));
+       noColourmenuItem.setName(ResidueColourScheme.NONE);
+       noColourmenuItem.addActionListener(new ActionListener() 
+       {
+         @Override
+         public void actionPerformed(ActionEvent e) 
+         {
+           client.changeColour_actionPerformed(ResidueColourScheme.NONE);
+         }
+       });
+       colourMenu.add(noColourmenuItem);
+       colours.add(noColourmenuItem);
 
     /*
      * scan registered colour schemes (built-in or user-defined)
@@ -90,11 +103,6 @@ public class ColourMenuHelper
             .getColourSchemes();
     for (ColourSchemeI scheme : colourSchemes)
     {
-      if (simpleOnly && !scheme.isSimple())
-      {
-        continue;
-      }
-
       /*
        * button text is i18n'd but the name is the canonical name of
        * the colour scheme (inspected in setColourSelected())