Merge branch 'feature/JAL-3080clearTreeGroups' into Jalview-BH/JAL-3026
authorgmungoc <g.m.carstairs@dundee.ac.uk>
Wed, 15 Aug 2018 10:02:47 +0000 (11:02 +0100)
committergmungoc <g.m.carstairs@dundee.ac.uk>
Wed, 15 Aug 2018 10:02:47 +0000 (11:02 +0100)
1  2 
src/jalview/gui/TreeCanvas.java

@@@ -27,7 -27,6 +27,7 @@@ import jalview.datamodel.Sequence
  import jalview.datamodel.SequenceGroup;
  import jalview.datamodel.SequenceI;
  import jalview.datamodel.SequenceNode;
 +import jalview.gui.JalviewColourChooser.ColourChooserListener;
  import jalview.schemes.ColourSchemeI;
  import jalview.schemes.ColourSchemeProperty;
  import jalview.schemes.UserColourScheme;
@@@ -842,17 -841,15 +842,17 @@@ public class TreeCanvas extends JPanel 
     */
    void chooseSubtreeColour()
    {
 -    Color col = JColorChooser.showDialog(this,
 -            MessageManager.getString("label.select_subtree_colour"),
 -            highlightNode.color);
 -    if (col != null)
 -    {
 -      setColor(highlightNode, col);
 -      PaintRefresher.Refresh(tp, ap.av.getSequenceSetId());
 -      repaint();
 -    }
 +    String ttl = MessageManager.getString("label.select_subtree_colour");
 +    ColourChooserListener listener = new ColourChooserListener() {
 +      @Override
 +      public void colourSelected(Color c)
 +      {
 +        setColor(highlightNode, c);
 +        PaintRefresher.Refresh(tp, ap.av.getSequenceSetId());
 +        repaint();
 +      }
 +    };
 +    JalviewColourChooser.showColourChooser(this, ttl,  highlightNode.color, listener);
    }
  
    @Override
            }
          }
          colourGroups(groups);
+         /*
+          * clear partition (don't show vertical line) if
+          * it is to the right of all nodes
+          */
+         if (groups.isEmpty())
+         {
+           threshold = 0f;
+         }
        }
  
        PaintRefresher.Refresh(tp, ap.av.getSequenceSetId());
  
        Vector<SequenceNode> l = tree.findLeaves(groups.get(i));
  
-       Vector<SequenceI> sequences = new Vector<SequenceI>();
+       Vector<SequenceI> sequences = new Vector<>();
  
        for (int j = 0; j < l.size(); j++)
        {