Skip to content

Fix some IPC and loadout prefs #5823

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

Merged
merged 4 commits into from
Mar 8, 2025
Merged

Conversation

Absolucy
Copy link
Member

@Absolucy Absolucy commented Mar 6, 2025

About The Pull Request

This place is not a place of honor...
No highly esteemed deed is commemorated here... nothing valued is here.
What is here was dangerous and repulsive to us. This message is a warning about danger.

i fixed some shitcode with prefs, especially IPC prefs - which should fix some performance issues. and made changing the IPC chassis actually update the character preview in the process.

Why It's Good For The Game

less lag, AND making stuff work better. win-win. code's still yuck tho.

Changelog

🆑
fix: Fixed some performance issues related to character customization.
fix: Changing the IPC chassis in preferences now properly updates the preview.
/:cl:

@Absolucy Absolucy added Process: do not merge don't merge this ffs Code: Fix fix da bug Code: Performance / Optimization the number going down makes me happy :3 labels Mar 6, 2025
@@ -61,8 +61,13 @@
return values

/datum/preference/choiced/ipc_chassis/apply_to_human(mob/living/carbon/human/target, value)
target.dna.features["ipc_chassis"] = value
target.regenerate_icons()
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the worst part is that this didn't even WORK

@Absolucy Absolucy merged commit 5cb6146 into Monkestation:master Mar 8, 2025
25 checks passed
@Absolucy Absolucy deleted the ipc-prefs-fix branch March 8, 2025 04:12
github-actions bot added a commit that referenced this pull request Mar 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code: Fix fix da bug Code: Performance / Optimization the number going down makes me happy :3 Process: do not merge don't merge this ffs Process: should testmerge PR should be testmerged first
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant