On Sat, 5 Sep 2026 09:18:25 GMT, John Hendrikx <[email protected]> wrote:
>> Michael Strauß has updated the pull request with a new target base due to a
>> merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request contains five additional
>> commits since the last revision:
>>
>> - review comments
>> - Merge branch 'master' into fixes/snapping-region
>> - more render scales
>> - fixes
>> - tests
>
> modules/javafx.graphics/src/main/java/javafx/scene/layout/Region.java line
> 2944:
>
>> 2942: } else {
>> 2943: snappedX = rawX;
>> 2944: snappedY = rawY;
>
> Is this inconsistent with what the `snapped` prefix means?
Yes in a way... I've changed it to always invoke the snapping function, like in
all the other places.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/2278#discussion_r3940474204
