Well, HDMI is good for plugging into the company projector or any old TV in the spur of the moment -- no carrying a dongle around.
The SD card reader is really nice for carrying enough stuff around to totally erase your Mac and reinstall everything without downloading anything, plus any document backups you might need. You will be able to buy one that fits flush to the case, so it doesn't stick out any and you just leave it there, no dongle, no carry around the card.
Magsafe is better than USB-c for power, constantly pugging in and unplugging a USB-c port will eventually wear something out.
Sounds good to me!
That said, I like lots of ports, a couple USB-A's would have been nice too.