When padding is set on a child element, a parent container with adaptWidthToChildren = true doesn’t factor in the child’s padding. Example PG Note also the commented out lines where you can workaround the problem by adding negative padding to the parent container, which is counter-intuitive. I as…

Ok, having looked at the code and performed some testings, I do now think the paddings should be taken into account in the computation :slight_smile:. PR: Fix width/height GUI container computation to take into account paddi… BabylonJS:master ← P…

Adding @Evgeni_Popov to the thread as he knows GUI a lot :slight_smile: or at least a lot more than me :slight_smile: Let me know if you can not have a look and I ll try to fix it, THX a ton !!!

That would be easy to change the computation and to take into account the padding of the children, but that would be a breaking change and maybe it was intended that way… It looks to me that the paddings are “inside” the control, meaning within the boundary of the width (for the left/right padding)…

Let s not disturb it in vacation :slight_smile: We should add the fix in if easy as it feels like a bug ? and I ll let him know when he comes back.

[image] sebavan: We should add the fix in if easy as it feels like a bug ? That’s the thing, I’m not sure it is a bug… From what I can tell, when you set a width + paddingLeft + paddingRight on a control, the width of the control is really “width”, it is not “width+paddingLeft+paddingRight”. …

I have changed the PG to remove the rectangle and directly add the text to the GUI texture. I have also highlighted the text so that we can see what is the bounding rectangle of the text: https://www.babylonjs-playground.com/#GL5SIM#31 If you comment the padding, you will see no change: so, the pr…

Thanks @sebavan and @Evgeni_Popov . That was a quick turnaround. Love this community! :relaxed:

GUI container.adaptWidthToChildren doesn't respect child horizontal padding

Bugs

sebavan June 22, 2020, 7:14pm 8

Merged it will be available in the next nightly, thanks a lot @Evgeni_Popov !!!

Topic		Replies	Views
adaptWidthToChildren on GUI.StackPanel only takes one child into account Bugs gui	3	379	February 11, 2021
Gui adaptWidthToChildren not working with adaptive scaling? Bugs gui	9	814	April 7, 2022
Any way to set a min width for GUI controls when using adaptWidthToChildren? Questions	4	406	July 13, 2023
Problem with GUI and control's padding Questions gui	4	408	November 25, 2020
Having the exact same padding in my buttons Questions	2	339	June 15, 2023

GUI container.adaptWidthToChildren doesn't respect child horizontal padding

Related topics