Be very wary on what crystal oscillator your fab puts on these boards. I had to redo 100 pcs which had a random no-name oscillator on them (even though I specifically told them that was not a generic part...) because the radio did not get the exact frequency to talk to my other CC1310 boards.
You should also break out DIO0 and DIO1 because they are internally connected to the UART that is used for bootloader. A very handy feature that you can also use to program your devices even without a JTAG or XDS device. You can also use DIO9, DIO10, DIO11 and DIO12 for bootloader over SPI, but that is more cumbersome.
You really don't need those loading caps on the main oscillator, the internally tunable ones are just fine, but of course it doesn't hurt to have the footprints unpopulated.
As long as you don't use a pcb antenna, and the balun and UFL connector are that close to the chip, I would not worry much about the exact 50 ohm controlled impedance on the feed line. For sub ghz it is not as crucial.
The board looks nice, maybe you can consider making a version with castellated headers (like an SMD module) since there's no bottom side components.