From 53846b89dd066d481297fc91e16332287c937879 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Nicola=CC=81s=20Luis=20Gonza=CC=81lez=20Paz?= Date: Wed, 6 May 2026 10:39:10 -0300 Subject: [PATCH] fix: use flex-basis:auto to prevent container layout collapse Replaces flex:1 1 0px with flex:1 1 auto in both the horizontal and vertical inner containers so TwinColGrid renders at its natural height when no explicit height is set. Corrects the no-op min-width:0px to min-height:0px in the vertical container. Close #201 --- .../flowingcode/vaadin/addons/twincolgrid/TwinColGrid.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/main/java/com/flowingcode/vaadin/addons/twincolgrid/TwinColGrid.java b/src/main/java/com/flowingcode/vaadin/addons/twincolgrid/TwinColGrid.java index e31626f..916477d 100644 --- a/src/main/java/com/flowingcode/vaadin/addons/twincolgrid/TwinColGrid.java +++ b/src/main/java/com/flowingcode/vaadin/addons/twincolgrid/TwinColGrid.java @@ -355,7 +355,7 @@ private HorizontalLayout createHorizontalContainer(boolean reverse) { } hl.setId(getLayoutId()); hl.getElement().getStyle().set("min-height", "0px"); - hl.getElement().getStyle().set("flex", "1 1 0px"); + hl.getElement().getStyle().set("flex", "1 1 auto"); hl.setMargin(false); hl.setWidthFull(); return hl; @@ -370,8 +370,8 @@ private VerticalLayout createVerticalContainer(boolean reverse) { vl = new VerticalLayout(available.layout, buttonContainer, selection.layout); } vl.setId(getLayoutId()); - vl.getElement().getStyle().set("min-width", "0px"); - vl.getElement().getStyle().set("flex", "1 1 0px"); + vl.getElement().getStyle().set("min-height", "0px"); + vl.getElement().getStyle().set("flex", "1 1 auto"); vl.setMargin(false); vl.setPadding(false); vl.setHeightFull();