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

Allow vulps to wag #1152

Conversation

pissdemon
Copy link
Contributor

@pissdemon pissdemon commented Apr 29, 2024

About the PR

Vulps can now toggle wagging if their tail supports it. Corgi and dalmatian tails also get a non-wagging variant courtesy of CD.

Why / Balance

People have wanted this for a while, I guess?

Known issues

  • Button is still displayed when the tail does not have a wagging animation (which a few do, I think it's: fennec, fox2, fox3, fluffy, otie, long). Nothing will happen if you click on it. Maybe a bit confusing.
  • Will log a warning to the server console when someone tries to wag with a tail that doesn't have a wagging animation.

Technical details

Basically ported from cosmatic-drift-14/cosmatic-drift#214. Uses the lizard tail wagging system. Old "wag" variants are removed and replaced with hidden (in character setup) "animated" variants.

Media

It's exactly the same as the lizard tail wagging.

Breaking changes

Database needs a migration to be run on it that fixes the markings field on profiles, since this PR removes the "wagging" variants we used to have and people have those selected. They need to be replaced with the non-wagging variant in the DB, just look at #rebase.

Changelog
🆑

  • tweak: Vulpkanins can now toggle wagging their tails. (Some tails cannot wag yet, please contribute sprites for them!)

@github-actions github-actions bot added Changes: Localization Changes any ftl files Changes: Sprite Changes any png or json in an rsi Changes: YML Changes any yml files labels Apr 29, 2024
Copy link
Contributor

github-actions bot commented Apr 29, 2024

RSI Diff Bot; head commit d6522dc merging into 511685e
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/DeltaV/Mobs/Customization/Vulpkanin/tail_markings.rsi

State Old New Status
corgi Added
dalmatian Added
tail-wag-icon Added

Edit: diff updated after d6522dc

@pissdemon

This comment was marked as resolved.

@pissdemon
Copy link
Contributor Author

Will probably make the migration part handled with an EF migration later

@pissdemon pissdemon marked this pull request as draft April 29, 2024 20:10
…u-please-get-my-itch-put-paws-on-your-chest-nyea
@github-actions github-actions bot added the S: Merge Conflict Fix your PR! label May 27, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@Samsterious
Copy link
Contributor

Cool. More non-intrusive roleplay mechanics that give people just a little more flexibility? Hell yeah. CD is a treasure trove of uninvasive HRP features that work just fine in MRP, and I am super down to see more PRs mirroring their successes.

@pissdemon
Copy link
Contributor Author

Closing this in favor of a port of this from EE or something.

@pissdemon pissdemon closed this Sep 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Changes any ftl files Changes: Sprite Changes any png or json in an rsi Changes: YML Changes any yml files S: Merge Conflict Fix your PR!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants