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

Not necessarily issues, just suggestions from Steam comments #2

Open
toytanker opened this issue Oct 24, 2019 · 0 comments
Open

Not necessarily issues, just suggestions from Steam comments #2

toytanker opened this issue Oct 24, 2019 · 0 comments

Comments

@toytanker
Copy link

  • Unless there was specific intent on making iceblock market value 3.5, consider aligning to base blocks of 0.9

  • Ice mining ultimately inherits JobDriver_AffectFloor which skill is connected to construction. Absent of creating a mirrored class of JobDriver_AffectFloor which instead was connected to mining [for example JobDriver_AffectFloorIce], then the only recourse is to patch JobDriver_AffectFloor but then if this is overridden for mining then all floor jobs will be connected to construction. You may have intended digging ice to be a construction job, but if not, then a new class would need to be created.

  • Water freezing effect was tricky to disable. I ended up having to remove the entire IceMapComponent class because I couldn’t figure out what section of code to comment out to effectively disable it. This goes hand in hand with medium affordance and why I suspect it was removed as walls built on tiles that could change from solid to liquid would result in illogical behavior. Rainbeau Flambe in is limitless knowledge and skills coded a mod GUI feature allowing his permafrost to be enabled/disabled. I know many mods have GUI functionality but I suspect it’s a lot of C#.

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

No branches or pull requests

1 participant