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

Fixes hop's skirt #22950

Merged
merged 1 commit into from
Jan 9, 2025
Merged

Conversation

Scrambledeggs00
Copy link
Contributor

Document the changes in your pull request

For whatever reason, HoP's skirt had 'civilian' in its type path in command.dm, and this was not repeated in other parts of the code. This caused issues, such as it not showing in HoP's vendor and it causing issues with the turtle neck. This PR removes 'civilian' from its typepath, fixing the issues.

Why is this good for the game?

Fixes #22909

Testing

Skirt now shows in the vendor:
image

Turtleneck skirt has worn sprites:
image
image
image

Something that I noticed while testing this: sometimes, when switching to the turtleneck's alt for the first time, the uniform's worn sprite would turn invisible for a second before becoming the alt sprite. I do not know why it does this, but it does not stay invisible so I guess it isn't too much of a problem.

Changelog

🆑
bugfix: HoP's skirt now shows in their vendor and turtleneck skirt has worn sprites
/:cl:

@Yogbot-13 Yogbot-13 added the Fix This fixes an issue. Please link issues in fix PRs label Jan 8, 2025
@ToasterBiome ToasterBiome merged commit 5dc794c into yogstation13:master Jan 9, 2025
15 checks passed
Yogbot-13 added a commit that referenced this pull request Jan 9, 2025
@Scrambledeggs00 Scrambledeggs00 deleted the HoP-skirt-fix branch January 9, 2025 02:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix This fixes an issue. Please link issues in fix PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

HoP alt unifoms missing sprites
3 participants