Re: RFR: JDK-8315569: Tests for the contract of SkinBase.layoutChildren(..) [v3]

2023-10-12 Thread Andy Goryachev
On Fri, 22 Sep 2023 10:11:12 GMT, Marius Hanl wrote: >> This PR adds a test that verifies the `SkinBase.layoutChildren(..)` method >> with different scales. >> While not explicitly documented, this method will receive the snapped and >> correctly calculated x, y, width and height values, so tha

Re: RFR: JDK-8315569: Tests for the contract of SkinBase.layoutChildren(..) [v3]

2023-10-12 Thread Marius Hanl
On Fri, 22 Sep 2023 17:14:30 GMT, Andy Goryachev wrote: >> Marius Hanl has updated the pull request incrementally with one additional >> commit since the last revision: >> >> JDK-8315569: Improve test name and add 2.25 scale > > modules/javafx.controls/src/test/java/test/javafx/scene/control/

Re: RFR: JDK-8315569: Tests for the contract of SkinBase.layoutChildren(..) [v3]

2023-09-22 Thread Andy Goryachev
On Fri, 22 Sep 2023 10:11:12 GMT, Marius Hanl wrote: >> This PR adds a test that verifies the `SkinBase.layoutChildren(..)` method >> with different scales. >> While not explicitly documented, this method will receive the snapped and >> correctly calculated x, y, width and height values, so tha

Re: RFR: JDK-8315569: Tests for the contract of SkinBase.layoutChildren(..) [v3]

2023-09-22 Thread Marius Hanl
> This PR adds a test that verifies the `SkinBase.layoutChildren(..)` method > with different scales. > While not explicitly documented, this method will receive the snapped and > correctly calculated x, y, width and height values, so that children of the > Control can be layouted correctly with

Re: RFR: JDK-8315569: Tests for the contract of SkinBase.layoutChildren(..) [v2]

2023-09-15 Thread Marius Hanl
On Tue, 12 Sep 2023 22:48:26 GMT, Andy Goryachev wrote: >> Do you a better idea? I guess it would make sense to rename it to something >> more generic. Maybe ControlLayoutTest? > > ControlLayoutChildrenContractTest perhaps, if that's the only thing being > tested here (that is, if you don't pla

Re: RFR: JDK-8315569: Tests for the contract of SkinBase.layoutChildren(..) [v2]

2023-09-12 Thread Andy Goryachev
On Tue, 12 Sep 2023 07:07:27 GMT, Marius Hanl wrote: >> modules/javafx.controls/src/test/java/test/javafx/scene/control/ControlContractTest.java >> line 44: >> >>> 42: import static org.junit.jupiter.api.Assertions.assertTrue; >>> 43: >>> 44: class ControlContractTest { >> >> Do you plan to a

Re: RFR: JDK-8315569: Tests for the contract of SkinBase.layoutChildren(..) [v2]

2023-09-12 Thread Marius Hanl
On Thu, 7 Sep 2023 22:04:46 GMT, Andy Goryachev wrote: >> Marius Hanl has updated the pull request incrementally with one additional >> commit since the last revision: >> >> JDK-8315569: Set a min size > > modules/javafx.controls/src/test/java/test/javafx/scene/control/ControlContractTest.jav

Re: RFR: JDK-8315569: Tests for the contract of SkinBase.layoutChildren(..) [v2]

2023-09-07 Thread Andy Goryachev
On Sat, 2 Sep 2023 13:09:14 GMT, Marius Hanl wrote: >> This PR adds a test that verifies the `SkinBase.layoutChildren(..)` method >> with different scales. >> While not explicitly documented, this method will receive the snapped and >> correctly calculated x, y, width and height values, so that

Re: RFR: JDK-8315569: Tests for the contract of SkinBase.layoutChildren(..) [v2]

2023-09-06 Thread Kevin Rushforth
On Sat, 2 Sep 2023 13:09:14 GMT, Marius Hanl wrote: >> This PR adds a test that verifies the `SkinBase.layoutChildren(..)` method >> with different scales. >> While not explicitly documented, this method will receive the snapped and >> correctly calculated x, y, width and height values, so that

Re: RFR: JDK-8315569: Tests for the contract of SkinBase.layoutChildren(..) [v2]

2023-09-02 Thread Marius Hanl
> This PR adds a test that verifies the `SkinBase.layoutChildren(..)` method > with different scales. > While not explicitly documented, this method will receive the snapped and > correctly calculated x, y, width and height values, so that children of the > Control can be layouted correctly with

RFR: JDK-8315569: Tests for the contract of SkinBase.layoutChildren(..)

2023-09-02 Thread Marius Hanl
This PR adds a test that verifies the `SkinBase.layoutChildren(..)` method with different scales. While not explicitly documented, this method will receive the snapped and correctly calculated x, y, width and height values, so that children of the Control can be layouted correctly without requir