Changeset 15687 in josm for trunk/src

Timestamp:

2020-01-11T21:02:08+01:00 (5 years ago)

Author:

simon04

Message:

see #13901 - Tag2Link: add popup menu separator

File:

• trunk/src/org/openstreetmap/josm/gui/util/AbstractTag2LinkPopupListener.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/gui/util/AbstractTag2LinkPopupListener.java

@@ -2 +2 @@
 package org.openstreetmap.josm.gui.util;
 
+import java.awt.Component;
 import java.util.ArrayList;
 import java.util.List;
 
-import javax.swing.JMenuItem;
 import javax.swing.JPopupMenu;
 import javax.swing.event.PopupMenuEvent;
@@ -17 +17 @@
 public abstract class AbstractTag2LinkPopupListener implements PopupMenuListener {
 
-    private final List<JMenuItem> itemList;
+    private final List<Component> itemList;
 
     protected AbstractTag2LinkPopupListener() {
@@ -35 +35 @@
 
     protected void addLinks(JPopupMenu popup, String key, String value) {
-        Tag2Link.getLinksForTag(key, value, (name, url) -> itemList.add(popup.add(new OpenBrowserAction(name, url))));
+        Tag2Link.getLinksForTag(key, value, (name, url) -> {
+            if (itemList.isEmpty()) {
+                itemList.add(popup.add(new JPopupMenu.Separator()));
+            }
+
+            itemList.add(popup.add(new OpenBrowserAction(name, url)));
+        });
     }
 }