search

FAForever / downlords-faf-client

Official client for Forged Alliance Forever
https://faforever.com
MIT License
196 stars 121 forks source link

Change generator UI for the new map generator version #3187

Closed BlackYps closed 3 months ago

BlackYps commented 4 months ago

We are changing the parameters for the map generator here: https://github.com/FAForever/Neroxis-Map-Generator/pull/396

grafik

BlackYps commented 4 months ago

Do you have an idea why there is excessive space between the heading and the spawn number spinner? It's not because of the hidden command line field, because even if I enable that, there is still too much space under the heading.

Sheikah45 commented 4 months ago

I never was exactly sure, something with the gridpane I think

BlackYps commented 4 months ago

There is a weird interaction going on. When the heading text is the longest of all elements in the column, then it needs significantly more vertical space. This can easily be observed by shortening the text. I have no idea why this happens though