1 files changed, 69 insertions, 15 deletions

diff --git a/src/jcgp/gui/settings/parameters/GUIIntegerParameter.java b/src/jcgp/gui/settings/parameters/GUIIntegerParameter.java
index 70b78a6..bf1a61e 100644
--- a/src/jcgp/gui/settings/parameters/GUIIntegerParameter.java
+++ b/src/jcgp/gui/settings/parameters/GUIIntegerParameter.java
@@ -1,49 +1,103 @@
 package jcgp.gui.settings.parameters;
 
+import javafx.beans.value.ChangeListener;
+import javafx.beans.value.ObservableValue;
+import javafx.event.EventHandler;
 import javafx.geometry.Pos;
 import javafx.scene.control.TextField;
+import javafx.scene.input.KeyEvent;
 import javafx.scene.layout.Priority;
 import javafx.scene.text.Text;
 import jcgp.backend.parameters.IntegerParameter;
+import jcgp.backend.parameters.ParameterStatus;
 import jcgp.gui.GUI;
 
 public class GUIIntegerParameter extends GUIParameter {
 
-	GUIIntegerParameter(IntegerParameter parameter) {	
+	private int originalValue;
+	
+	GUIIntegerParameter(final IntegerParameter parameter) {	
 		super();
 		
 		this.parameter = parameter;
 		
+		originalValue = parameter.get();
+		
 		name = new Text(parameter.getName());
-		value = new TextField(String.valueOf(parameter.get()));
+		valueControl = new TextField(String.valueOf(parameter.get()));
+		valueControl.setStyle(GUI.VALID_PARAMETER_STYLE);
 		
-		((TextField) value).setAlignment(Pos.CENTER_RIGHT);
+		((TextField) valueControl).setAlignment(Pos.CENTER_RIGHT);
 		
-		setHgrow(value, Priority.ALWAYS);
+		setHgrow(valueControl, Priority.ALWAYS);
 		
 		name.setWrappingWidth(GUI.WRAP_WIDTH);
 		
-		((TextField) value).setEditable(!parameter.isMonitor());
-
+		((TextField) valueControl).setEditable(!parameter.isMonitor());
+		
+		// bind if monitor, else set listeners
 		if (parameter.isMonitor()) {
 			makeLightBinding();
+		} else {
+			valueControl.addEventFilter(KeyEvent.KEY_TYPED, new EventHandler<KeyEvent>() {
+				public void handle( KeyEvent t ) {
+					char ch = t.getCharacter().toCharArray()[t.getCharacter().toCharArray().length - 1];
+					if (!(ch >= '0' && ch <= '9')) {
+						t.consume();
+					}
+				}
+			});
+			((TextField) valueControl).textProperty().addListener(new ChangeListener<String>() {
+				@Override
+				public void changed(
+						ObservableValue<? extends String> observable,
+						String oldValue, String newValue) {
+					if (!newValue.isEmpty()) {
+						parameter.set(Integer.parseInt(newValue));
+						if (parameter.getStatus() == ParameterStatus.INVALID) {
+							valueControl.setStyle(GUI.INVALID_PARAMETER_STYLE);
+							valueControl.setTooltip(tooltip);
+							tooltip.setText(parameter.getStatus().getDetails());
+						} else if (parameter.getStatus() == ParameterStatus.WARNING) {
+							valueControl.setStyle(GUI.WARNING_PARAMETER_STYLE);
+							valueControl.setTooltip(tooltip);
+							tooltip.setText(parameter.getStatus().getDetails());
+						} else {
+							valueControl.setStyle(GUI.VALID_PARAMETER_STYLE);
+							valueControl.setTooltip(null);
+						}
+					}
+
+				}
+			});
+			valueControl.focusedProperty().addListener(new ChangeListener<Boolean>() {
+				@Override
+				public void changed(
+						ObservableValue<? extends Boolean> observable,
+						Boolean oldValue, Boolean newValue) {
+					if (!newValue && ((TextField) valueControl).getText().isEmpty()) {
+						((TextField) valueControl).setText(String.valueOf(parameter.get()));
+					}
+				}
+			});
 		}
 				
-		getChildren().addAll(name, value);
+		getChildren().addAll(name, valueControl);
 	}
 
 	@Override
 	public void refreshValue() {
-		((TextField) value).setText(String.valueOf(((IntegerParameter) parameter).get()));
+		((TextField) valueControl).setText(String.valueOf(((IntegerParameter) parameter).get()));
 	}
 
 	@Override
-	public boolean applyChange() {
-		if (Integer.valueOf(((TextField) value).getText()) != ((IntegerParameter) parameter).get() && (!parameter.isMonitor())) {
-			// apply value
-			((IntegerParameter) parameter).set(Integer.valueOf(((TextField) value).getText()));
-			return parameter.isCritical();
-		}
-		return false;
+	public boolean requiresReset() {
+		return parameter.isCritical() && ((IntegerParameter) parameter).get() != originalValue;
+	}
+	
+	@Override
+	public void applyValue() {
+		originalValue = ((IntegerParameter) parameter).get();
 	}
+
 }