On Fri, 21 Apr 2023 13:52:55 GMT, Karthik P K <k...@openjdk.org> wrote:

>> Usage of `getBounds()` method instead `getVisualBounds()` was giving 
>> unreliable screen bound values and color value was read very close to the 
>> edge of the window.
>> 
>> Updated the code to use `getVisualBounds()` instead of `getBounds()` and 
>> moved the coordinates inside the window from where the color value is read.
>> 
>> Ran the tests individually and along with all system tests in following 
>> systems. No failure found after the fix.
>> Mac M1 with Ventura 13.3
>> Window 11
>
> Karthik P K has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Updated visual bounds value for x y coordinates to pick color and changed 
> offset from edge to 10

Addressed changes suggested above. Please re-review.

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

PR Comment: https://git.openjdk.org/jfx/pull/1103#issuecomment-1519395368

Reply via email to