Skip to content

Conversation

@maddie480
Copy link
Member

Closes #20.

This time, I didn't make it extend JumpThru, since those have little in common (... the orientation to begin with: jump thrus extend on their width).

Most of how it works is an IL patch applied on multiple methods, that adds collide checks against sideways jumpthrus everywhere the game checks for solids. This way, Madeline can climb on and walljump off sideways jumpthrus.

(Note to self: the Julia part seems to need a tab to space pass.)

Copy link
Member

@0x0ade 0x0ade left a comment

Choose a reason for hiding this comment

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

C# portion looks mostly good to me, for the same reasons as the upside-down jumpthru 👍

Awaiting @Cruor's review of the Ahorn plugin.

}



Copy link
Member

Choose a reason for hiding this comment

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

What's up with the whitespace here?

Copy link
Member Author

Choose a reason for hiding this comment

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

That was to separate hook code from the entity code 😅 I think I should drop a comment in here instead.

@0x0ade 0x0ade requested a review from Cruor January 21, 2020 23:56
Copy link
Contributor

@Cruor Cruor left a comment

Choose a reason for hiding this comment

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

allowLeftToRight sounds a bit strange to me, makes it sound like it is normally fully solid, but you can set the flag to go through it from left -> right. Doesn't the base game just do left or right for these kinds of things?
Looks good to me otherwise though.

@maddie480
Copy link
Member Author

I'll double-check what the base game does for orientation settings. Would "right" mean the upper side of the jumpthru is on the right side, so you can go left to right? 🤔

@maddie480
Copy link
Member Author

Attribute renamed on the Ahorn plugin side, comment added instead of that awkward gap on the C# side.

@Cruor Cruor merged commit 238613b into master Jan 22, 2020
@maddie480 maddie480 deleted the sideways_jumpthrus branch January 23, 2020 17:44
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.

Multi-directional jumpthrus

4 participants