Add feature to view all mods sliders on Uniques by default (config to turn it off)#9187
Merged
LocalIdentity merged 3 commits intoPathOfBuildingCommunity:devfrom Nov 22, 2025
Conversation
Signed-off-by: Justin Stitt <[email protected]>
Signed-off-by: Justin Stitt <[email protected]>
Contributor
Author
|
uhhhh i broke rare items... 9b1adf5 fixes it though :) |
LocalIdentity
approved these changes
Nov 22, 2025
Contributor
LocalIdentity
left a comment
There was a problem hiding this comment.
Nice feature. I'm going to change this to be on by default as I think it's more helpful for users to be that way. Nice work
github-actions bot
pushed a commit
to PathOfBuildingCommunity/PathOfBuilding-PoE2
that referenced
this pull request
Dec 15, 2025
LocalIdentity
pushed a commit
to PathOfBuildingCommunity/PathOfBuilding-PoE2
that referenced
this pull request
Dec 15, 2025
…t (config to turn it off) (#1592) * Apply changes from PathOfBuildingCommunity/PathOfBuilding#9187 * Fix merge conflicts * PR#1593 changes Merged the changes from the other PR into this one --------- Co-authored-by: JustinStitt <[email protected]> Co-authored-by: LocalIdentity <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of the problem being solved:
When altering item stats it can be quite cumbersome to change each affix one at a time while jumping between options in a dropdown. For example,
Timeclasphas 6 affixes with ranges and to modify this one to your liking would require many many clicks.My PR adds an option that auto-expands this affix dropdown into a stacked list of affix+slider pairs.
Steps taken to verify a working solution:
Optionsand enableShow all item affixes sans dropdownShow all items affixes without dropdownoverflowed the options pane)Link to a build that showcases this PR:
Empty build
Before screenshot:
After screenshot:
The option is off by default. I think most people will want it on, though. Should we turn it on by default?