JAL-2361 make a modified saved scheme the backout checkpoint for Cancel
authorgmungoc <g.m.carstairs@dundee.ac.uk>
Thu, 6 Apr 2017 15:52:23 +0000 (16:52 +0100)
committergmungoc <g.m.carstairs@dundee.ac.uk>
Thu, 6 Apr 2017 15:52:23 +0000 (16:52 +0100)
src/jalview/gui/UserDefinedColours.java

index 369dde6..0187aa3 100755 (executable)
@@ -685,16 +685,18 @@ public class UserDefinedColours extends GUserDefinedColours implements
     if (value == JalviewFileChooser.APPROVE_OPTION)
     {
       File file = chooser.getSelectedFile();
-      addNewColourScheme(file.getPath());
+      UserColourScheme updatedScheme = addNewColourScheme(file.getPath());
       saveToFile(file);
 
       /*
        * changes saved - apply to alignment if we are changing 
-       * the currently selected colour scheme
+       * the currently selected colour scheme; also make the updated
+       * colours the 'backout' scheme on Cancel
        */
       if (oldColourScheme != null
               && name.equals(oldColourScheme.getSchemeName()))
       {
+        oldColourScheme = updatedScheme;
         applyButton_actionPerformed();
       }
     }
@@ -707,8 +709,9 @@ public class UserDefinedColours extends GUserDefinedColours implements
    * the colour scheme.
    * 
    * @param filePath
+   * @return
    */
-  protected void addNewColourScheme(String filePath)
+  protected UserColourScheme addNewColourScheme(String filePath)
   {
     /*
      * update the delimited list of user defined colour files in
@@ -739,6 +742,8 @@ public class UserDefinedColours extends GUserDefinedColours implements
     {
       ap.alignFrame.buildColourMenu();
     }
+
+    return ucs;
   }
 
   /**