diasurgical / devilutionX

Diablo build for modern operating systems
Other
8.08k stars 795 forks source link

display affix rolls/tier #7400

Open qndel opened 2 months ago

qndel commented 2 months ago

devxitemtier

current issues:

zutmkr commented 2 months ago

Maybe better to just show roll. L:ike in your example Zodiac +20 (15-30). And add in options "Show advanced tooltips"

qndel commented 2 months ago

Maybe better to just show roll. L:ike in your example Zodiac +20 (15-30). And add in options "Show advanced tooltips"

it won't fit

zutmkr commented 2 months ago

Then maybe make font smaller in the preview window when the option is selected.

StephenCWills commented 2 months ago

That's the smallest font size we have.

julealgon commented 2 months ago

Not a fan of this if only because it spoils the game for new players. Yes, I know the information is widely available online, but a person playing the game for the first time is not expected to have checked items online first.

This makes the initial experience worse. And that's not necessarily just the "first" game, since you can play this game for a while and not see all the affixes still.

If this was to make it, IMHO it should be disabled by default at the very least, and in the best case scenario, it should only ever show what the player has already found (which would be a bit of effort to implement since this is not kept track of anywhere currently).

kphoenix137 commented 1 month ago

Cool feature, but I'm not entirely sure it's necessary.