X-Git-Url: http://git.nikiroo.be/?a=blobdiff_plain;f=ui%2FListModel.java;h=d76c5dc2c169a595e6a90b48d41b110bac7d6a46;hb=587139b9fbf372c79b1650a86fc4d316c36c6bed;hp=137897aee45b68bbc682eae949fa8fcd9e15c230;hpb=a1d54d1127742f5cf7094d74dd51a8ce2b627dba;p=fanfix.git

diff --git a/ui/ListModel.java b/ui/ListModel.java
index 137897a..d76c5dc 100644
--- a/ui/ListModel.java
+++ b/ui/ListModel.java
@@ -14,9 +14,9 @@ import javax.swing.JPopupMenu;
 import javax.swing.ListCellRenderer;
 import javax.swing.SwingWorker;
 
-import be.nikiroo.utils.compat.DefaultListModel6;
-import be.nikiroo.utils.compat.JList6;
-import be.nikiroo.utils.compat.ListCellRenderer6;
+import be.nikiroo.utils.ui.compat.DefaultListModel6;
+import be.nikiroo.utils.ui.compat.JList6;
+import be.nikiroo.utils.ui.compat.ListCellRenderer6;
 
 /**
  * A {@link javax.swing.ListModel} that can maintain 2 lists; one with the
@@ -251,7 +251,7 @@ public class ListModel<T> extends DefaultListModel6<T> {
 	 */
 	@SuppressWarnings({ "unchecked", "rawtypes" }) // JList<?> not in Java 1.6
 	public ListModel(final JList list, final JPopupMenu popup,
-			TooltipCreator<T> tooltipCreator) {
+			final TooltipCreator<T> tooltipCreator) {
 		this.list = list;
 		this.tooltipCreator = tooltipCreator;
 
@@ -276,14 +276,9 @@ public class ListModel<T> extends DefaultListModel6<T> {
 					fireElementChanged(oldIndex);
 					fireElementChanged(index);
 
-					synchronized (tooltipWatcher) {
-						if (tooltip != null) {
-							tooltip.setVisible(false);
-						}
-						tooltip = null;
-					}
+					if (tooltipCreator != null) {
+						showTooltip(null);
 
-					if (ListModel.this.tooltipCreator != null) {
 						tooltipWatcher.delay("tooltip",
 								new SwingWorker<Void, Void>() {
 									@Override
@@ -294,12 +289,7 @@ public class ListModel<T> extends DefaultListModel6<T> {
 
 									@Override
 									protected void done() {
-										synchronized (tooltipWatcher) {
-											if (tooltip != null) {
-												tooltip.setVisible(false);
-											}
-											tooltip = null;
-										}
+										showTooltip(null);
 
 										if (index < 0
 												|| index != hoveredIndex) {
@@ -311,12 +301,7 @@ public class ListModel<T> extends DefaultListModel6<T> {
 											return;
 										}
 
-										synchronized (tooltipWatcher) {
-											if (tooltip != null) {
-												tooltip.setVisible(false);
-											}
-											tooltip = newTooltip(index, me);
-										}
+										showTooltip(newTooltip(index, me));
 									}
 								});
 					}
@@ -358,12 +343,7 @@ public class ListModel<T> extends DefaultListModel6<T> {
 								list.locationToIndex(e.getPoint()));
 					}
 
-					Window oldTooltip = tooltip;
-					tooltip = null;
-					if (oldTooltip != null) {
-						oldTooltip.setVisible(false);
-					}
-
+					showTooltip(null);
 					popup.show(list, e.getX(), e.getY());
 				}
 			}
@@ -581,9 +561,7 @@ public class ListModel<T> extends DefaultListModel6<T> {
 
 	private Window newTooltip(final int index, final MouseEvent me) {
 		final T value = ListModel.this.get(index);
-
 		final Window newTooltip = tooltipCreator.generateTooltip(value, true);
-
 		if (newTooltip != null) {
 			newTooltip.addMouseListener(new MouseAdapter() {
 				@Override
@@ -595,18 +573,33 @@ public class ListModel<T> extends DefaultListModel6<T> {
 								me.getYOnScreen());
 						promotedTooltip.setVisible(true);
 					}
-					
+
 					newTooltip.setVisible(false);
 				}
 			});
 
 			newTooltip.setLocation(me.getXOnScreen(), me.getYOnScreen());
-			newTooltip.setVisible(true);
+			showTooltip(newTooltip);
 		}
 
 		return newTooltip;
 	}
 
+	private void showTooltip(Window tooltip) {
+		synchronized (tooltipCreator) {
+			if (this.tooltip != null) {
+				this.tooltip.setVisible(false);
+				this.tooltip.dispose();
+			}
+
+			this.tooltip = tooltip;
+
+			if (tooltip != null) {
+				tooltip.setVisible(true);
+			}
+		}
+	}
+
 	/**
 	 * Generate a {@link ListCellRenderer} that supports {@link Hoverable}
 	 * elements.