Improved message coloring, messageStatus display only for own messages

additionally loaded the message status icons smaller and (visually) hid
the vertical scrollbars (they can still be used, but cannot be seen).
Lastly checked all context menu functions for a NullPointerException, as
also no element can be selected when the context menu is created.

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

@@ -35,14 +35,11 @@ public class ContactListCell extends ListCell<Contact> {
 			final var vbox = new VBox(new Label(contact.getName()));
 			if (contact instanceof User) {
 				// Online status
-				final var user = (User) contact;
+				final var	user		= (User) contact;
 				final var	statusLabel	= new Label(user.getStatus().toString());
 				statusLabel.getStyleClass().add(user.getStatus().toString().toLowerCase());
 				vbox.getChildren().add(statusLabel);
-			} else {
+			} else vbox.getChildren().add(new Label(((Group) contact).getContacts().size() + " members"));
-				// Member count
-				vbox.getChildren().add(new Label(((Group) contact).getContacts().size() + " members"));
-			}
 			setGraphic(vbox);
 		}
 	}

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

@@ -35,7 +35,7 @@ public class MessageListCell extends ListCell<Message> {
 
 	static {
 		try {
-			statusImages = IconUtil.loadByEnum(MessageStatus.class, 32);
+			statusImages = IconUtil.loadByEnum(MessageStatus.class, 16);
 		} catch (final IOException e) {
 			e.printStackTrace();
 			EnvoyLog.getLogger(MessageListCell.class).log(Level.WARNING, "could not load status icons: ", e);
@@ -54,9 +54,11 @@ public class MessageListCell extends ListCell<Message> {
 			setText(null);
 			setGraphic(null);
 		} else {
-			final var cell = new VBox(new Label(dateFormat.format(message.getCreationDate())), new Label(message.getText()),
+			final var cell = new VBox(new Label(dateFormat.format(message.getCreationDate())), new Label(message.getText()));
-					new Label("", new ImageView(statusImages.get(message.getStatus()))));
+			if (message.getRecipientID() == user.getID()) {
-			cell.getStyleClass().add(message.getRecipientID() == user.getID() ? "received-message" : "own-message");
+				cell.getChildren().add(new Label("", new ImageView(statusImages.get(message.getStatus()))));
+				cell.getStyleClass().add("own-message");
+			} else cell.getStyleClass().add("received-message");
 			cell.paddingProperty().setValue(new Insets(5, 20, 5, 20));
 			setGraphic(cell);
 		}

src/main/java/envoy/client/ui/controller/ChatScene.java

@@ -321,34 +321,35 @@ public final class ChatScene {
 
 	@FXML
 	private void copyMessage() {
-		Toolkit.getDefaultToolkit()
+		try {
-			.getSystemClipboard()
+			Toolkit.getDefaultToolkit()
-			.setContents(new StringSelection(messageList.getSelectionModel().getSelectedItem().getText()), null);
+				.getSystemClipboard()
+				.setContents(new StringSelection(messageList.getSelectionModel().getSelectedItem().getText()), null);
+		} catch (final NullPointerException e) {}
 	}
 
 	@FXML
-	private void deleteMessage() {}
+	private void deleteMessage() { try {} catch (final NullPointerException e) {} }
 
 	@FXML
-	private void forwardMessage() {}
+	private void forwardMessage() { try {} catch (final NullPointerException e) {} }
 
 	@FXML
-	private void quoteMessage() {}
+	private void quoteMessage() { try {} catch (final NullPointerException e) {} }
 
 	@FXML
-	private void deleteContact() {
+	private void deleteContact() { try {} catch (final NullPointerException e) {} }
-		// TODO: Fill with content
-		userList.getSelectionModel().getSelectedItem();
-	}
 
 	@FXML
 	private void copyAndPostMessage() {
-		final var messageText = messageTextArea.getText();
+		try {
-		Toolkit.getDefaultToolkit().getSystemClipboard().setContents(new StringSelection(messageText), null);
+			final var messageText = messageTextArea.getText();
-		postMessage();
+			Toolkit.getDefaultToolkit().getSystemClipboard().setContents(new StringSelection(messageText), null);
-		messageTextArea.setText(messageText);
+			postMessage();
-		updateRemainingCharsLabel();
+			messageTextArea.setText(messageText);
-		postButton.setDisable(messageText.isBlank());
+			updateRemainingCharsLabel();
+			postButton.setDisable(messageText.isBlank());
+		} catch (final NullPointerException e) {}
 	}
 
 }

src/main/resources/css/base.css

@@ -26,6 +26,11 @@
 	-fx-background-color: transparent;
 }
 
+.scroll-bar:horizontal, .scroll-bar *, .scroll-bar:horizontal > *{
+	-fx-background-color: transparent;
+	-fx-text-fill: transparent;
+}
+
 .online {
 	-fx-text-fill: limegreen;
 }

src/main/resources/css/dark.css

@@ -7,7 +7,7 @@
 }
 
 .button {
-	-fx-background-color: darkviolet;
+	-fx-background-color: rgb(105.0,0.0,153.0);
 }
 
 .button:pressed {
@@ -23,15 +23,15 @@
 }
 
 .list-cell:selected, .list-cell:selected > *, .menu-item:hover {
-    -fx-background-color: darkviolet;
+    -fx-background-color: rgb(105.0,0.0,153.0);
 }
 
 .received-message {
-	-fx-background-color: seagreen;
+	-fx-background-color: gray;
 }
 
 .own-message {
-	-fx-background-color: gray;
+	-fx-background-color: #8fa88f;
 }
 
 .alert.information.dialog-pane, .alert.warning.dialog-pane, .alert.error.dialog-pane {

src/main/resources/css/light.css

@@ -7,10 +7,10 @@
 	-fx-text-fill: black;
 }
 
-.received-message {
+.received-message, .menu-item {
-	-fx-background-color: lightgreen;
-}
-
-.own-message , .menu-item {
 	-fx-background-color: lightgray;
 }
+
+.own-message {
+	-fx-background-color: lightgreen;
+}