Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Map generator upgrades #4514

Merged
merged 6 commits into from
Jun 12, 2023
Merged

Conversation

NickAragua
Copy link
Member

Kind of a large pull request, but it adds a lot of stuff to the random map generator.

-Tundra
-Ultra Rough
-Ultra Rubble (and greater regular rubble variety)
-Ultra Woods
-Jungle (standard/heavy/ultra)

Copy link
Member

@SJuliez SJuliez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one comment

@@ -769,6 +844,10 @@ public static void makeCompactGrid(Container parent,
y = Spring.sum(y, Spring.sum(height, Spring.constant(yPad)));
}

// for some reason the bottom border runs over the bottom textboxes in
// some of these, so we add an experimentally established padding to the bottom
y = Spring.sum(y, Spring.constant(13));
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The reason is the call to adjustDialog to adapt it to the gui scale, which doesn't relayout the dialog. I've fought with this too and can't say I won. A slightly less hacky way to deal with it is to add a pack() in RandomMapDialog after l.476, directly after the call to adaptToGUIScale().

@NickAragua NickAragua merged commit 8290ff2 into MegaMek:master Jun 12, 2023
@NickAragua NickAragua deleted the map_generator_upgrades branch June 12, 2023 05:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants