Macro Help!

[Elvenbane-veknilash (Elvenbane)]

Both those are wrong, @cursor and @mouseover are mutually exclusive in terms of the spells they work with.

Use one of these templates:

@Elvenbane-veknilash

Help/Harm Macro

This will cast one spell when mousing over or targeting a friend and another when mousing over or targeting an enemy.

HELP_SPELL as Default

#showtooltip
/cast [@mouseover,help,nodead] HELP_SPELL; [@mouseover,harm,nodead] [harm] HARM_SPELL; HELP_SPELL

HARM_SPELL as Default

#showtooltip
/cast [@mouseover,harm,nodead] HARM_SPELL; [@mouseover,help,nodead] [help] HELP_SPELL; HARM_SPELL

That said, the issue with the first one is probably the key you have it bound to. UnitFrames eat mouse clicks, you need to use an addon like Clique to get around it.