Nothing wrong with what you've chosen.
Depending on your budget, I'd go a bit cheaper with some of your components and get an SSD too. Specificly, I think an i5 750 with an 4870 just to decrease the price a bit to pay for the SSD. Even without the SSD, I'd carefully examine ATI 4870/4890...