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

[Testmerge ONLY!!] Fixes chest-mounted equipment from displaying on top of clothes when they shouldn't. #3282

Merged
merged 1 commit into from
Oct 5, 2023

Conversation

KathrinBailey
Copy link
Contributor

@KathrinBailey KathrinBailey commented Oct 5, 2023

About The Pull Request

Reverts a change of #448 that broke a lot of clothing items.
What it did was make it so the game stops looking at what parts of your body your clothes cover up, and instead look at a new variable that should do the same thing, but still allow you 'female body armor™'. This new variable, however, is not set on a lot of clothing items, thus breaking them.

It this works, we can re-implant the changes of #448 as a simple "show-it-anyway" override. So instead, we still check at what parts are covered up, but we also check for this override that'll be set to no by default.

Pre-Merge Checklist

  • You tested this on a local server.
  • This code did not runtime during testing. This code did not runtime during testing to anything related.
  • You documented all of your changes.

Changelog

🆑
fix: Fixes a lot of clothing items to only show your off your enterprise resource planning assets when they should be.
/:cl:

@Tk420634 Tk420634 merged commit a6fb2e2 into ARF-SS13:master Oct 5, 2023
12 checks passed
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

Successfully merging this pull request may close these issues.

2 participants