Skip to content

Hopefully fix Tyler's HUD#113

Open
eno26 wants to merge 8 commits into
Mantibro:betafrom
eno26:tyler
Open

Hopefully fix Tyler's HUD#113
eno26 wants to merge 8 commits into
Mantibro:betafrom
eno26:tyler

Conversation

@eno26

@eno26 eno26 commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Attempted to fix Tyler's HUD that borked in the previous pull request, and also make the hunt text actually function.

Comment thread lua/slashco/slasher/tyler.lua Outdated
return
end

if state == 3 then

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please use TYLER_DESTROYER (the code further above probably needs to be updated at some point too)
I really dislike magic numbers

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed, I initially used TYLER_DESTROYER but I don't remember why I changed it, same goes for the rest of the code that should use TYLER_SPECTER, TYLER_CREATOR, etc.

@RaphaelIT7 RaphaelIT7 Jul 14, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

You should do state == TYLER_DESTROYER as TYLER_DESTROYER like the others are enums to use for comparison xd
Doing if TYLER_DESTROYER then will falsely trigger it aways

@RaphaelIT7 RaphaelIT7 added the bug Something isn't working label Jul 11, 2026
eno26 and others added 7 commits July 14, 2026 18:49
Added a "Hunt Mode" text that displays the amount of time that Tyler has left before his Destroyer form ends.
Tested a gazillion times, the hud should ACTUALLY be visible now, if not, you have my permission to castrate me.

@RaphaelIT7 RaphaelIT7 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Should look good now.
I've resolved the merge conflict and did a last fix.

@eno26 can you look over it once to see if it's fine for you too?
I can't tell if you intended to keep or remove the hud.TitleCard.Label:PaintOver() function

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants