From eb21dfaec1eef8ddbccb002b417ece827fa17043 Mon Sep 17 00:00:00 2001
From: Brendan Robert <brendan.robert@gmail.com>
Date: Thu, 25 Feb 2016 01:40:27 -0600
Subject: [PATCH] Issue #14: Notification message is now formatted properly and
 multiple notifications no longer overlap

---
 src/main/java/jace/JaceUIController.java | 19 +++++++++----------
 1 file changed, 9 insertions(+), 10 deletions(-)

diff --git a/src/main/java/jace/JaceUIController.java b/src/main/java/jace/JaceUIController.java
index 48f47da..7b5c3c1 100644
--- a/src/main/java/jace/JaceUIController.java
+++ b/src/main/java/jace/JaceUIController.java
@@ -41,6 +41,9 @@ import javafx.scene.input.KeyEvent;
 import javafx.scene.input.MouseEvent;
 import javafx.scene.input.TransferMode;
 import javafx.scene.layout.AnchorPane;
+import javafx.scene.layout.Background;
+import javafx.scene.layout.BackgroundFill;
+import javafx.scene.layout.CornerRadii;
 import javafx.scene.layout.HBox;
 import javafx.scene.layout.StackPane;
 import javafx.scene.paint.Color;
@@ -245,24 +248,20 @@ public class JaceUIController {
     
     Label currentNotification = null;
     public void displayNotification(String message) {
-        if (currentNotification != null) {
-            Label remove = currentNotification;
-            Application.invokeLater(() -> {            
-                stackPane.getChildren().remove(remove);
-            });        
-        }
+        Label oldNotification = currentNotification;
         Label notification = new Label(message);
-        notification.setEffect(new DropShadow(1.0, Color.DARKGREY));
+        currentNotification = notification;
+        notification.setEffect(new DropShadow(2.0, Color.BLACK));
         notification.setTextFill(Color.WHITE);
-        Application.invokeLater(() -> {            
+        notification.setBackground(new Background(new BackgroundFill(Color.rgb(0,0,80, 0.7), new CornerRadii(5.0), new Insets(-5.0))));
+        Application.invokeLater(() -> {  
+            stackPane.getChildren().remove(oldNotification);
             stackPane.getChildren().add(notification);
         });
-        currentNotification = notification;
         
         notificationExecutor.schedule(()->{
             Application.invokeLater(() -> {            
                 stackPane.getChildren().remove(notification);
-                currentNotification = null;
             });                    
         }, 4, TimeUnit.SECONDS);
     }