Minor cleanup of various methods in CardHand and BattlefieldZoneView
Removed some minor commends
Removed code duplication by making `removeCardActive` inside `CardView`
Clearing targetable and clearing active is the same thing
`setTargetable` is now an abstract method in `CardView`