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

Add persistent bunny mask enhancement #537

Merged
merged 1 commit into from
Aug 18, 2024

Conversation

garrettjoecox
Copy link
Contributor

@garrettjoecox garrettjoecox commented May 27, 2024

This enhancement allows you to always keep the bunny hood on, in combination with other masks and even other transformations.

The user experience of using this should be intuitive, it allows you to equip/un-equip the mask like normal, and lets you press "A" on the mask in kaleido to equip/unequip it

I intend to also add an option for doing the same thing with the Great fairy mask, but want to keep this PR separate from that.

Build Artifacts

@garrettjoecox garrettjoecox added the enhancement New feature or request label May 27, 2024
@garrettjoecox garrettjoecox force-pushed the persistant-masks branch 3 times, most recently from a102d4b to 8d5b3ca Compare May 29, 2024 01:48
@garrettjoecox
Copy link
Contributor Author

Now using VB for restrictions override, and fixed an issue where the bunny hood would stay on if you didn't have it obtained

@garrettjoecox garrettjoecox force-pushed the persistant-masks branch 6 times, most recently from 5353fb7 to 843a730 Compare June 3, 2024 02:58
Copy link
Contributor

@inspectredc inspectredc left a comment

Choose a reason for hiding this comment

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

I think the hooks that are registered in RegisterPersistentMasks could do with some comments explaining what each one is achieving above it. Otherwise just some nitpicks/small things

mm/2s2h/Enhancements/GameInteractor/GameInteractor.h Outdated Show resolved Hide resolved
mm/2s2h/Enhancements/Masks/PersistentMasks.cpp Outdated Show resolved Hide resolved
mm/2s2h/Enhancements/Masks/PersistentMasks.cpp Outdated Show resolved Hide resolved
mm/2s2h/Enhancements/Masks/PersistentMasks.cpp Outdated Show resolved Hide resolved
@garrettjoecox
Copy link
Contributor Author

I think the hooks that are registered in RegisterPersistentMasks could do with some comments explaining what each one is achieving above it. Otherwise just some nitpicks/small things

Comments added!

@garrettjoecox garrettjoecox mentioned this pull request Jun 4, 2024
31 tasks
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.

Nice work, this looks great! I left one comment that I think we should decide on.

Leaving my approval now, pending a decision on the below.

@garrettjoecox garrettjoecox force-pushed the persistant-masks branch 2 times, most recently from 4cef72e to 310be36 Compare August 18, 2024 15:16
@Archez Archez merged commit 53faf37 into HarbourMasters:develop Aug 18, 2024
5 checks passed
@garrettjoecox garrettjoecox deleted the persistant-masks branch September 13, 2024 14:03
mckinlee pushed a commit to mckinlee/2ship2harkinian that referenced this pull request Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants