[PATCH] 8161282: FXCanvas does not forward horizontal mouse scroll events to the embedded scene
Kevin Rushforth
kevin.rushforth at oracle.com
Wed Sep 7 00:08:28 UTC 2016
Approved, assuming the changes to the wild-import are done by Alexander
along with a correctly formatted changeset commit message.
-- Kevin
Kevin Rushforth wrote:
> Uploaded the new version.
>
> Alexander Zvegintsev will sponsor this patch, and I will review it.
>
> My one quick comment -- in addition to the wild-card imports which
> Alexander Z caught, and said he would take care of -- is that the
> format of the commit message is not quite right. See the following
> page for formatting rules on the changeset commit message:
>
> http://openjdk.java.net/guide/producingChangeset.html
>
> Thanks.
>
> -- Kevin
>
>
>
> Alexander Nyssen wrote:
>> You might even take the one I attached. I just recognized I still had
>> some unused imports in the manual test case (I am still not familiar
>> with IntelliJ).
>>
>> Regards,
>> Alexander
>>
>>
>> ------------------------------------------------------------------------
>>
>>
>>> Am 15.08.2016 um 19:26 schrieb Kevin Rushforth
>>> <kevin.rushforth at oracle.com <mailto:kevin.rushforth at oracle.com>>:
>>>
>>> OK, I'll upload this revised version of the patch today.
>>>
>>> -- Kevin
>>>
>>>
>>> Alexander Nyssen wrote:
>>>> Hi Kevin,
>>>>
>>>> please consider the following updated patch instead, which contains
>>>> an additional null-check.
>>>>
>>>> Regards
>>>> Alexander
>>>>
>>>>
>>>> ------------------------------------------------------------------------
>>>>
>>>>
>>>>
>>>>> Am 12.08.2016 um 16:04 schrieb Alexander Nyssen
>>>>> <alexander at nyssen.org <mailto:alexander at nyssen.org>>:
>>>>>
>>>>> Hi Kevin,
>>>>>
>>>>> attached please find an initial patch for
>>>>> https://bugs.openjdk.java.net/browse/JDK-8161282.
>>>>>
>>>>> The patch is not as minimal as I had hoped, as the
>>>>> EmbeddedSceneInterface had to be changed to differentiate between
>>>>> mouse and scroll events (while up to now, scroll events are
>>>>> handled as mouse events), but for me this seemed necessary to fix
>>>>> this issue properly. As a result, JFXPanel had to be adjusted as
>>>>> well to comply to the changes in the EmbeddedSceneInterface, while
>>>>> its behavior should not have changed.
>>>>>
>>>>> As horizontal mouse events cannot be synthesized via
>>>>> Display.post(Event) yet (an open issue for SWT), I did not add an
>>>>> automated test, but instead added a manual one
>>>>> (FXCanvasMouseWheelEventsTest). Therefore, this patch does not
>>>>> depend on the patch I provided earlier for JDK-8160325.
>>>>>
>>>>> Best Regards,
>>>>> Alexander
>>>>>
>>>>> <JDK-8161282_12_08_16.patch>
>>>>>
>>>>>
>>>>
>>>> =
>>
>> =
More information about the openjfx-dev
mailing list