Hard to see from the pics but it looks a hair too long and collar could be bigger. I always thought a raglan shoulder might work on that kind of unstructured jacket, but I’ve never seen it done so maybe it looks crap
Edit: are you doing the olive one up thread? I’ll take one if so
That last button still seems low - but maybe a trick of the model having his hands in the pockets.
I like 5 button personally.
Pointed Button loop on the sleeve cuff is nice