I think it would be great if instead of using a pin named ’ Target’, you could clearly define what object the function should be called from.
The current way for giving a function target makes it look like you’re passing a variable to the function. I feel like it’s not appropriate to place a Target pin in a list that should exclusively be meant for input.
What I would like to see is a small pin on the top left of the blueprint node, without any real obvious markings. When you drag in a target object, the text should change from:
Fire Weapon
Target is Self
to
Fire Weapon
Target is BP_WeaponBase
Nothing else would be needed to make clear that the blueprint node is called from an instance, and even with a quick glance you would see a wire going into the top of the Blueprint node. I feel like a distinct split of the target and the input variables would be better.