Merge branch 'feature/JAL-3143ensemblJSON' into
authorgmungoc <g.m.carstairs@dundee.ac.uk>
Thu, 1 Nov 2018 14:31:16 +0000 (14:31 +0000)
committergmungoc <g.m.carstairs@dundee.ac.uk>
Thu, 1 Nov 2018 14:31:16 +0000 (14:31 +0000)
Jalview-BH/JAL-3026-JAL-3063-JAXB

Conflicts:
src/jalview/gui/FeatureTypeSettings.java

1  2 
src/jalview/gui/FeatureTypeSettings.java
src/jalview/schemes/FeatureColour.java
test/jalview/schemes/FeatureColourTest.java

@@@ -62,8 -62,8 +62,7 @@@ import javax.swing.JLabel
  import javax.swing.JPanel;
  import javax.swing.JRadioButton;
  import javax.swing.JSlider;
- import javax.swing.JTabbedPane;
  import javax.swing.JTextField;
 -import javax.swing.SwingConstants;
  import javax.swing.border.LineBorder;
  import javax.swing.event.ChangeEvent;
  import javax.swing.event.ChangeListener;
@@@ -148,16 -149,16 +147,15 @@@ public class FeatureTypeSettings extend
     * radio button group, to select what to colour by:
     * simple colour, by category (text), or graduated
     */
 -  private JRadioButton simpleColour = new JRadioButton();
 +  JRadioButton simpleColour = new JRadioButton();
  
 -  private JRadioButton byCategory = new JRadioButton();
 +  JRadioButton byCategory = new JRadioButton();
  
 -  private JRadioButton graduatedColour = new JRadioButton();
 +  JRadioButton graduatedColour = new JRadioButton();
  
--  /**
--   * colours and filters are shown in tabbed view or single content pane
--   */
--  JPanel coloursPanel, filtersPanel;
++  JPanel coloursPanel;
++  
++  JPanel filtersPanel;
  
    JPanel singleColour = new JPanel();
  
        ex.printStackTrace();
        return;
      }
-     updateColoursTab();
-     updateFiltersTab();
+     
 -    updateColoursTab();
++    updateColoursPanel();
+     
 -    updateFiltersTab();
++    updateFiltersPanel();
+     
      adjusting = false;
+     
      colourChanged(false);
+     
      String title = MessageManager
              .formatMessage("label.display_settings_for", new String[]
              { theType });
-     initDialogFrame(this, true, blocking, title, 500, 500);
+     initDialogFrame(this, true, false, title, 500, 500);
 -    
      waitForInput();
    }
  
    /**
--   * Configures the widgets on the Colours tab according to the current feature
++   * Configures the widgets on the Colours panel according to the current feature
     * colour scheme
     */
--  private void updateColoursTab()
++  private void updateColoursPanel()
    {
      FeatureColourI fc = fr.getFeatureColours().get(featureType);
  
      };
  
      /*
--     * first panel/tab: colour options
++     * first panel: colour options
       */
      JPanel coloursPanel = initialiseColoursPanel();
      this.add(coloursPanel, BorderLayout.NORTH);
  
      /*
--     * second panel/tab: filter options
++     * second panel: filter options
       */
      JPanel filtersPanel = initialiseFiltersPanel();
      this.add(filtersPanel, BorderLayout.CENTER);
      fr.setColour(featureType, acg);
      ap.paintAlignment(updateStructsAndOverview, updateStructsAndOverview);
  
--    updateColoursTab();
++    updateColoursPanel();
    }
  
    /**
     * for adding a condition. This should be called after a filter has been
     * removed, added or amended.
     */
--  private void updateFiltersTab()
++  private void updateFiltersPanel()
    {
      /*
       * clear the panel and list of filter conditions
      fr.setFeatureFilter(featureType, combined.isEmpty() ? null : combined);
      ap.paintAlignment(true, true);
  
--    updateFiltersTab();
++    updateFiltersPanel();
    }
  }