Error in popup menu size fixed

8a6f5161 · claes · c517727b · 8a6f5161 · 8a6f5161
Commit 8a6f5161 authored Jun 17, 2004 by claes
Show whitespace changes
Inline Side-by-side

Showing with 4 additions and 4 deletions

java/jpwr/jop/src/GeDynOptionMenu.java java/jpwr/jop/src/GeDynOptionMenu.java +2 -2

src/jpwr/jop/src/GeDynOptionMenu.java src/jpwr/jop/src/GeDynOptionMenu.java +2 -2

No files found.
--- a/java/jpwr/jop/src/GeDynOptionMenu.java
+++ b/java/jpwr/jop/src/GeDynOptionMenu.java
@@ -122,7 +122,6 @@ public class GeDynOptionMenu extends GeDynElem {
      JMenuItem item;
      JPopupMenu popup = new JPopupMenu();
-      popup.setPopupSize( width, popup.getHeight());
      for ( int i = 0; i < 32; i++) {
        if ( itemsText[i] != null) {
 	  popup.add( item = new JMenuItem( itemsText[i]));
@@ -133,6 +132,7 @@ public class GeDynOptionMenu extends GeDynElem {
      popup.setBackground(color);
      popup.addPopupMenuListener( this);
      popup.show( invoker, x, y);
+      popup.setPopupSize( width, popup.getHeight());
    }
    public void popupMenuWillBecomeVisible( PopupMenuEvent e) {}
@@ -166,7 +166,7 @@ public class GeDynOptionMenu extends GeDynElem {
 	  }
 	  else
 	    return;
-	  System.out.println("Action " + event.getActionCommand());
+	  // System.out.println("Action " + event.getActionCommand());
 	  break;
        }
      }

--- a/src/jpwr/jop/src/GeDynOptionMenu.java
+++ b/src/jpwr/jop/src/GeDynOptionMenu.java
@@ -122,7 +122,6 @@ public class GeDynOptionMenu extends GeDynElem {
      JMenuItem item;
      JPopupMenu popup = new JPopupMenu();
-      popup.setPopupSize( width, popup.getHeight());
      for ( int i = 0; i < 32; i++) {
        if ( itemsText[i] != null) {
 	  popup.add( item = new JMenuItem( itemsText[i]));
@@ -133,6 +132,7 @@ public class GeDynOptionMenu extends GeDynElem {
      popup.setBackground(color);
      popup.addPopupMenuListener( this);
      popup.show( invoker, x, y);
+      popup.setPopupSize( width, popup.getHeight());
    }
    public void popupMenuWillBecomeVisible( PopupMenuEvent e) {}
@@ -166,7 +166,7 @@ public class GeDynOptionMenu extends GeDynElem {
 	  }
 	  else
 	    return;
-	  System.out.println("Action " + event.getActionCommand());
+	  // System.out.println("Action " + event.getActionCommand());
 	  break;
        }
      }