Proper use of the texture system, as in like using Black50, Black25 and so on, that way I know the majority of the playerbase is going to see the intended colors instead of blank white crap. Having one weapon for each logical class (assault rifle, marksman rifle, sniper rifle, shotgun) is a plus. Having two weapons for each class and having one weapon come from 1st word and the other coming from the 2nd world is a big plus.
I don't like hit-scan crap or weapons that are incredibly inaccurate unless you're like crouching and only firing once per year. I don't like invisible projectiles. I don't like intrusive overly complicated animations like you see in that ancient L4D pack, mostly because it looks like stuff for anyone who has more than like 0.2 yoctoseconds of latency
Balancing is cool but I think a lot of weapon makers go overboard with it and end up making guns that feel really weak and lame to shoot or they end up making weapons that can't hit the broad side of a barn door at 10ft. Just having weapons that function logically is enough for me
Don't like weapon that prints a ton of stuff to my screen. I get the need for ammo counters but it almost always looks like crap imo and it really peeves me when the gamemode and player weapon are fighting for a space on the centerprint/bottomprint