Skip to content

Conversation

akleshchev
Copy link
Contributor

Made it distinguish wearable types to be more reliable, but we probably want to prevent this option from being used in first place since it can't work reliably for objects, would work fine only for wearables.

@akleshchev akleshchev requested a review from Copilot October 8, 2025 19:23
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR improves the wearing of multiple items by distinguishing between wearable types to make the replacement behavior more reliable. The change addresses issue #4782 by modifying how the system determines when to replace existing items when wearing multiple items simultaneously.

  • Introduced a new needs_to_replace function to handle replacement logic per item type
  • Modified replacement tracking to use separate counters for objects vs. clothing types
  • Added documentation about limitations when replacing attachments with multiple objects

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@akleshchev akleshchev force-pushed the andreyk/viewer_4782 branch from 4f06c5f to 519ed4d Compare October 8, 2025 19:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants