partially revert menu tweaks to restore old look of diablo kill indicators

[qndel]

Restores look from #5210
partial revert of 9e7d223

current look (getting reverted) as a reference

[AJenbo]

I feel the colors take to much focus, and since they are not radio buttons it feels unnecessary that it show indications when they are not active.

I would prefer if we keep it so that they switch between nothing and gray.

[Trihedraf]

I preferred having the red

[qndel]

I was confused when some characters had nothing and others had some grey dots, thought it was some bug, empty slots always being drawn indicate there's something you can do to fill them 🤷

Also most people seemed to prefer to have them outside of the portrait:

[AJenbo]

I'm allergic to colors. To me it really clashes with how things look :/

But how about red and nothing, that way it matches the small icons:

[qndel]

how about drawing nothing if character hasn't killed diablo but drawing remaining empty sockets after there's at least 1 colored gem?

[agris-codes]

I have two suggestions.

1. Location: the icons as proposed above, except for those outside the class portrait, are left-justified such that they crowd the faces in the portrait. I think moving the icons to the bottom-right of each portrait will look less cluttered, as heads are in the left segment of the frame and weapons tend to be in the top-right. I have no strong opinion on horizontal vs vertical alignment, although horizontal may look a bit less cluttered with the Sorcerer’s hand.

2. Icon appearance: I agree with @qndel re: drawing nothing when a player hasn’t defeated Diablo, although I share @AJenbo ’s concern that the outlines make them look like radio toggles. My suggestion is to use either the blood stone inventory graphic in small (16x16? 24x24?) dimensions, or a simple pentagram. The blood stone looks enough like the soul stone to be evocative, and the pentagram is self explanatory.