From: gmungoc <g.m.carstairs@dundee.ac.uk>
Date: Thu, 22 Dec 2016 14:03:20 +0000 (+0000)
Subject: JAL-2360 fix bug that prematurely created a SequenceGroup
X-Git-Tag: Release_2_10_3b1~357^2~30
X-Git-Url: http://source.jalview.org/gitweb/?a=commitdiff_plain;h=313fff566aa8e673b50c4c4e4b216d2704c5ceae;p=jalview.git

JAL-2360 fix bug that prematurely created a SequenceGroup
---

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)
     {
       /*