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

Disable Fix Vine Fall when Climb Everything is enabled #3439

Conversation

Malkierian
Copy link
Contributor

@Malkierian Malkierian commented Nov 24, 2023

The combo of the two ended up making entering crawlspaces completely impossible. Just Climb Everything made it difficult, but doing a neutral roll in front of it will still let you enable the Enter button. Enabling Fix Vine Fall at the same time would remove even that ability. This adds a check for Climb Everywhere being disabled in order for the Fix Vine Fall code to run.

Build Artifacts

@Malkierian
Copy link
Contributor Author

Fixes #3369.

Copy link
Contributor

@Archez Archez left a comment

Choose a reason for hiding this comment

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

My two cents, but since it's an enhancement conflicting with a cheat, I don't think it's worth disabling the fix if the cheat is on (specifically I mean disabling the check box from being toggled).

The user will not notice any difference if the cheat is on or off. Just silently prevent the fix code from running if the cheat is on is fine enough.

@Malkierian
Copy link
Contributor Author

Ah, yeah, that would make sense.

@briaguya-ai briaguya-ai merged commit 3cf9d65 into HarbourMasters:develop-macready Nov 26, 2023
8 checks passed
A-Green-Spoon pushed a commit to A-Green-Spoon/Shipwright that referenced this pull request Nov 30, 2023
…s#3439)

* Disable Fix Vine Fall when Climb Everything is enabled.

* Remove option disabling.
@Malkierian Malkierian deleted the fix-vine-fall-climb-everything branch January 2, 2024 05:49
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.

3 participants