@ToryDav, in general, pretty good. A few comments I’d make are:
Physical - why keep all the uplinks and downlinks in the top and bottom members of the stack? Why not spread them out more so that a failure doesn’t take out so many links?
Layer 2 - I’d use aggregated ports (LACP) on the links between the MDF and IDF so you can utilise the bandwidth of both links. It also simplifies the STP model as you have a single logical link. The links to the MX have to rely on STP.
Laye 3 - both the MX have to be in the same VLAN as they share an IP address on the LAN side. I’d use the MS for the Layer 3 core, with the transit VLAN from the MXs. You’ll need another VLAN on the MXs though for all your management IPs to reside in (the management IP on the core switch shouldn’t be in the same VLAN as one of the Layer 3 interfaces in the stack). Make sure the MS you are using supports DHCP… 😀
Hope that’s some help.