RFR: 8313709: Wrong layout of a FlowPane in a BorderPane in a ScrollPane, if padding is too big [v2]
Jose Pereda
jpereda at openjdk.org
Tue Aug 8 23:44:58 UTC 2023
> So far, BorderPane does the calculation for the children min/pref width/height taken into account only the margin applied to them, if any, but not the total padding that could be applied as well to the BorderPane itself.
>
> However, this padding needs to be taken into account as well, and this PR modifies BorderPane to subtract its insets from its size while doing the children min/pref width/height calculations.
>
> A parameterized test has been included.
>
> It is a simplified version of the test case attached to https://bugs.openjdk.org/browse/JDK-8313709, but still shows how without this patch, two of the cases (padding with or without margin) fail, while pass with it.
Jose Pereda has updated the pull request incrementally with one additional commit since the last revision:
Migrate old tests to JUnit 5
-------------
Changes:
- all: https://git.openjdk.org/jfx/pull/1203/files
- new: https://git.openjdk.org/jfx/pull/1203/files/ce6c0dad..e0c79da5
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jfx&pr=1203&range=01
- incr: https://webrevs.openjdk.org/?repo=jfx&pr=1203&range=00-01
Stats: 9 lines in 1 file changed: 5 ins; 2 del; 2 mod
Patch: https://git.openjdk.org/jfx/pull/1203.diff
Fetch: git fetch https://git.openjdk.org/jfx.git pull/1203/head:pull/1203
PR: https://git.openjdk.org/jfx/pull/1203
More information about the openjfx-dev
mailing list