On Wed, 3 May 2023 20:46:16 GMT, Andy Goryachev <ango...@openjdk.org> wrote:

> Not related to these changes specifically, but why does GridPane adds a 
> listener in the constructor
> 
> ```
> getChildren().addListener((Observable o) -> requestLayout());
> ```
> 
> and VBox/HBox do not?

I don't think it is necessary. If you actually look at the `children` list it 
already takes care about layouting stuff and more. So requesting it here for 
every change seems wrong. But this should be tested carefully.

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

PR Comment: https://git.openjdk.org/jfx/pull/1125#issuecomment-1533754981

Reply via email to