I think the outfit is fine. That said, I agree with the short sleeve shirt and waistcoat comment. Although the short sleeve shirt may not be "traditional", I do think it would look nicer without the waistcoat. There is something about a waistcoat for me that makes me prefer the look of a long sleeve shirt--even if the sleeves are then rolled back.