X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;ds=inline;f=src%2Fjalview%2Fgui%2FUserDefinedColours.java;h=37824fe1688913d7726caaf9837d1544e2ae07b9;hb=2799bb913bbda0455b0f5f43737da678794e1961;hp=7bf2e6c4429d0296731f42a8fa68b5dbe2781bf6;hpb=be5302e00fffeacc62877c2aca981eb9ede3525e;p=jalview.git

diff --git a/src/jalview/gui/UserDefinedColours.java b/src/jalview/gui/UserDefinedColours.java
index 7bf2e6c..37824fe 100755
--- a/src/jalview/gui/UserDefinedColours.java
+++ b/src/jalview/gui/UserDefinedColours.java
@@ -454,14 +454,26 @@ public class UserDefinedColours extends GUserDefinedColours implements
   @Override
   protected void okButton_actionPerformed(ActionEvent e)
   {
-    applyButton_actionPerformed(null);
-
-    try
-    {
-      frame.setClosed(true);
-    } catch (Exception ex)
-    {
-    }
+		//Check if the user have done any selection
+		boolean showWarning = (upperCaseButtons==null) ||
+					((upperCaseButtons!=null) && (upperCaseButtons.size()==0)) ||
+					(lowerCaseButtons==null) ||
+					((lowerCaseButtons!=null) && (lowerCaseButtons.size()==0));
+		if (showWarning){
+	        JOptionPane.showMessageDialog(Desktop.desktop,
+	                MessageManager.getString("label.no_colour_selection_in_scheme"),MessageManager.getString("label.no_colour_selection_warn"),
+	                JOptionPane.WARNING_MESSAGE);
+			
+		}else{
+		    applyButton_actionPerformed(null);
+		
+		    try
+		    {
+		      frame.setClosed(true);
+		    } catch (Exception ex)
+		    {
+		    }
+		}
   }
 
   /**
@@ -473,7 +485,18 @@ public class UserDefinedColours extends GUserDefinedColours implements
   @Override
   protected void applyButton_actionPerformed(ActionEvent e)
   {
-    UserColourScheme ucs = getSchemeFromButtons();
+	//Check if the user have done any selection
+	boolean showWarning = (upperCaseButtons==null) ||
+				((upperCaseButtons!=null) && (upperCaseButtons.size()==0)) ||
+				(lowerCaseButtons==null) ||
+				((lowerCaseButtons!=null) && (lowerCaseButtons.size()==0));
+	if (showWarning){
+        JOptionPane.showMessageDialog(Desktop.desktop,
+                MessageManager.getString("label.no_colour_selection_in_scheme"),MessageManager.getString("label.no_colour_selection_warn"),
+                JOptionPane.WARNING_MESSAGE);
+		
+	}
+	UserColourScheme ucs = getSchemeFromButtons();
     ucs.setName(schemeName.getText());
 
     if (seqGroup != null)
@@ -496,10 +519,18 @@ public class UserDefinedColours extends GUserDefinedColours implements
 
     Color[] newColours = new Color[24];
 
-    for (int i = 0; i < 24; i++)
-    {
-      JButton button = upperCaseButtons.get(i);
-      newColours[i] = button.getBackground();
+    int length = upperCaseButtons.size();
+    if (length<24){
+    	int i = 0;
+    	for (JButton btn:upperCaseButtons){
+    		newColours[i] = btn.getBackground();
+    		i++;
+    	}
+    }else{
+    	for (int i = 0; i < 24; i++){
+    		JButton button = (JButton) upperCaseButtons.get(i);
+    		newColours[i] = button.getBackground();
+    	}
     }
 
     UserColourScheme ucs = new UserColourScheme(newColours);
@@ -507,10 +538,18 @@ public class UserDefinedColours extends GUserDefinedColours implements
     if (caseSensitive.isSelected())
     {
       newColours = new Color[23];
-      for (int i = 0; i < 23; i++)
-      {
-        JButton button = lowerCaseButtons.get(i);
-        newColours[i] = button.getBackground();
+      length = lowerCaseButtons.size();
+      if (length<23){
+    	  int i = 0;
+    	  for (JButton btn:lowerCaseButtons){
+    		  newColours[i] = btn.getBackground();
+    		  i++;
+    	  }
+      }else{
+    	  for (int i = 0; i < 23; i++){
+    		  JButton button = (JButton) lowerCaseButtons.get(i);
+    		  newColours[i] = button.getBackground();
+    	  }
       }
       ucs.setLowerCaseColours(newColours);
     }