[Rseding] [1.1.32] Buggy base game GUI when using large relative GUIs

[Xorimuth]

I can work around this by setting `maximal_height = 930` on the outermost frame, but I'm pretty sure the behaviour is unwanted.

To reproduce, load the save with the attached mod (not the same as the version on the mod portal), and click on the spidertron. You can use the patrol remote (click on the ground) to expand the height of the UI and the 'cross' or 'reset' buttons to shrink it. When the height gets too large, the bottom of the GUI is just blank and unresponsive, and the top of the relative GUI pops out above the top of the main GUI.

This issue only occurs on larger UI scales, but it is present at 1080p 100% scale (see screenshot). It doesn't occur on smaller-than-recommend scalings, or the default 175% scale on my 3072 × 1920 display (which is 16:10, so more space anyway). Using the `maximal_height = 930` workaround means that on the high-DPI, 16:10 display it doesn't use the full height like it used to.

Screenshot 2021-05-01 at 23.41.16.png (291.11 KiB) Viewed 4289 times

[Xorimuth]

My hardcoded maximal_height workaround doesn't even work when another mod is taking up some of the relative space: https://mods.factorio.com/mod/Spidertro ... 5c11dd15bf

[Xorimuth]

This does not cause any issues with https://mods.factorio.com/mod/production-potential (v1.2.0) which has a relative GUI, a scroll box, but no `maximal_height`, implying that it is an issue specific to entity GUIs.

[Rseding91]

Thinking about this more; I'm not sure how this should work in the end. Should the contents of the relative GUIs get squashed down to the height/width of the base-game GUI. Or should the base-game GUI stretch to match the relative GUI?... I'm not sure. I don't think either is always "correct" which is where I'm currently stuck.

[Xorimuth]

Thinking about this more; I'm not sure how this should work in the end. Should the contents of the relative GUIs get squashed down to the height/width of the base-game GUI. Or should the base-game GUI stretch to match the relative GUI?... I'm not sure. I don't think either is always "correct" which is where I'm currently stuck.

Well, I personally prefer the current behaviour of stretching the base game GUI to match the relative GUI (up to a maximum height, ofc). Particularly with multiple mods adding relative GUIs, space would get quite limited otherwise. If a specific mod wants to ‘turn off’ this behaviour then they can do it by limiting the height of their outer frame.

Vanilla inventory GUIs already expand along with their inventory size.

On the other hand, expanding the GUI dynamically leads to viewtopic.php?f=48&t=98136&p=545580, which you didn’t want to fix…


However, I’m not sure if this debate has much to do with the reported bug - the base-game GUI already stretches to match the relative GUI, just not always correctly.

[Rseding91]

This is now fixed for the next release thanks to tburrows13.