svn commit: r1937425 - pdfbox/trunk/debugger/src/main/java/org/apache/pdfbox/debugger/pagepane

[tilman]

Author: tilman
Date: Tue Aug 25 12:52:04 2026
New Revision: 1937425

Log:
PDFBOX-6240: add screen-bound popup positioning, as suggested by copilot

Modified:
   
pdfbox/trunk/debugger/src/main/java/org/apache/pdfbox/debugger/pagepane/PagePane.java

Modified: 
pdfbox/trunk/debugger/src/main/java/org/apache/pdfbox/debugger/pagepane/PagePane.java
==============================================================================
--- 
pdfbox/trunk/debugger/src/main/java/org/apache/pdfbox/debugger/pagepane/PagePane.java
       Tue Aug 25 12:52:01 2026        (r1937424)
+++ 
pdfbox/trunk/debugger/src/main/java/org/apache/pdfbox/debugger/pagepane/PagePane.java
       Tue Aug 25 12:52:04 2026        (r1937425)
@@ -557,7 +557,15 @@ public class PagePane implements ActionL
         hoverWindow.pack();
 
         Point screen = e.getLocationOnScreen();
-        hoverWindow.setLocation(screen.x + 12, screen.y + 12);
+        Dimension size = hoverWindow.getSize();
+        Dimension screenSize = Toolkit.getDefaultToolkit().getScreenSize();
+
+        int x = Math.min(screen.x + 12, screenSize.width - size.width);
+        int y = Math.min(screen.y + 12, screenSize.height - size.height);
+        x = Math.max(0, x);
+        y = Math.max(0, y);
+
+        hoverWindow.setLocation(x, y);
         if (!hoverWindow.isVisible())
         {
             hoverWindow.setVisible(true);