You're missing the entire point, firstly on it's own it does look good but when put into context of said character it doesn't. This is because when someone is designed to be a little extra buffed it portrays signs of both strength and confidence, the former of which he doesn't/shouldn't based on both his own actions and dialog from him and others. I'm not saying he can't have muscles as that would be bad as he goes to the gym, or in your words is a gym trainer. It would be better if his character design gave off the feeling of his insecurities, fear and/or trauma; this is so when seen you can tell or guess his personality without the need of heavy storytelling. As a said with this he just looks like a generic muscle guy/jock that's friendly just a tad shy and would always put himself in front of all danger. On a side note not all gym trainers have a lot of muscle, in fact a good one would look on the more lean side. A great example would be to look at physical instructors; finally there seems to be a trend among the newer disgns of everyone being muscular and it's better to have diversity to distinguish them. I won't be a yes man as I've seen others do and I prefer to give constructive criticism, I really feel like the old design gave a better portrayal of his character