Move user status color to CSS, refactor ContactListCell

src/main/java/envoy/client/ui/ContactListCell.java

@@ -31,33 +31,19 @@ public class ContactListCell extends ListCell<Contact> {
 			setText(null);
 			setGraphic(null);
 		} else {
-			// the infoLabel displays specific contact info, i.e. status of a user or amount
+			// Container with contact name
-			// of members in a group
+			final var vbox = new VBox(new Label(contact.getName()));
-			Label infoLabel = null;
 			if (contact instanceof User) {
-				// user specific info
+				// Online status
-				infoLabel = new Label(((User) contact).getStatus().toString());
+				final var user = (User) contact;
-				String textColor = null;
+				final var	statusLabel	= new Label(user.getStatus().toString());
-				switch (((User) contact).getStatus()) {
+				statusLabel.getStyleClass().add(user.getStatus().toString().toLowerCase());
-					case ONLINE:
+				vbox.getChildren().add(statusLabel);
-						textColor = "limegreen";
+			} else {
-						break;
+				// Member count
-					case AWAY:
+				vbox.getChildren().add(new Label(((Group) contact).getContacts().size() + " members"));
-						textColor = "orangered";
+			}
-						break;
+			setGraphic(vbox);
-					case BUSY:
-						textColor = "red";
-						break;
-					case OFFLINE:
-						textColor = "gray";
-						break;
-				}
-				// infoLabel.setTextFill(textColor) does not work as it gets overridden by CSS;
-				infoLabel.setStyle("-fx-text-fill: " + textColor);
-			} else
-				// group specific infos
-				infoLabel = new Label(String.valueOf(((Group) contact).getContacts().size()) + " members");
-			setGraphic(new VBox(new Label(contact.getName()), infoLabel));
 		}
 	}
 }

src/main/resources/css/base.css

@@ -12,3 +12,19 @@
 	-fx-opacity: 1;
 	-fx-background-color: transparent;
 }
+
+.online {
+	-fx-text-fill: limegreen;
+}
+
+.away {
+	-fx-text-fill: orangered;
+}
+
+.busy {
+	-fx-text-fill: red;
+}
+
+.offline {
+	-fx-text-fill: gray;
+}