Skip to content

Disable Fix Vine Fall when Climb Everything is enabled#3439

Merged
briaguya0 merged 2 commits intoHarbourMasters:develop-macreadyfrom
Malkierian:fix-vine-fall-climb-everything
Nov 26, 2023
Merged

Disable Fix Vine Fall when Climb Everything is enabled#3439
briaguya0 merged 2 commits intoHarbourMasters:develop-macreadyfrom
Malkierian:fix-vine-fall-climb-everything

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.

@briaguya0 briaguya0 merged commit 3cf9d65 into HarbourMasters:develop-macready Nov 26, 2023
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