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

Feature Request / Offer: Space Age Decorative Platform Tiles #263

Open
EugeneKay opened this issue Dec 16, 2024 · 0 comments
Open

Feature Request / Offer: Space Age Decorative Platform Tiles #263

EugeneKay opened this issue Dec 16, 2024 · 0 comments

Comments

@EugeneKay
Copy link

Is your feature request related to a problem? Please describe.
Space Age added "space-platform-foundation" tiles, which have the property allows_being_covered = false,. This prevents Decorative tiles from being placed on top, such as Hazard Concrete around my Nuclear Reactor.

Describe the solution you'd like
Add "space-platform-foundation-decorative" tile type, with allows_being_covered = true. For Balance, give it a a higher weight value (250, vs default of 200), and reduce the Rocket Capacity to match. Add a Recipe for this Tile type which requires 1 Space-Platform-Foundation + 1 Stone brick. Will probably use the "Stone Brick" graphics, since I don't have the art skills to create my own.

Describe alternatives you've considered
Modify the "space-platform-foundation" tile to allows_being_covered = true directly. I don't like this because it would encourage placing Decorative tiles across the entire Platform.

Additional context
Version 2.0.26 Release Notes included : "Fixed incorrect space platform bounds and weight when space platform foundations were covered by other tiles." This inspired me to explore if Hazard Concrete is placeable in-game over Foundation (it is not); but it does work in the Scenario Editor (so it is possible), and with this Bug resolved there are no obvious Mechanics Cheesing that could occur.

This is a Feature Offer (not a Request). I have already begun working to develop this locally for my own usage. Before I submit a working Pull Request, I would like to ask if this is likely to be accepted, and if I am on the right track for implementation:

  • The logic to enable/disable should go in tiles.lua
  • A new DECT.ENABLED flag should be added, "space-platform-foundation" in config.lua, and a matching localization
  • Should I add a new Tile type as-proposed; or keep it simple and only modify the existing space-platform-foundation?
  • New recipe (if desired) would go into flooring.lua
  • Ensure that the new Recipe is unlocked aside standard space-platform-foundation

Are there any other pieces I should consider? Or is this a bad idea and I should forget it?

Thank you for reading!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant