I can only say about faces.
Almost everyone uses the same face in female drow (with freckles). Isn't this an indicator that the rest of the faces just don't make sense? I mean, they're not attractive enough... It doesn't matter how many faces you add, as long as only one of them is attractive. Also I don't agree that elves too tall. There's nothing wrong with height, elves shouldn't be short, as far as I know.
Well from a lore perspective the races have very detailed characteristic discriptions in the PHB. Elves are skinny/slender and have very noticeable heart shaped facial features. They are noticeablely shorter and less stocky than the average human. Half-elves are somewhere in the middle. I am not saying this is what I personally feel this is how the races should look, I am pointing out the differences from lore.
Rly? You think that things, that have hands almost bigger than their own heads are good? O_o
Just looked, they are pretty massive.