Add support for showing Required Attributes in comparison tooltips#9813
Add support for showing Required Attributes in comparison tooltips#9813Peechey wants to merge 1 commit intoPathOfBuildingCommunity:devfrom
Conversation
|
Didn't see #6413 until posting this, I could remove the m_max I'm doing in the diff and show the "overcapped" attributes, though it would be displayed like "-x Intelligence Required" in Positive Color without additional changes. I'd probably try to use a different label. Although... they kind of want it in the sidebar and that isn't the same as what this PR applies 🤷 |
|
FYI, two branches on my fork around the same ideas :
|
Description of the problem being solved:
A lot of times when I'm comparing gear or pasting items from trade the stats look PERFECT until I slot it and realize "oh that item was giving me strength that I needed" and I have to start my search all over and manually add the attribute in the search or switch gear around yada yada. This PR will show the required stat during comparisons. If you're over the required stat value and add more or removing the node/item doesn't put you below the requirement, it will not show in the tooltip.
I'd like to update the trader weights or at least add the ability to include the required attributes as a trader weight as well, but I haven't thought that one through and I'm not sure how much people want it anyways.
Steps taken to verify a working solution:
Link to a build that showcases this PR:
After screenshot:
Scorching Ray 155 Int Required
