On Thu, 11 May 2023 16:24:20 GMT, Kevin Rushforth <k...@openjdk.org> wrote:

>> modules/javafx.graphics/src/main/native-glass/mac/GlassDraggingSource.m line 
>> 44:
>> 
>>> 42:     return self->dragOperation;
>>> 43: }
>>> 44: 
>> 
>> `sourceOperationMaskForDraggingContext` replaces the deprecated 
>> `draggingSourceOperationMaskForLocal:` over in GlassViewDelegate.m. There's 
>> some logic there that needs to be copied over here. Basically the Apple 
>> documentation on how the Cmd key filters the set of available operations 
>> doesn't match reality and we have to compensate for that.
>
> This sounds like it could be the reason for the functional regression in 
> handling the CMD modifier while dragging.

Good catch, I'll take a look into that

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1089#discussion_r1192270396

Reply via email to