It LOOKS fine. But then, looks aren't the criteria you should be worried about. That would be performance. And this thing's not going to perform.
For starters, I can tell that you're trying to build one of these mythical "does everything" builds. And yes, they can be built. But in general, they're loads larger than this; trying to build a "does everything" in a cab this small never works. And the results are also apparent here...it's missing a lot of the "boring" modules that actually allow the "sexy" ones to operate at their full capabilities.
First of all, yank out all of those drum modules. No, seriously. Add up how much all of the drum modules cost, as well as the cost of the cab space they cover (this gets calculated like this: cab price/hp count=price per hp). Then jump over to the website of any major music gear retailer and look at their drum machines. Ultimately, you'd do MUCH better with a separate drum machine than trying to build one in here. Not only would it free space in the cab, be easier to lock up with a DAW, and so on...it's definitely going to be loads cheaper unless you get something crazy-spendy.
Once you've freed up space in here, then you'll have room for the modulation generators, utilities, and other devices that the rest of the modules require. At that point, you should probably change the layout so that it makes more sense. Put all the audio sources in one area, filters in another, VCAs, submixers, etc etc. You get the idea. By doing this, it simplifies the "map" of the panel and makes this much easier to patch. As an example, have a look at an ARP 2600 or, for that matter, Behringer's clone of it...and you'll see this some of this sort of layout paradigm in action.