Skip to content

Add mapgen descriptions #16237

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

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Add mapgen descriptions #16237

wants to merge 12 commits into from

Conversation

Xeno333
Copy link
Contributor

@Xeno333 Xeno333 commented Jun 7, 2025

Adds mapgen description when creating a world. Closes #1033

To do

Ready for Review.

How to test

Pull, compile, create world in desired game, observe descriptions.

@Zughy Zughy added Roadmap: supported by core dev PR not adhering to the roadmap, yet some core dev decided to take care of it @ Mapgen Feature ✨ PRs that add or enhance a feature UI/UX @ Mainmenu labels Jun 7, 2025
@sfan5 sfan5 self-requested a review June 17, 2025 20:00
retval = retval ..
"label[0,2;" .. fgettext("Mapgen") .. "]"..
"dropdown[0,2.5;6.3;dd_mapgen;" .. mglist .. ";" .. selindex .. "]"..
"textarea[0.5,3.2;6,2;;;" .. mapgens_descriptions[current_mg] .. "]"
Copy link
Collaborator

Choose a reason for hiding this comment

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

so the dropdown will disappear if for some reason there is no description, why?

Copy link
Contributor

@appgurueu appgurueu Jul 23, 2025

Choose a reason for hiding this comment

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

^ this is still unresolved?

fractal = fgettext("Wold with fractal structure."),
singlenode = fgettext("Empty world, use for lua defined mapgens."),
v6 = fgettext("Simple mapgen with few features, not recommended."),
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

nitpick: none of these are full sentences so there should be no full stop at the end

@sfan5 sfan5 added the Action / change needed Code still needs changes (PR) / more information requested (Issues) label Jun 20, 2025
flat = fgettext("Flat world terrain"),
fractal = fgettext("World with a fractal structure"),
singlenode = fgettext("Empty world, commonly used for Lua-defined mapgens"),
v6 = fgettext("Simple mapgen with few features"),
Copy link
Contributor

Choose a reason for hiding this comment

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

What "new" features? A few words would be good

Copy link
Contributor

@DragonWrangler1 DragonWrangler1 Jul 22, 2025

Choose a reason for hiding this comment

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

If you're refering to V6. It says few features.

@Zughy Zughy added Action / change needed Code still needs changes (PR) / more information requested (Issues) and removed Action / change needed Code still needs changes (PR) / more information requested (Issues) labels Jul 2, 2025
@Xeno333 Xeno333 requested a review from sfan5 July 3, 2025 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Action / change needed Code still needs changes (PR) / more information requested (Issues) Feature ✨ PRs that add or enhance a feature @ Mainmenu @ Mapgen Roadmap: supported by core dev PR not adhering to the roadmap, yet some core dev decided to take care of it UI/UX
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Mapgen selection needs info text
6 participants