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

ShadowTag (7.0.1) #2825

Merged
merged 5 commits into from
May 28, 2023
Merged

ShadowTag (7.0.1) #2825

merged 5 commits into from
May 28, 2023

Conversation

Caladius
Copy link
Contributor

@Caladius Caladius commented May 3, 2023

Wallmaster follows link everywhere

Added ShadowTag Checkbox to Extra Modes
Spawn Code in mods.cpp
Category Change for Clear Room Exclusion
Kill Actor if Mode is Disabled

Build Artifacts

Spawn Code in mods.cpp
Category Change for Clear Room Exclusion
Kill Actor if Mode is Disabled
@Caladius Caladius changed the title ShadowTag (7.0.0) ShadowTag (7.0.1) May 12, 2023
… firing on quick room changes like falling to B1 in Tree and Lost Woods loading triggers.
Copy link
Contributor

@briaguya-ai briaguya-ai left a comment

Choose a reason for hiding this comment

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

overall this looks great! couple little naming/flow suggestions. one question about behavior, but this looks super close to ready!

soh/soh/Enhancements/mods.cpp Outdated Show resolved Hide resolved
soh/soh/Enhancements/mods.cpp Outdated Show resolved Hide resolved
soh/soh/Enhancements/mods.cpp Outdated Show resolved Hide resolved
soh/soh/Enhancements/mods.cpp Outdated Show resolved Hide resolved
soh/soh/Enhancements/mods.cpp Outdated Show resolved Hide resolved
soh/soh/Enhancements/mods.cpp Outdated Show resolved Hide resolved
soh/soh/Enhancements/mods.cpp Outdated Show resolved Hide resolved
soh/soh/Enhancements/mods.cpp Outdated Show resolved Hide resolved
Comment on lines +435 to +436
if (!CVarGetInteger("gShadowTag", 0)) {
Actor_Kill(&this->actor);
Copy link
Contributor

Choose a reason for hiding this comment

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

so is the idea here that after turning off shadow tag you need to hit the wallmaster once (or wait for it to return to the ceiling) before it'll go away?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Exactly, instead of interrupting the execution process, just let it finish out the last process and kill it. (or change rooms/scene)

soh/src/overlays/actors/ovl_En_Wallmas/z_en_wallmas.c Outdated Show resolved Hide resolved
Copy link
Contributor Author

@Caladius Caladius left a comment

Choose a reason for hiding this comment

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

Updated namings

Copy link
Contributor

@briaguya-ai briaguya-ai left a comment

Choose a reason for hiding this comment

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

:shipit:

@leggettc18 leggettc18 merged commit edc5e8f into HarbourMasters:develop May 28, 2023
7 checks passed
@garrettjoecox garrettjoecox added this to the Sulu (7.1.x) milestone Jun 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants