As mentioned by Billybob, untuck your sweater and as for the brogues, they’re really nice but too formal to be worn with those jeans. You need blucher brogues not balmorals when you wear denim. That said, the rest of the outfit is very nice. What brand of denim are you wearing? I like the natural fade on those.