fixing menu dynamic add/remove

hansonr · hansonr · commit d38d2a91c62e · 2018-11-15T18:14:40.000-06:00
diff --git a/sources/net.sf.j2s.java.core/src/swingjs/plaf/JSMenuItemUI.java b/sources/net.sf.j2s.java.core/src/swingjs/plaf/JSMenuItemUI.java
@@ -2,6 +2,7 @@
 
 
 import java.awt.Dimension;
+import java.beans.PropertyChangeEvent;
 
 import javax.swing.JComponent;
 import javax.swing.JMenuItem;
@@ -48,4 +49,17 @@ public void installUI(JComponent jc) {
         "MenuItem.font");		
 	}
 	
+	@Override
+	public void propertyChange(PropertyChangeEvent e) {
+		super.propertyChange(e);
+		String prop = e.getPropertyName();
+		if (jc.isVisible()) {
+			if (prop == "ancestor") {
+				if (jc.getParent() != null) {
+					((JSComponentUI) jc.getParent().getUI()).setHTMLElement();
+				}
+			}
+		}
+	}
+
 }
diff --git a/sources/net.sf.j2s.java.core/src/test/JalviewJSTest.java b/sources/net.sf.j2s.java.core/src/test/JalviewJSTest.java
@@ -40,6 +40,8 @@ public static void main(String[] args)
     new JalviewJSTest().doTest();
   }
 
+private JMenuItem mb5;
+
   /**
    * Put some content in a JFrame and show it
    */
@@ -187,6 +189,11 @@ Container getVisualPaneContent(JMenu menu)
     mb4.setComponentOrientation(ComponentOrientation.RIGHT_TO_LEFT);
     mb4.setHorizontalTextPosition(SwingConstants.RIGHT);
 
+    mb5 = new JMenuItem("added");
+    mb5.setFont(font);
+    mb5.setComponentOrientation(ComponentOrientation.RIGHT_TO_LEFT);
+    mb5.setHorizontalTextPosition(SwingConstants.RIGHT);
+
     
     JMenu m1 = new JMenu("left");
     JMenu m2 = new JMenu("right");
@@ -227,8 +234,9 @@ private ImageIcon getImage(String name) {
 @Override
 public void menuSelected(MenuEvent e) {
 	System.out.println("menuSelected " + e.getSource().toString());
-	
-}
+	JMenu menu = (JMenu) e.getSource();
+	menu.add(mb5);
+}	
 
 @Override
 public void menuDeselected(MenuEvent e) {
