On Wed, 2 Apr 2025 17:05:58 GMT, Jose Pereda <jper...@openjdk.org> wrote:

> This PR fixes an issue when dragEvents occur in a secondary screen which 
> doesn't have the same height or (macOS) origin as the main screen.
> 
> The calculations for `xAbs, yAbs` are defined from the main screen macOS 
> absolute origin, an in order to flip the coordinates defined from the JavaFX 
> origin, the height from the main screen should be used, and not the height 
> from secondary screens, which was the case in 
> `GlassViewDelegate::sendJavaDndEvent`.
> 
> The PR doesn't include tests because it needs a complex setup (two monitors), 
> but the change has been tested on macOS 15.3.2 with the test case from 
> https://bugs.openjdk.org/browse/JDK-8353548, using two monitors (built-in 
> Retina and external 4K display), rearranging their layout (left-bottom-right 
> from main screen), and relative resolution.

Looks safe from documentation of [[NSScreen 
screens]](https://developer.apple.com/documentation/appkit/nsscreen/screens?language=objc).
Tested with 2 external monitors, tried several combinations and samples. LGTM.

-------------

Marked as reviewed by arapte (Reviewer).

PR Review: https://git.openjdk.org/jfx/pull/1756#pullrequestreview-2738961436

Reply via email to