Skip to content

Added Trade button near each item slot, leading to the trade site wit… - #9354

Closed
CyberNinja-Code wants to merge 2 commits into
PathOfBuildingCommunity:devfrom
CyberNinja-Code:ItemsTradeAbility
Closed

CyberNinja-Code wants to merge 2 commits into
PathOfBuildingCommunity:devfrom
CyberNinja-Code:ItemsTradeAbility

Conversation

@CyberNinja-Code

Copy link
Copy Markdown

…h the items mods filtered

Fixes # .

Description of the problem being solved:

Created more eazy way to search in trade the items in POB

Steps taken to verify a working solution:

  • Testing nothing brakes from the item configuration
  • Testing all the item's possiabiles (Rarity, number of modes etc.)

Link to a build that showcases this PR:

Before screenshot:

image

After screenshot:

image

@Nightblade

Copy link
Copy Markdown
Contributor

Which LLM did you use for this?

@CyberNinja-Code

Copy link
Copy Markdown
Author

Unfortunately i didn't choose something spesific for this, the cursor helped on "Auto" mode

@Blitz54

Blitz54 commented Nov 26, 2025

Copy link
Copy Markdown
Contributor

The UI is all funky with these changes. Weapon Set is way farther to the right, as are all the item lists. The item lists are also lower for some reason.

@Nightblade

Copy link
Copy Markdown
Contributor

What "AI" did you use? ChatGPT, Copilot, Claude, Gemini, etc.

@CyberNinja-Code

Copy link
Copy Markdown
Author

The UI is all funky with these changes. Weapon Set is way farther to the right, as are all the item lists. The item lists are also lower for some reason.

I already fixed it in the second commit, thanks
image

@CyberNinja-Code

Copy link
Copy Markdown
Author

What "AI" did you use? ChatGPT, Copilot, Claude, Gemini, etc.

As I saied before I used Cursor and it was on "Auto" mode so it used few kinds of AI models, which the Cursor didn't spesifiy how and when he used each modle that the "Auto" mode choose

@LocalIdentity

Copy link
Copy Markdown
Contributor

This had been added by a different PR and integrated with the existing code base better

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.

5 participants