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

Make the baseball diamond square and centered #38755

Merged
merged 2 commits into from
Mar 13, 2020

Conversation

wapcaplet
Copy link
Contributor

Summary

SUMMARY: Bugfixes "Make the baseball diamond square and centered"

Purpose of change

Fixes #38753

Describe the solution

  • Four submaps merged into a single map for easier editing in the future
  • Baseball diamond made into correct proportion, and centered in the field better
  • Original items and monster spawns preserved

Describe alternatives you've considered

Considered re-orienting the diamond to follow x/y paths instead of diagonals (since I think that would look better in most tilesets), and also considered embellishing the field with foul lines, spectator stands, and some more interesting terrain - but I would prefer to leave that to someone who actually cares about baseball.

Testing

Spawned some baseball fields with the debug menu and verified that it looks OK, and has expected items and monsters.

Additional context

image

Thanks @curstwist for the help on this one

No items or monsters; won't load for some reason, but using as a
starting point for next edits (which will be a lot easier with the whole
map in one place).
@curstwist
Copy link
Contributor

Looks much better!

@ZhilkinSerg ZhilkinSerg added <Bugfix> This is a fix for a bug (or closes open issue) [JSON] Changes (can be) made in JSON Map / Mapgen Overmap, Mapgen, Map extras, Map display labels Mar 13, 2020
@ZhilkinSerg ZhilkinSerg merged commit 2807a1e into CleverRaven:master Mar 13, 2020
@wapcaplet wapcaplet deleted the fix-baseball-diamond branch March 21, 2020 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bugfix> This is a fix for a bug (or closes open issue) [JSON] Changes (can be) made in JSON Map / Mapgen Overmap, Mapgen, Map extras, Map display
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Baseball diamond is rectangular
3 participants