Looks right, though perhaps not the most efficient way to do it. LED's are generally placed in series so they all have the same brightness (same current). You may see one LED quite a bit brighter than the others.
Thanks for answering my question on the pot- I thought I should hook it to the wiper. So what should the other end of the pot go to?
Ah, thanks for that! I had a brain fart on the series vs parallel thing. I was thinking voltage determined the brightness, but of course LEDs are current driven!