On Thu, 9 Jul 2026 19:04:02 GMT, Kevin Rushforth <[email protected]> wrote:
>> Andy Goryachev has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> demos
>
> modules/jfx.incubator.richtext/src/main/java/jfx/incubator/scene/control/richtext/RichTextArea.java
> line 550:
>
>> 548: * @since 27
>> 549: */
>> 550: public final void setDropTarget(double screenX, double screenY) {
>
> Should there be a variant of this method that takes a `TextPos`?
Not really, the caller of this method is a mouse event handler, and it's better
to use screen coordinates, similar to `getTextPosition()`
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/2196#discussion_r3555177568
