That is a REALLY cute pattern! I love the belt loops also. I think it looks retro but it would look better a little shorter. Maybe a couple inches above your knee, IMO hitting at your knee is an awkward length, looks a little "nursing student". You did a great job!
You know, I agree with that. I think it looks great how it is, this would only improve it.