Skip to content

Center direct pet menus and add toy launch sounds - #35

Merged
Nowkoo merged 1 commit into
mainfrom
fix/menu-centering-toy-sound
Oct 9, 2026
Merged

Nowkoo merged 1 commit into
mainfrom
fix/menu-centering-toy-sound

Conversation

@Nowkoo

@Nowkoo Nowkoo commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

With a pet outside its Pet House, the direct care menu places Tricks, Send to Pet House and Release in columns 3, 5 and 7, keeping the last row centered. Rendering and click handling use the same positions, including after refreshing the menu or returning from Tricks.

Successfully launching any toy now plays the vanilla snowball throw sound at the player's location, audible to nearby players. Failed launches remain silent and keep the existing item refund behavior.

Validation:

  • mvn clean verify: 995 tests, no failures or errors, three opt-in tests skipped; 100% production line coverage.
  • Regression coverage for centered button actions, refreshes between stored/outside states, successful launch sounds and failed launches.
  • Deployed to TFMCDev01: CompanionPets starts with 19 pet types; configuration and eight saved pets preserved.

@coderabbitai

coderabbitai Bot commented Oct 9, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: de90ae55-1987-460f-b74f-39f0a23a610c

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Nowkoo
Nowkoo merged commit dde9512 into main Oct 9, 2026
2 checks passed
@Nowkoo
Nowkoo deleted the fix/menu-centering-toy-sound branch October 9, 2026 22:34
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.

1 participant