fix(gui): detect if a window is opened inside a visible screen (PR #521)

skylot · Mar 26, 2019 · 3537f84 · 3537f84
1 parent 9557f04
commit 3537f84
Showing 1 changed file with 16 additions and 2 deletions.
diff --git a/jadx-gui/src/main/java/jadx/gui/settings/JadxSettings.java b/jadx-gui/src/main/java/jadx/gui/settings/JadxSettings.java
@@ -1,6 +1,9 @@
 package jadx.gui.settings;
 
-import java.awt.*;
+import java.awt.Font;
+import java.awt.GraphicsDevice;
+import java.awt.GraphicsEnvironment;
+import java.awt.Window;
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.HashMap;
@@ -142,14 +145,25 @@ public void saveWindowPos(Window window) {
 
 	public boolean loadWindowPos(Window window) {
 		WindowLocation pos = windowPos.get(window.getClass().getSimpleName());
-		if (pos == null) {
+		if (pos == null || !isContainedInAnyScreen(pos)) {
 			return false;
 		}
+
 		window.setLocation(pos.getX(), pos.getY());
 		window.setSize(pos.getWidth(), pos.getHeight());
 		return true;
 	}
 
+	private static boolean isContainedInAnyScreen(WindowLocation pos) {
+		for (GraphicsDevice gd : GraphicsEnvironment.getLocalGraphicsEnvironment().getScreenDevices()) {
+			if (gd.getDefaultConfiguration().getBounds().contains(
+					pos.getX(), pos.getY(), pos.getWidth(), pos.getHeight())) {
+				return true;
+			}
+		}
+		return false;
+	}
+
 	public boolean isShowHeapUsageBar() {
 		return showHeapUsageBar;
 	}