Skip to content

Comments

Winterize The Cove [DO NOT FULL MERGE]#223

Open
EtheoBoxxman wants to merge 6 commits intoCausticCove:mainfrom
EtheoVteran:winterify
Open

Winterize The Cove [DO NOT FULL MERGE]#223
EtheoBoxxman wants to merge 6 commits intoCausticCove:mainfrom
EtheoVteran:winterify

Conversation

@EtheoBoxxman
Copy link

About The Pull Request

Recently saw a GM snowify the town, but it felt kind of incomplete what with the rest of the map being warm and dry.
This PR just adds a bunch of snowy turf sprites (sprited by TheGrendel on discord) that are applied if the matching turf is initialized in an outdoor area. So far, it looks pretty complete!

This doesnt modify the map, just icon states for a lot of natural terrain and foliage. There should be no complications with any map changes.

Brief to-do:
Make this a permanent change by adding a true or false toggle to area/rogue/outdoors to allow for quick and easy toggling of this effect for, optimistically, future winter seasons. When I do this, I'll mark the caustic edits accordingly

Developer's checklist

  • I have modularized whatever changes I could.
  • I have marked the start and end of my edits outside the caustic modular folder (if applicable) for changes that I couldn't modularize.
  • It compiled locally, and I tested new features, or potential issues with related features, to the best of my abilities.
  • This change applies mainly to our server, and wouldn't be in a better place upstream / cannot wait until the weekly upstream update.

Testing Evidence

image image image

Why It's Good For The Game

Changelog

🆑
add: Added new mechanics or gameplay changes
add: Added more things
del: Removed old things
qol: made something easier to use
balance: rebalanced something
fix: fixed a few things
sound: added/modified/removed audio or sound effects
image: added/modified/removed some icons or images
map: added/modified/removed map content
spellcheck: fixed a few typos
code: changed some code
refactor: refactored some code
config: changed some config setting
admin: messed with admin stuff
server: something server ops should know
/:cl:

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants