RFR: 8365635: Add MOUSE_DRAG_DONE event type [v2]
Kevin Rushforth
kcr at openjdk.org
Fri Oct 10 19:40:22 UTC 2025
On Sat, 30 Aug 2025 15:46:29 GMT, Nir Lisker <nlisker at openjdk.org> wrote:
>> Adds `MOUSE_DRAG_DONE` event type to `MouseDragEvent` and appropriate handlers.
>
> Nir Lisker has updated the pull request incrementally with one additional commit since the last revision:
>
> Missing @since
This looks like a good addition.
I left a comment recommending to revert the markdown-style comments. There doesn't seem to be a good reason to use them for the two new properties and new static final constant.
modules/javafx.graphics/src/main/java/javafx/scene/Node.java line 7714:
> 7712: }
> 7713:
> 7714: public final EventHandler<? super MouseDragEvent> getOnMouseDragDone() {
We normally put the property methods that will inherit the docs below the method (or private property as the case may be) that has the javadoc comments.
I recommend to move these two methods below the `onMouseDragDoneProperty` method.
-------------
PR Review: https://git.openjdk.org/jfx/pull/1873#pullrequestreview-3325642083
PR Review Comment: https://git.openjdk.org/jfx/pull/1873#discussion_r2421880414
More information about the openjfx-dev
mailing list