On Mon, 30 Jan 2023 17:24:36 GMT, Kevin Rushforth <k...@openjdk.org> wrote:
>> On Windows platforms with more than one screen, a PopupWindow created for a >> Stage that straddles two windows will be drawn with an incorrect position >> and screen scale if the majority of the Stage is on one screen, and the >> popup is positioned on the other screen. In this case, the Stage is drawn >> using the screen scale of the screen that most of the window is on, while >> the popup is drawn using the scale of the screen that it is (typically >> entirely) on. >> >> The most common way this can happen is when you have two screens of a >> different scale with the secondary screen on the left or above the primary >> screen. If you position the Stage such that most of it is still on the >> primary screen (thus the Stage is drawn using the scale of the primary >> screen), with a menu, a control with a context menu, or a control with a >> Tooltip now on the secondary screen, the popup window for the menu or >> Tooltip will be drawn using the screen scale of the secondary window and >> thus won't be positioned or sized correctly relative to the menu bar, or >> control in the main window. >> >> The fix implemented by this PR is to always use the screen of the owner >> window, including the screen scales, when rendering a popup window. This >> matches the behavior of native Windows apps, such as Notepad. > > Kevin Rushforth has updated the pull request with a new target base due to a > merge or a rebase. The incremental webrev excludes the unrelated changes > brought in by the merge/rebase. The pull request contains three additional > commits since the last revision: > > - Merge branch 'master' into 8251862-multi-screen-popup > - Merge remote-tracking branch 'origin/master' into > 8251862-multi-screen-popup > - 8251862: Wrong position of Popup windows at the intersection of 2 screens I filed [JDK-8301386](https://bugs.openjdk.org/browse/JDK-8301386) to track the additional problem in the code that adjusts the position of the popup to keep it on the same screen as the anchor. ------------- PR: https://git.openjdk.org/jfx/pull/971
