ComboBox putting menu AFTER body

hansonr · hansonr · commit 073c2da5b7d4 · 2019-07-12T14:27:51.000+01:00
diff --git a/sources/net.sf.j2s.java.core/src/swingjs/jquery/JQueryUI.java b/sources/net.sf.j2s.java.core/src/swingjs/jquery/JQueryUI.java
@@ -2,7 +2,7 @@
 
 /**
  * This "class" simply checks for jQuery.ui having been loaded and loads it if it has not been loaded.
- * Used by swingjs.plaf.JSSliderUI and swingjs.plaf.JSPopupMenuUI
+ * Used by swingjs.plaf.JSSliderUI JSPopupMenuUI JSComboBoxUI
  * 
  * Note that if you change one of the JavaScript files in this directory, 
  * the transpiler will not transfer it to the site directory unless you touch this java file.
diff --git a/sources/net.sf.j2s.java.core/src/swingjs/jquery/j2sComboBox.js b/sources/net.sf.j2s.java.core/src/swingjs/jquery/j2sComboBox.js
@@ -83,8 +83,8 @@ $( function() {
         this.on2(this.list, 'click mousedown touchstart mousemove touchmove mouseup touchend mousewheel mouseover mouseout mouseenter mouseexit'.split(' '), '_mouse');
         this.popup.append(this.list);        
         this.element.append(this.cont);
-        // important to add popup to body, because it is 
-        $('body').append(this.popup);
+        // important to add popup after body so that it does not take on any body attributes 
+        $('body').after(this.popup);
         this.updateCSS();    	
         this.on( [this.head, this.btn, this.cont], { click: '_open' });
         this.on( [this.popup, this.list], {mouseover: stopT });
diff --git a/sources/net.sf.j2s.java.core/src/swingjs/jquery/j2sMenu.js b/sources/net.sf.j2s.java.core/src/swingjs/jquery/j2sMenu.js
@@ -3,8 +3,6 @@
 // author: Bob Hanson, hansonr@stolaf.edu
 // last edited 1/23/2019 -- fixes for mouse-down effector (mac) operation
 
-// TODO: test for dynamic update of menu items, as in Jmol
-
 /*! jQuery UI - v1.9.2 - 2012-12-17
 * http://jqueryui.com
 * Includes: jquery.ui.core.js, jquery.ui.widget.js, jquery.ui.mouse.js, jquery.ui.position.js, jquery.ui.menu.js

Original file line number	Diff line number	Diff line change
`@@ -2,7 +2,7 @@`
`2`	`2`
`3`	`3`	`/**`
`4`	`4`	`* This "class" simply checks for jQuery.ui having been loaded and loads it if it has not been loaded.`
`5`		`- * Used by swingjs.plaf.JSSliderUI and swingjs.plaf.JSPopupMenuUI`
	`5`	`+ * Used by swingjs.plaf.JSSliderUI JSPopupMenuUI JSComboBoxUI`
`6`	`6`	`*`
`7`	`7`	`* Note that if you change one of the JavaScript files in this directory,`
`8`	`8`	`* the transpiler will not transfer it to the site directory unless you touch this java file.`