Hi, Joseph.
First off, sirtoby is absolutely right. At any rate, I'm going to post here because GC strikes me as the proper forum for this topic. Just a few comments ...
Nicely drawn, but a bit jumbled. Since each channel is identical, you might want to make a schematic with just a single channel. In my opinion, this helps simplify the drawing, leaving it easier to read. (In my early morning grogginess after spending all last night proofreading a thesis draft, I'm having a harder time following those traces than I'd like to admit.
) This also makes revision simpler, as you only need to modify a single channel.
Your tri-color LED connections are unclear. After staring a bit, I've decided that it's what you must mean by the #1 Hi, #1 low connections. Is that correct? You really might consider drawing the correct symbols for the LEDs and placing them in your schematic. Here's an example with bi-color LEDs:
(One color indicates full voltage to the fan, the other variable voltage. I wouldn't fret much about the rest of the design at the moment.)
It looks to me like you're trying to use some sort of voltage divisor-like circuit to drop the voltage from 5V for your LEDs. However, it appears to me that as you've placed the resistors parallel to the LEDs and there are no resistors preceding or following those LEDs, they will still receive a full 5V. You should modify your design to have the resistors on the ground side of the LEDs. Furthermore, you can eliminate some resistors from your design in that way.
Lastly, I don't see anything to indicate an on/off switch for each channel. I see 5V / 12V, but no ON/OFF. As such, I see nothing there to switch the tri-color LED to its third state. A logic circuit could be used to take care of that, I suppose.
Good luck, and have fun! -- Paul