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

Horses! 🐎🐎🐎 #572

Merged
merged 9 commits into from
Jun 4, 2024
Merged

Horses! 🐎🐎🐎 #572

merged 9 commits into from
Jun 4, 2024

Conversation

thechriskent
Copy link
Contributor

Added the tiny horse pet with 11 colors for all your tiny horse needs!

Tiny Horse Color
black_idle_8fps Black
brown_walk_8fps Brown
white_with_ball_8fps White
paint_brown_idle_8fps Paint Brown
paint_black_swipe_8fps Paint Black
paint_beige_walk_fast_8fps Paint Beige
socks_brown_walk_8fps Socks Brown
socks_beige_idle_8fps Socks Beige
socks_black_run_8fps Socks Black
magical_idle_8fps Magical
warrior_swipe_8fps_Looped Warrior

Animations adapted from the beautiful Horse Sprite With Rider Asset Pack by Onfe which is free to use with attribution.

I've been using this extension for years and kept thinking it would be cool if it had some horses - then I realized you accept PRs 😀. Please let me know any adjustments you'd like or if there's anything else you need from me to get the people the horses they demand. Thank you for the amazing and fun extension!

thechriskent and others added 3 commits June 1, 2024 19:21
@codecov-commenter
Copy link

codecov-commenter commented Jun 3, 2024

Codecov Report

Attention: Patch coverage is 15.27778% with 61 lines in your changes missing coverage. Please review.

Project coverage is 16.58%. Comparing base (3c3a51e) to head (bab8101).
Report is 7 commits behind head on master.

Files Patch % Lines
src/panel/pets/horse.ts 10.90% 49 Missing ⚠️
src/panel/states.ts 16.66% 10 Missing ⚠️
src/panel/pets.ts 60.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #572      +/-   ##
==========================================
- Coverage   16.80%   16.58%   -0.23%     
==========================================
  Files          22       23       +1     
  Lines        1166     1236      +70     
  Branches      158      167       +9     
==========================================
+ Hits          196      205       +9     
- Misses        964     1025      +61     
  Partials        6        6              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@tonybaloney
Copy link
Owner

This is fantastic!!

Some feedback:

  • Please can we have the pet type as "horse" instead of "tiny-horse". You can set the pet size to large, so they're pretty big (well not compared to an actual horse), but nothing else in the extension is tiny-xxx and they're all relatively the same size
  • I'm not sure if "warrior" should be "red" as this is already a color.
  • I'm not a fan of the climbing state and the jump down animation. I've never seen a horse climb vertically before. Please can we remove those.
  • The horses are pretty active. The idle state is currently only designed for the left side, but it would be nice if they were idle on both sides. I grew up with horses and they spend most of the day eating grass. You could override the idle action to have longer idle times and make a flipped idle (see how the walk-right and walk-left actions work)

'Tony the Pony',
'Vigo Horsenberg',
'Ol Jetrho',
'Pony Soprano',
Copy link
Owner

Choose a reason for hiding this comment

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

Thank you for not including Pie-oh-My

Clipped their wings
Added a Stand state used for grazing with the horses. Now they eat facing either direction. Also reworked states to slow them down a bit and emphasize grazing. Wowee!
@thechriskent
Copy link
Contributor Author

Thank you for looking at this so quickly!

I've made several updates as requested:

  • tiny-horse is now just horse everywhere!
    • Ironically, I only switched it to tiny-horse shortly before submitting the PR since they looked so small next to some of the other pets that they'd normally be way bigger than 😆. But makes sense to keep it consistent.
  • Climbing related states are now gone.
  • Added stand left and right states for grazing. Also reworked the states to slow them down overall and emphasize grazing. Let me know if this isn't quite what you were looking for.

I'd like to keep the warrior "color" if possible as warrior horses is a very fun concept to me and one I use a lot for my open-source demos. But, this is your extension, and I will respect your opinion on the matter. Just let me know if you'd still like it updated to red and I'll take care of it right away.

@thechriskent thechriskent changed the title Tiny Horses! Horses! Jun 3, 2024
@tonybaloney
Copy link
Owner

I added another path into the run states as I noticed they weren't running unless you throw the ball. bab8101

In another change, I'd like to make the standing action a bit more dynamic and have each pet type be able to override it's "idleness". The cats are unrealistically active 🐱

A fun side-effect of the friendship tree is that the horses will run around in herds sometimes.

Other than that, LGTM! Thanks for the contribution

@tonybaloney
Copy link
Owner

I'd like to keep the warrior "color" if possible as warrior horses is a very fun concept to me

Sure. It's your PR 👍🏻

"orange": "Orange",
"blue": "Blue",
"akita": "Akita",
"socks black": "Black (Socks)",
Copy link
Owner

Choose a reason for hiding this comment

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

Added the new colours so people can add the translations later

@tonybaloney tonybaloney changed the title Horses! Horses! 🐎🐎🐎 Jun 4, 2024
@tonybaloney tonybaloney merged commit 580aeb4 into tonybaloney:master Jun 4, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants