Looks about right to me, may want to lower your saddle a bit though
why lower the saddle?
Knee looks a little too extended at the bottom of the stroke, although I'm not a bike fitter and OP should do whatever works for them if they're comfortable and injury-free.
A cleat may add a little stack height too (if they plan on using clipless pedals).
from what I know the knee should be extended fully at the bottome of the stroke!