AWT Frame default close operation; JFrame setVisible(true)

hansonr · hansonr · commit 8e8c636ee556 · 2019-03-15T04:17:46.000-05:00
AWT has OPERATION_DO_NOTHING default for window closing

JFrame should set domNode visibility along with outerNode in
setVisible(true);
diff --git a/sources/net.sf.j2s.java.core/src/swingjs/a2s/Frame.java b/sources/net.sf.j2s.java.core/src/swingjs/a2s/Frame.java
@@ -36,6 +36,7 @@ public Frame(GraphicsConfiguration gc) {
 	public Frame(String title, GraphicsConfiguration gc) {
 		super(title, gc);
 		listener = new A2SListener();
+		setDefaultCloseOperation(DO_NOTHING_ON_CLOSE);
 		Util.setAWTWindowDefaults(this);
 	}
 
diff --git a/sources/net.sf.j2s.java.core/src/swingjs/plaf/JSFrameUI.java b/sources/net.sf.j2s.java.core/src/swingjs/plaf/JSFrameUI.java
@@ -344,7 +344,7 @@ public void setVisible(boolean b) {
 		  }
 		  DOMNode.setVisible(modalNode, b);
 	  }
-		  
+	  DOMNode.setVisible(domNode, b);		  
 	}
 
 }

Original file line number	Diff line number	Diff line change
`@@ -36,6 +36,7 @@ public Frame(GraphicsConfiguration gc) {`
`36`	`36`	`public Frame(String title, GraphicsConfiguration gc) {`
`37`	`37`	`super(title, gc);`
`38`	`38`	`listener = new A2SListener();`
	`39`	`+ setDefaultCloseOperation(DO_NOTHING_ON_CLOSE);`
`39`	`40`	`Util.setAWTWindowDefaults(this);`
`40`	`41`	`}`
`41`	`42`
Original file line number	Diff line number	Diff line change
`@@ -344,7 +344,7 @@ public void setVisible(boolean b) {`
`344`	`344`	`}`
`345`	`345`	`DOMNode.setVisible(modalNode, b);`
`346`	`346`	`}`
`347`		`-`
	`347`	`+ DOMNode.setVisible(domNode, b);`
`348`	`348`	`}`
`349`	`349`
`350`	`350`	`}`