Re: RFR: 8319147: Add regression test for JDK-8317836 [v2]

2023-11-06 Thread Prasanta Sadhukhan
On Mon, 6 Nov 2023 06:06:35 GMT, Prasanta Sadhukhan wrote: >> A manual regression test for JFXPanel orientation fix in >> [JDK-8317836](https://bugs.openjdk.org/browse/JDK-8317836) is added > > Prasanta Sadhukhan has updated the pull request incrementally with two > additional commits since

Re: RFR: 8319147: Add regression test for JDK-8317836 [v2]

2023-11-06 Thread Andy Goryachev
On Mon, 6 Nov 2023 19:30:28 GMT, Kevin Rushforth wrote: > Can you file an enhancement to consider this for the next test sprint? Created [JDK-8319555](https://bugs.openjdk.org/browse/JDK-8319555) - PR Comment: https://git.openjdk.org/jfx/pull/1276#issuecomment-1796178835

Re: RFR: 8319147: Add regression test for JDK-8317836 [v2]

2023-11-06 Thread Kevin Rushforth
On Mon, 6 Nov 2023 18:43:48 GMT, Andy Goryachev wrote: > I was waiting for changes to set the window position, but can re-approve if > necessary. His earlier comments suggested that he felt it wasn't needed for this PR; with the adjusted window sizes, the windows no longer overlap. > A side

Re: RFR: 8319147: Add regression test for JDK-8317836 [v2]

2023-11-06 Thread Andy Goryachev
On Mon, 6 Nov 2023 06:06:35 GMT, Prasanta Sadhukhan wrote: >> A manual regression test for JFXPanel orientation fix in >> [JDK-8317836](https://bugs.openjdk.org/browse/JDK-8317836) is added > > Prasanta Sadhukhan has updated the pull request incrementally with two > additional commits since

Re: RFR: 8319147: Add regression test for JDK-8317836 [v2]

2023-11-06 Thread Kevin Rushforth
On Mon, 6 Nov 2023 06:06:35 GMT, Prasanta Sadhukhan wrote: >> A manual regression test for JFXPanel orientation fix in >> [JDK-8317836](https://bugs.openjdk.org/browse/JDK-8317836) is added > > Prasanta Sadhukhan has updated the pull request incrementally with two > additional commits since

Re: RFR: 8319147: Add regression test for JDK-8317836 [v2]

2023-11-06 Thread Kevin Rushforth
On Mon, 6 Nov 2023 06:06:55 GMT, Prasanta Sadhukhan wrote: > Is there any in-house or public fx documentation that I can peruse (guess we > dont have similar to > https://docs.oracle.com/en/java/javase/21/docs/api/index.html) Andy already answered your specific question. As for where the

Re: RFR: 8319147: Add regression test for JDK-8317836 [v2]

2023-11-06 Thread Andy Goryachev
On Mon, 6 Nov 2023 06:06:55 GMT, Prasanta Sadhukhan wrote: > can you please specify how to do it for my future reference as I dont see > Scene accepting x,y params Use Stage (Window) setX(), setY() before show(). - PR Comment:

Re: RFR: 8319147: Add regression test for JDK-8317836 [v2]

2023-11-05 Thread Prasanta Sadhukhan
On Sat, 4 Nov 2023 13:48:48 GMT, Kevin Rushforth wrote: > make the test window smaller so they don't overlap Done > Maybe set the X and Y of the instruction window Although it is probably not required now, but can you please specify how to do it for my future reference as I dont see Scene

Re: RFR: 8319147: Add regression test for JDK-8317836 [v2]

2023-11-05 Thread Prasanta Sadhukhan
> A manual regression test for JFXPanel orientation fix in > [JDK-8317836](https://bugs.openjdk.org/browse/JDK-8317836) is added Prasanta Sadhukhan has updated the pull request incrementally with two additional commits since the last revision: - Formatting - Modify window size -

Re: RFR: 8319147: Add regression test for JDK-8317836

2023-11-04 Thread Kevin Rushforth
On Thu, 2 Nov 2023 06:04:23 GMT, Prasanta Sadhukhan wrote: > A manual regression test for JFXPanel orientation fix in > [JDK-8317836](https://bugs.openjdk.org/browse/JDK-8317836) is added I verified that this test passes with the fix for JDK-8317836 and fails with that fix locally reverted.

Re: RFR: 8319147: Add regression test for JDK-8317836

2023-11-02 Thread Andy Goryachev
On Thu, 2 Nov 2023 06:04:23 GMT, Prasanta Sadhukhan wrote: > A manual regression test for JFXPanel orientation fix in > [JDK-8317836](https://bugs.openjdk.org/browse/JDK-8317836) is added looks good, works as expected with the latest master. - Marked as reviewed by angorya

RFR: 8319147: Add regression test for JDK-8317836

2023-11-02 Thread Prasanta Sadhukhan
A manual regression test for JFXPanel orientation fix in [JDK-8317836](https://bugs.openjdk.org/browse/JDK-8317836) is added - Commit messages: - Test fix - Test fix Changes: https://git.openjdk.org/jfx/pull/1276/files Webrev: https://webrevs.openjdk.org/?repo=jfx=1276=00