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

More Leash Tweaks #414

Merged
merged 6 commits into from
Dec 15, 2024
Merged

Conversation

Mnemotechnician
Copy link
Collaborator

Description

This performs the following changes:

  • Leashes now have configurable length. You can choose the preferred length in the verb menu.
  • The short leash proto has been removed and an appropriate prototype migration has been added.
  • Leashes were made anchorable (so we can finally keep Seb safe)
  • Leash sprites were visually scaled down (70% the original size)
  • Shock and bell collars have been reparented to ClothingNeckCollarBase.
  • Leash anchors now a visual offset field, that properly incorporates the visual rotation and scaling of the entity.
  • Leashes now use a brand new overlay renderer (a refactored JointVisualsOverlay)
  • Fixed a bug where the "remove leash" verb would not appear on the respective clothing.

Also performed some cleanup and refactor.

If someone wants to bother, they can adjust the leash anchor offsets on station pets; The existing system should properly support those. I don't have the mental capacity to do it right now, though, so only collars and scugcats have proper offsets.

Media

Anchorability:

weeee-2024-12-15_22.15.37.mp4

Demonstrating configurable lengths:

god.I.hate.naming.mp4


Changelog

🆑

  • add: You can now configure the length of a leash using the verb menu. Subsequently, the dedicated short leash has been removed.
  • tweak: Leashes can now be anchored, have a smaller sprite, and proper visual offsets.
  • fix: Fixed a number of bugs with leashes, including the "remove" verb not appearing on clothing.

@github-actions github-actions bot added Status: Needs Review Someone please review this Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: YML Changes any yml files labels Dec 15, 2024
@Floof-Station-Bot Floof-Station-Bot changed the title More leash tweaks More Leash Tweaks Dec 15, 2024
@Fansana Fansana merged commit f9070fb into Fansana:master Dec 15, 2024
17 checks passed
Floof-Station-Bot added a commit that referenced this pull request Dec 15, 2024
@Fansana Fansana mentioned this pull request Dec 15, 2024
sleepyyapril added a commit to TheDenSS14/TheDen that referenced this pull request Jan 5, 2025
<!--
This is a semi-strict format, you can add/remove sections as needed but
the order/format should be kept the same
Remove these comments before submitting
-->

# Description

<!--
Explain this PR in as much detail as applicable

Some example prompts to consider:
How might this affect the game? The codebase?
What might be some alternatives to this?
How/Who does this benefit/hurt [the game/codebase]?
-->

adds PR #414 from Floof Station
Fansana/floofstation1#414

- Leashes now have configurable length. You can choose the preferred
length in the verb menu.
- The short leash proto has been removed and an appropriate prototype
migration has been added.
- Leashes were made anchorable (so we can finally keep Seb safe)
- Leash sprites were visually scaled down (70% the original size)
- Shock and bell collars have been reparented to ClothingNeckCollarBase.
- Leash anchors now a visual offset field, that properly incorporates
the visual rotation and scaling of the entity.
- Leashes now use a brand new overlay renderer (a refactored
JointVisualsOverlay)
- Fixed a bug where the "remove leash" verb would not appear on the
respective clothing.
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 Changes: Localization Changes any ftl files Changes: YML Changes any yml files Status: Needs Review Someone please review this
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants