Skip to content

feat: fixes the doors on the consulate closet#386

Draft
Luiluix wants to merge 1 commit intoglacier-modding:mainfrom
Luiluix:ConsulateClosetFix
Draft

feat: fixes the doors on the consulate closet#386
Luiluix wants to merge 1 commit intoglacier-modding:mainfrom
Luiluix:ConsulateClosetFix

Conversation

@Luiluix
Copy link
Contributor

@Luiluix Luiluix commented Mar 14, 2025

Fixes the doors on the consulate closet being reset in the wrong position after being opened once (this fix only affects the ones inside the consulate, but I actually noticed it on other closets using the same template)

@Luiluix Luiluix changed the title Fixes the doors on the consulate closet feat: Fixes the doors on the consulate closet Mar 14, 2025
@Luiluix Luiluix changed the title feat: Fixes the doors on the consulate closet feat: fixes the doors on the consulate closet Mar 14, 2025
Fixes the doors on the consulate closet being reset in the wrong position after being opened once (this fix only affects the ones inside the consulate, but I actually noticed it on other closets using the same template)
@Luiluix Luiluix force-pushed the ConsulateClosetFix branch from e449f0f to ae33a68 Compare March 14, 2025 17:44
@Ocean-Minnow
Copy link
Contributor

I've checked a few closets in the consulate with this patch installed, but it seems like the doors are still popping upwards slightly after climbing in and out. I've been mainly testing with the one upstairs, near Strandberg's office, if that matters.

@Luiluix
Copy link
Contributor Author

Luiluix commented Apr 6, 2025

I've also used this closet while doing this fix, it's possible it doesn't work as I remembered it,
needs more work done so it can work on all closets anyway,
to give info on the bug, it's been around since 2016 iirc and seems to be caused by incorrect values inside the curve used to animate the doors

@Ocean-Minnow Ocean-Minnow marked this pull request as draft May 30, 2025 14:12
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