Dimensions are 7' x 12', with a 3' path thru the middle. The marigolds are along the outside of the garden, & the side opposite butts up to the house. There are some empty spaces, to be filled in with something but I don't know what yet.
Am I on the right track? Or do I go back to the drawing board?

It looks to me as if the left side is a little tight, i.e. sunflower and cucumbers space a little tight, as well as the tomato space for 3 tomatoes and that much basil.

On the right side, I think you have given too much space in several areas, i.e, to the rosemary.

Solution: Give your space to the veggies and squeeze the herbs, they'll do okay anywhere as long as there is sun. But you know your strawberries will take over that bed, don't you? Think groundcover!! Unless they are alpines. Oh, and a question: are you climbing your cucumbers on the sunflowers or planting underneath or sunflowers after the cucumbers are toast?

It will be beautiful and packed just right so not much weeding. And by the second year, you'll know what you need to do to change it. Enjoy it.

I'm wondering why so many marigolds? That would be a great space to put a nice row of herbs to free up space elsewhere. You can interplant the marigolds throughout.

If you have room for pots..some things can go in pots as well to save space. The 3ft path is good..I'd stick to that as things will start to tumble into it as the season progresses.

Good Luck!

I have a good reason for all the basil: by the time I realized what I'd done, I had 5 different basil seeds started. Good thing I have friends who love to cook!

The sunflowers will be the supports for the cucumbers (I read somewhere that you could do that, so I thought I'd give it a shot.)

No real reason for the marigolds, other than I like the color. I had some in my garden last year, & when I was working on this one I dumped out the seeds that were still on the plants. Don't know if they'll take, but I figure I'm not out anything by trying.