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

White Bandana - Small addition #10425

Merged
merged 1 commit into from
Jan 14, 2024
Merged

Conversation

Nakalon
Copy link
Contributor

@Nakalon Nakalon commented Jan 5, 2024

About The Pull Request

Before this PR, when you tried to make a white bandana with cloth you'd make a botany bandana. Why? I don't know.

Now the white bandana exists.

I also made the white bandana the base for all bandanas, this is so to make adding more colours in the future easier and prepare for the Gagsification of bananas (my evil plan).

I plan to do this to most articles of clothing, I already made a white version of some items in #10371

This also allows players to dye them in whatever colour they want in game with a spray can, so win win.

I also removed some unnecessary bandana icon states that were duplicates (head sprites in mask file).

Why It's Good For The Game

I don't think there's anything I should mention here that I have nor already mentioned. I guess Consistency and player freedom.

Testing Photographs and Procedure

Screenshots&Videos

image

White bandana

image

Painting the bandana

Changelog

🆑 Pink Solene
add: added a white bandana
imageadd: added a white bandana sprite
imagedel: removed duplicate bandana sprites
tweak: creating a white bandana out of cloth actually creates a white bandana instead of a botany bandana now
code: changed the parent of bandanas to be a white bandana and made a bandana/botany variant for the botany one, then made the necessary changes to ensure botany bandanas appear where they are expected.
/:cl:

Adds white bandana
@itsmeow itsmeow added this pull request to the merge queue Jan 14, 2024
Merged via the queue into BeeStation:master with commit 17e9722 Jan 14, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants