Snarvid wrote:
Anyway - a bunch of the skills have something like “Dual?” or “Shield?” listed after them. I get that kind of inkling myself when I look at some skills (that is, “is this one that requires dual or not?”) so it’s less helpful and I still have to check the wiki. But maybe the “?”s aren’t intended to still be there?
Ah, okay, in retrospect, I can see where that would be confusing. The (?Shld) annotation is supposed to be a combination of the (?) annotation that means "you don't meet the preconditions to use this talent" and the (Shld) annotation that means "you have to be wielding a shield to use this talent". You're right, though, I should probably do that differently.
Snarvid wrote:
IIRC you’re also missing Sun Paladin’s shield requirement on their Guardian skills, or at least on Shield of Light. I might have a screenshot, will check later (am not posting from that compy). Did you catch Mechanical Arm’s offhand mind star req?
[sound F/X: source diving] Hmm, Mechanical Arms doesn't have an on_pre_use() method, which is how the addon detects these things. And I'm not sure I could parse that particular requirement out from the raw binary dump of the method anyway. Shield of Light looks like it should work properly, though.
Snarvid wrote:
And I kinda like the white on blue, it fades when I’m not looking for it while being plenty legible when I look.
(shrug) Good enough.
Snarvid wrote:
Edit: This is probably beyond the UI’s ability to teach effectively, but with including a saw notation I thought I would mention that (offhand?) steamsaws satisfy shield requirements, dual steamsaws satisfy dual use requirements, and the two-handed artifact steamsaws satisfy two-handed weapon requirements as well as both the preceeding. Maybe just a note in the documentation?
True. This part is kind of a hack, frankly; all the addon is really reporting here is whether the talent's on_pre_use() method calls Actor:hasTwoHandedWeapon() or Actor:hasShield() or Actor:hasDualWeapon() or Actor:hasWeaponType("steamsaw"). Hmm, I wonder if I could add tooltips to that talent list…
Snarvid wrote:
You can see that Crusade has no notation for requiring a shield.
[sound F/X: source diving] Aaand Crusade doesn't have an on_pre_use() method either. Man, I was
really hoping I could get away with not hardcoding a bunch of this stuff… Okay, give me some time to work on this.