Skip to content

Conversation

thechriskent
Copy link
Contributor

The pets all sharing the same size but varying in the gifs themselves can create a strange experience. I'm extra aware since I caused a lot of it with the horses 😄. This has been requested here #604 and here #725.

I've added a relativeSize property to the pets that you configure when spawning a new pet. This works in conjunction with the existing size property as a multiplier.

image

So, if you keep the default of Normal, you'll still have the monster chickens and tiny horses as expected:

image

But you can now adjust that by selecting different relative sizes (shown below with horse normal and chicken smaller):

image

Makes for some fun times!

image

Relative size has also been added to the state api and is included in pet-list import/export.

Hope this helps! Love this extension as always!

Added launch configuration to debug with all other extensions disabled (helpful when other extensions are filling the console)
- Added PetRelativeSize enum
- Added relative size choice after color when spawning a pet
- Added relativeSize to state
- Relative Size is applied to width/height in addition to Pet Size
- Relative Size also has minor effects on speed (bigger pets go a little faster and smaller pets a little slower) - makes the gifs look better
- Relative Size included in pet list
@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 20.75472% with 42 lines in your changes missing coverage. Please review.

Project coverage is 18.80%. Comparing base (9848d1f) to head (6b27bfb).

Files with missing lines Patch % Lines
src/panel/basepettype.ts 0.00% 20 Missing ⚠️
src/extension/extension.ts 18.75% 13 Missing ⚠️
src/panel/main.ts 0.00% 5 Missing ⚠️
src/panel/pets.ts 20.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #749      +/-   ##
==========================================
+ Coverage   18.53%   18.80%   +0.26%     
==========================================
  Files          32       32              
  Lines        1570     1606      +36     
  Branches      204      215      +11     
==========================================
+ Hits          291      302      +11     
- Misses       1269     1294      +25     
  Partials       10       10              

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

2 participants