Add support for Imbued Supports#9670
Conversation
…e else because it's an infestation
…s an imbued don't show imbued label/dropdown for skills from items
update scrollBar in SkillsTab to fit imbued change Clear button to X design of the other gems
|
I found a few issues. (sorry) The hover tooltip on the imbued shows the tooltip for the second gem in the socket group. Whether it be a support or an active gem. If there is no second gem, no tooltip pops up. Opening the Imbued selection then clicking away to close it, will cause the imbued select to change to the top most support, instead of reverting to the current selected gem. Related to the first two, the tooltip does not show on Import until a change is made (quality, count, new gem, etc). Then it will follow the logic above. Typing to search for the gems is different than the normal dropdowns. I can't type "spell" to have spell echo show up. With Fireball as the skill. I type s, it auto fills with Sacrifice. Then, any further letters are added onto that gem name, so "Sacrificepell". If you backspace to remove the text, once it's fully erased, backspacing again will show the hex color code. |
…lor, click away from dropdown fix
Wires77
left a comment
There was a problem hiding this comment.
- The dropdown should be moved under "Socketed In:" for portrait mode users.
- Tooltip suggestion:
"Socketed in" must be set...or"Socketed in" item must be setto make it clear you're talking about a UI item - You could restrict the dropdown to only show the support gems that would receive a checkmark, since only those are explicitly allowed
This otherwise looks pretty good, any reason for using colorCodes.POSITIVE instead of colorCodes.CRAFTED to better match in-game?
fix - disable remove button on disabled imbued socketGroup
Remove gems from the list that cannot support the active skill Adds code in calc tools to grab the skillTypes from the base skill so any added skill types are ignored
Cherrypicked from #9670 to make the diff from that PR cleaner
* Remove Quality type dropdown Cherrypicked from #9670 to make the diff from that PR cleaner * Remove more quality stuff * Last quality fix * Remove quality types from export * Remove dummy stat quality export --------- Co-authored-by: LocalIdentity <localidentity2@gmail.com>
Importing a skill gem that has an imbued support for a different gear slot would override the existing gems in the build
Setting a imbued gem on a skill set would block all the other skill sets from also setting a gem for the same item slot
Using something like Forbidden Shako was having the support gems from the helmet apply to active skills granted by the helmet when they are not meant to apply
The sortCache now only runs on non-exceptional supports to loads a lot faster when clicked
The buttons for sorting by support or active skill gem were invisible for the imbued box but still applying if clicked. Now they do not apply when clicking in that area
Phase 2/3 of port. Removes all qualityId / alternateGemQualityPrefixMap references (upstream PathOfBuildingCommunity#9758 removed alt quality). Adapts imbued upgrade simulation to upstream PathOfBuildingCommunity#9670 API: socketGroup.imbuedSupport + skillsTab.imbuedSupportBySlot[slotName] instead of gemInstance.imbuedSupport. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Phase 2/3 of port. Removes all qualityId / alternateGemQualityPrefixMap references (upstream PathOfBuildingCommunity#9758 removed alt quality). Adapts imbued upgrade simulation to upstream PathOfBuildingCommunity#9670 API: socketGroup.imbuedSupport + skillsTab.imbuedSupportBySlot[slotName] instead of gemInstance.imbuedSupport. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Phase 2/3 of port. Removes all qualityId / alternateGemQualityPrefixMap references (upstream PathOfBuildingCommunity#9758 removed alt quality). Adapts imbued upgrade simulation to upstream PathOfBuildingCommunity#9670 API: socketGroup.imbuedSupport + skillsTab.imbuedSupportBySlot[slotName] instead of gemInstance.imbuedSupport. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Phase 2/3 of port. Removes all qualityId / alternateGemQualityPrefixMap references (upstream PathOfBuildingCommunity#9758 removed alt quality). Adapts imbued upgrade simulation to upstream PathOfBuildingCommunity#9670 API: socketGroup.imbuedSupport + skillsTab.imbuedSupportBySlot[slotName] instead of gemInstance.imbuedSupport. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Key changes: - Imbued Supports system (PathOfBuildingCommunity#9670) — replaces alternate quality code (PathOfBuildingCommunity#9758), with follow-up fixes for gem list visibility (PathOfBuildingCommunity#9782, PathOfBuildingCommunity#9796), trigger supports (PathOfBuildingCommunity#9793), and indexing bugs (PathOfBuildingCommunity#9763). This is the in-league replacement for Awakened gems and changes how transfigured supports scale. - Calc fixes wave: skills that can repeat (PathOfBuildingCommunity#9723), Vaal Flameblast base AoE (PathOfBuildingCommunity#9704), Generals Cry clones restricted by attack time, Maven Memory Game damage (PathOfBuildingCommunity#9757), Hextoad breakdown (PathOfBuildingCommunity#9683), CwC crash with multiple link groups (PathOfBuildingCommunity#9682), Searing Bond of Detonation power charges (PathOfBuildingCommunity#9695), Tinctures applying without Mana (PathOfBuildingCommunity#9769), Ghost Reaver leech with Life Leech sources (PathOfBuildingCommunity#9774), Poisonous Concoction added damage from disabled life flasks (PathOfBuildingCommunity#9339), Blade Blast of Dagger Detonation Varunastra counting (PathOfBuildingCommunity#9733), converted damage with poison stacks (PathOfBuildingCommunity#8991), Chieftain resist nodes Foulborn Choir double-apply (PathOfBuildingCommunity#9643), Turbulent catalyst flat lightning scaling (PathOfBuildingCommunity#9775), some buffs applying to player and minions wrongly (PathOfBuildingCommunity#9789), negative eHP and NaN Max hit (PathOfBuildingCommunity#9772), Dual Wield mastery thrusting+non-thrusting (PathOfBuildingCommunity#9761). - Ring slot 3 fixes (PathOfBuildingCommunity#9686, PathOfBuildingCommunity#9730) — mods applying when Unseen Hand isn't allocated. - Build comparison tab (PathOfBuildingCommunity#9543) — pure UI, headless mode unaffected. - New spectres: Conjuror of Rot (PathOfBuildingCommunity#9702), Unholy Aberration (PathOfBuildingCommunity#9731). - Trader improvements (PathOfBuildingCommunity#9691), Watcher's eye trade search (PathOfBuildingCommunity#9649), duplicate-search fix (PathOfBuildingCommunity#9767), stale state guard (PathOfBuildingCommunity#9742). - Lesson of Seasons spell barkskin mod (PathOfBuildingCommunity#9820). - Lingering Blades extends to Animate Weapon of Ranged Arms. - Runtime DLLs bumped to SimpleGraphic v2.5.3 (PathOfBuildingCommunity#9395) — affects desktop PoB UI only; headless API runtime is the bundled LuaJIT. - Releases 2.64.0 and 2.65.0.

Description of the problem being solved:
Full implementation of Imbued Supports, including Importing from Character, Saving and Loading to the XML, and fixing "too many gems in slot" warning. The selection automatically filters for Level 1 Non-Exceptional Support gems. The colour of "Imbued Support:" will follow the application options Positive Values Hex colour.
You are only allowed one imbued support per Slot.
PS: This PR also removes the Variant column in the skillsTab. It happened long enough ago I forgot it was in this branch. If you don't want it in this change, let me know.
After screenshot: