fixed some issues on add/remove positions

todo: positions in editEmployee?

SchedulerApp/src/main/java/com/example/schedulerapp/Controller.java

@@ -8,6 +8,7 @@ import javafx.event.Event;
 import javafx.event.EventHandler;
 import javafx.fxml.FXML;
 import javafx.fxml.FXMLLoader;
+import javafx.geometry.Insets;
 import javafx.scene.Node;
 import javafx.scene.Parent;
 import javafx.scene.Scene;
@@ -563,16 +564,21 @@ public class Controller {
     }
     public void addRemovePosition(){
         VBox tempBox = new VBox();
+        tempBox.setPadding(new Insets(20));
+        tempBox.setSpacing(20);
         ComboBox tempCombo = new ComboBox();
         Button addPosition = new Button("Add Position");
         Button removePosition = new Button("Remove Position");
-        tempBox.getChildren().addAll(tempCombo,addPosition,removePosition);
+        Label result = new Label("");
+        tempBox.getChildren().addAll(tempCombo,addPosition,removePosition, result);
         tempCombo.setOnMouseClicked(this::populatePositionBox);
 
         addPosition.setOnMouseClicked(new EventHandler<MouseEvent>() {
             @Override
             public void handle(MouseEvent mouseEvent) {
                 VBox tempVBox = new VBox();
+                tempVBox.setSpacing(20);
+                tempVBox.setPadding(new Insets(30));
                 TextField tempPositionText = new TextField("Enter Position Here");
                 TextField tempWageText = new TextField("Enter Wage Here");
                 Button addPositionSubmit = new Button("Submit");
@@ -591,16 +597,11 @@ public class Controller {
                 });
             }
         });
-        removePosition.setOnMouseClicked(new EventHandler<MouseEvent>() {
-            @Override
-            public void handle(MouseEvent mouseEvent) {
-                int boxIndex = tempCombo.getSelectionModel().getSelectedIndex();
-                if(boxIndex >= 0){
-                    // TODO: remove position
-                    System.out.println("remove position clicked");
-                }
-            }
-        });
+        removePosition.setOnMouseClicked(e -> {
+            if (tempCombo.getValue() == null) result.setText("No position selected.");
+            else result.setText(model.removePosition(tempCombo.getSelectionModel().getSelectedItem().toString()));
+                });
+
         root.setCenter(tempBox);
     }
 

SchedulerApp/src/main/java/com/example/schedulerapp/Model.java

@@ -697,6 +697,9 @@ public class Model {
     }
 
     String removePosition(String position) {
+        if (position == null){
+            return "No position selected.";
+        }
         if (position.contains("/") || position.contains(",")){
             return "Illegal character(s)";
         }