“Atifas Yishmoelim should be done and therefore covering the face is a mistake.”
The M”A says to cover up to the upper lip (don’t cover the eyes), and M”B seems to prefer the opinion that one should cover down to the upper lip (do cover the eyes).
My minhag is like the M”A, and I don’t really understand the logic of covering the eyes, but we aren’t really in a place to call it a “mistake.” Sphardim also seem to leave the eyes uncovered, but they seem to scrunch the tallis up more like a scarf when they do it. I’m not really sure why this is.