From 2553d1149488f0c65e3cdcbac35459d39832ffe9 Mon Sep 17 00:00:00 2001
From: Reinhard Pointner <reinhard.pointner@gmail.com>
Date: Fri, 14 Jun 2019 00:53:11 +0700
Subject: [PATCH] Enhanced `Conflict Dialog` with detailed explanations

---
 CHANGES.md                                       |  1 +
 source/net/filebot/ui/rename/ConflictDialog.java | 12 ++++++++++--
 2 files changed, 11 insertions(+), 2 deletions(-)

diff --git a/CHANGES.md b/CHANGES.md
index a4158e11..3e55cd6e 100644
--- a/CHANGES.md
+++ b/CHANGES.md
@@ -1,6 +1,7 @@
 Next Release (4.8.6)
 ====================
 * Enhanced `Selection Dialog` with thumbnails and tooltips
+* Enhanced `Conflict Dialog` with detailed explanations
 * Improved support for mapping episode information between different databases and numbering schemes (e.g. via `AnimeLists` or `XEM`)
 * Added `{seasonize}` binding (e.g. map `AniDB` numbers to `TheTVDB` SxE numbers and vice versa)
 * Added `{history}` binding for looking up the original file path of `{f}` (e.g. useful for `-exec` post-processing commands)
diff --git a/source/net/filebot/ui/rename/ConflictDialog.java b/source/net/filebot/ui/rename/ConflictDialog.java
index 0f728bc0..08179f6c 100644
--- a/source/net/filebot/ui/rename/ConflictDialog.java
+++ b/source/net/filebot/ui/rename/ConflictDialog.java
@@ -29,6 +29,7 @@ import javax.swing.JScrollPane;
 import javax.swing.KeyStroke;
 import javax.swing.ListSelectionModel;
 import javax.swing.SwingUtilities;
+import javax.swing.border.CompoundBorder;
 
 import ca.odell.glazedlists.BasicEventList;
 import ca.odell.glazedlists.EventList;
@@ -36,6 +37,7 @@ import ca.odell.glazedlists.swing.DefaultEventListModel;
 import net.filebot.ResourceManager;
 import net.filebot.UserFiles;
 import net.filebot.media.VideoQuality;
+import net.filebot.util.ui.DashedSeparator;
 import net.filebot.util.ui.DefaultFancyListCellRenderer;
 import net.miginfocom.swing.MigLayout;
 
@@ -135,14 +137,20 @@ class ConflictDialog extends JDialog {
 
 	private static class ConflictCellRenderer extends DefaultFancyListCellRenderer {
 
+		public ConflictCellRenderer() {
+			setHighlightingEnabled(false);
+			setBorder(new CompoundBorder(new DashedSeparator(2, 4, getColor(0xEEEEEE), getPanelBackground()), getBorder()));
+		}
+
 		@Override
 		protected void configureListCellRendererComponent(JList list, Object value, int index, boolean isSelected, boolean cellHasFocus) {
-			setHighlightingEnabled(false);
-
 			Conflict conflict = (Conflict) value;
 			super.configureListCellRendererComponent(list, conflict.getDetails(), index, isSelected, cellHasFocus);
 			setIcon(ResourceManager.getIcon("status.warning"));
 			setToolTipText(formatToolTip(conflict));
+
+			// don't paint border on last element
+			setBorderPainted(index < list.getModel().getSize() - 1);
 		}
 
 		private String formatToolTip(Conflict conflict) {