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

Unable to change townnames in Scenario editor after pr8566 #8686

Open
EratoNysiad opened this issue Feb 17, 2021 · 3 comments
Open

Unable to change townnames in Scenario editor after pr8566 #8686

EratoNysiad opened this issue Feb 17, 2021 · 3 comments
Labels
enhancement Issue would be a good enhancement; we accept Pull Requests! good first issue Good for newcomers

Comments

@EratoNysiad
Copy link
Contributor

Version of OpenTTD

pr8680, related to change in pr8566

Expected result

When placing towns in the scenario editor, you'd want to be able to change what townnames you're using. This used to be in the Game Options, but it's been moved to the terrain generation UI.

Actual result

It's no longer in the game options.

Steps to reproduce

Open OpenTTD in the mentioned version https://preview.openttd.org/pr8680/
Open up the scenario editor
Open up the town placement menu

@2TallTyler
Copy link
Member

Scenario Editor uses the standard map generation window when generating a new random map or opening a heightmap. The only way to bypass this is if you choose the scenario type "Flat land" when creating a new scenario.

flatland

If you landscape the flat land to your liking and then realize the town names aren't set properly, you might assume you're stuck — although you can save the world as a heightmap and then open it again to get the full generation window.

I don't know how much of an edge case this would be. Sculpting from Flat land is just tedious and I've always built scenarios from heightmaps.

Also note that even when town names were in Options, the moment you placed your first town, your selected town name set was locked in. So there is not even a risk of a player losing hand-placed town or industries (which can't be placed without a parent town) if they have to save as a heightmap and reload. But this certainly isn't an obvious workaround.

I see two potential solutions:

  • Add a town name selector to the Town Generation toolbar in SE
  • Add a town name selector to the Flat land type landscape (not sure exactly how this would look) and always open the Scenario Type dialog before loading SE from the main menu to select Flat map or Random land.

Thoughts?

@EratoNysiad
Copy link
Contributor Author

The first solution would quite quickly resolve the issue.

The second solution has an additional problem. If that's the new default starting window for the Scenario editor, then you'd also want a load scenario/heightmap button in the menu.

@frosch123
Copy link
Member

The town-name generator is actually stored per town. In scenario-editor it is possible to use different name generators for different towns.
As such, adding it to the town-generation-toolbar, and possibly to the town-rename window makes most sense.

@2TallTyler 2TallTyler added enhancement Issue would be a good enhancement; we accept Pull Requests! good first issue Good for newcomers labels Oct 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Issue would be a good enhancement; we accept Pull Requests! good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants