She is implied to be sad*stic and overwhelmingly corrupt, to the point of finding joy in torturing students. Harry describes Umbridge to Sirius Black, saying that she's vile enough to be a Death Eater, despite not being one.