From 313fff566aa8e673b50c4c4e4b216d2704c5ceae Mon Sep 17 00:00:00 2001
From: gmungoc <g.m.carstairs@dundee.ac.uk>
Date: Thu, 22 Dec 2016 14:03:20 +0000
Subject: [PATCH] JAL-2360 fix bug that prematurely created a SequenceGroup

---
 src/jalview/gui/PopupMenu.java |   19 +++++++++----------
 1 file changed, 9 insertions(+), 10 deletions(-)

diff --git a/src/jalview/gui/PopupMenu.java b/src/jalview/gui/PopupMenu.java
index c660ab5..ffa8b9c 100644
--- a/src/jalview/gui/PopupMenu.java
+++ b/src/jalview/gui/PopupMenu.java
@@ -987,15 +987,14 @@ public class PopupMenu extends JPopupMenu implements ColourChangeListener
     });
     createGroupMenuItem.setText(MessageManager
             .getString("action.create_group"));
-    createGroupMenuItem
-.addActionListener(new ActionListener()
-            {
-              @Override
-              public void actionPerformed(ActionEvent e)
-              {
-                createGroupMenuItem_actionPerformed();
-              }
-            });
+    createGroupMenuItem.addActionListener(new ActionListener()
+    {
+      @Override
+      public void actionPerformed(ActionEvent e)
+      {
+        createGroupMenuItem_actionPerformed();
+      }
+    });
 
     outline.setText(MessageManager.getString("action.border_colour"));
     outline.addActionListener(new ActionListener()
@@ -1270,7 +1269,7 @@ public class PopupMenu extends JPopupMenu implements ColourChangeListener
    */
   protected void buildColourMenu()
   {
-    SequenceGroup sg = getGroup();
+    SequenceGroup sg = ap.av.getSelectionGroup();
     if (sg == null)
     {
       /*
-- 
1.7.10.2