refactor in melee targetting

height was an unused parameter in validMeleeRange(), so that's gone.
also there was a bunch of unnecessary code in the action menu state that
may have been the potential cause of the diagonal targetting bug, it now
calls validMeleeRange() instead.