Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

display affix rolls/tier #7400

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

qndel
Copy link
Member

@qndel qndel commented Sep 2, 2024

devxitemtier

current issues:

  • obsidian ring shows 4/5 tier despite obsidian being top tier on rings because a higher tier exists for weapons/shields etc.
  • showing high durability stats is kind of meaningless
  • probably more

@qndel qndel added the enhancement New feature or request label Sep 2, 2024
@zutmkr
Copy link
Contributor

zutmkr commented Sep 2, 2024

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

@qndel
Copy link
Member Author

qndel commented Sep 2, 2024

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
Copy link
Contributor

zutmkr commented Sep 2, 2024

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

@StephenCWills
Copy link
Member

That's the smallest font size we have.

@julealgon
Copy link
Contributor

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
Copy link
Collaborator

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants