Fix line refresh

Recycling was wrong in some cases.
zapek · Nov 13, 2023 · 23d1c02 · 23d1c02
1 parent 03336f9
commit 23d1c02
Showing 1 changed file with 7 additions and 9 deletions.
diff --git a/ui/src/main/java/io/xeres/ui/custom/ChatListCell.java b/ui/src/main/java/io/xeres/ui/custom/ChatListCell.java
@@ -46,7 +46,7 @@ public class ChatListCell implements Cell<ChatLine, TextFlow>
 	private final TextFlow content;
 	private final Label time;
 	private final Label action;
-	private boolean isComplex;
+	private boolean isRich;
 
 	public ChatListCell(ChatLine line)
 	{
@@ -73,27 +73,28 @@ public TextFlow getNode()
 	@Override
 	public boolean isReusable()
 	{
-		return !isComplex;
+		return !isRich;
 	}
 
 	@Override
 	public void reset()
 	{
 		if (isReusable())
 		{
-			content.getChildren().remove(2);
+			content.getChildren().remove(2); // keep time and action only
 		}
 	}
 
 	@Override
 	public void updateItem(ChatLine line)
 	{
-		isComplex = line.isRich();
+		isRich = line.isRich();
 
 		time.setText(formatter.format(line.getInstant()));
+
 		action.setText(line.getAction());
-		var nicknameColor = line.getNicknameColor();
 		action.getStyleClass().removeAll(allColors);
+		var nicknameColor = line.getNicknameColor();
 		if (nicknameColor != null)
 		{
 			action.getStyleClass().add(nicknameColor);
@@ -103,10 +104,7 @@ public void updateItem(ChatLine line)
 				.map(Content::getNode)
 				.toList();
 
-		if (!isComplex)
-		{
-			content.pseudoClassStateChanged(passivePseudoClass, !line.isActiveAction());
-		}
+		content.pseudoClassStateChanged(passivePseudoClass, !line.isActiveAction());
 
 		content.getChildren().addAll(nodes);
 	}