Skip to content

Interaction Verb Minor Fixes#965

Merged
VMSolidus merged 4 commits intoSimple-Station:masterfrom
Mnemotechnician:fix/interaction-verbs-minor-fixes
Sep 30, 2024
Merged

Interaction Verb Minor Fixes#965
VMSolidus merged 4 commits intoSimple-Station:masterfrom
Mnemotechnician:fix/interaction-verbs-minor-fixes

Conversation

@Mnemotechnician
Copy link
Copy Markdown
Contributor

Description

  • Updates the standing state requirement and the change standing state action to use the new laying down system (CurrentState instead of the old boolean Standing).
  • Makes it so that you cannot use any interaction verbs on entities you cannot interact with (fixes it being possible to look at entities while being in crit)
  • Changes the interaction popup clipping algorithm to include a predicate that ignores any entity that does not occlude light (fixes mobs, welding tanks, and some other non-opaque things preventing you from seeing interaction popups)
  • Makes it so that the verb requirements are checked one additional time before the verb is interaction is actually executed. (this e.g. prevents the MakeSleepOther from working if the target stands up during the do-after)

Changelog

🆑

  • fix: Fixed multiple minor issues with interaction verbs. Most importantly, interaction popups will now always be shown correctly.

@github-actions github-actions bot added the Changes: C# Changes any cs files label Sep 27, 2024
@SimpleStation14 SimpleStation14 changed the title Interaction verb minor fixes Interaction Verb Minor Fixes Sep 27, 2024
@ShadesMars ShadesMars requested review from a team, DEATHB4DEFEAT, OldDanceJacket, Peptide90, Pspritechologist and VMSolidus and removed request for a team September 28, 2024 02:44
@ShadesMars ShadesMars added Size: 4-Small For small issues/PRs Priority: 3-Medium Needs to be resolved at some point labels Sep 28, 2024
@github-actions github-actions bot added the Status: Needs Review Someone please review this label Sep 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changes: C# Changes any cs files Priority: 3-Medium Needs to be resolved at some point Size: 4-Small For small issues/PRs Status: Needs Review Someone please review this

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants