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

Jewel Support Part 3: Skill Tree UI #542

Merged
merged 18 commits into from Dec 22, 2019
Merged

Jewel Support Part 3: Skill Tree UI #542

merged 18 commits into from Dec 22, 2019

Conversation

brather1ng
Copy link
Member

This adds jewel support to the skill tree UI. That includes:

  • Socketed jewel icons are drawn.
  • Tooltips of socketed jewels include the jewel item's tooltip and, if relevant to the jewel, the amount of allocated attributes in radius.
  • For filled jewel sockets, only the radius, if any, of the socketed jewel is drawn. This radius is always drawn, not only when hovering over the socket.
  • When hovering a socket, nodes in radius are highlighted. To better distinguish the different radiuses, each radius has a different color.
  • The Equipment tab shows the skill tree area around a jewel when hovering over the item slot.
  • Items can add connections to the passive tree, e.g. from Intuitive Leap. These aren't really added as actual connections, but they are handled correctly when skilling and refunding nodes.
  • (not entirely related) Passive nodes allocated by items (e.g. Keystones from Uniques, annointed amulets) are displayed as active. Blighted notables have their correct node surround asset used.

image
image

@brather1ng brather1ng mentioned this pull request Dec 20, 2019
5 tasks
@brather1ng brather1ng added this to the 3.0.0-beta.1 milestone Dec 20, 2019
@brather1ng brather1ng merged commit 3dae7bf into master Dec 22, 2019
@brather1ng brather1ng deleted the jewel-ui branch December 22, 2019 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant