[TsunamiVince]

Which reminds me, how about:

if (we already displayed over production warning popup)
{
throw out all remaining clicks on the exact same thing clicked on
}

instead of

if (over production)
{
display warning popup
}

I constantly click on something to max it out, then have to wait through 25-30 "you have maxed out production" popups as it unwinds through every single extra click on the production button. I'm glad you finally have over-production popups instead of just simply throwing out over production with no indication of what is going on but as usual you fail at the actual implementation of this detail. One over production pop-up and then toss subsequent identical clicks on the same production. Since I have to spell this out in all gory detail:

click()
{
if (! over production)
{
last displayed popup = null;
produce
return
}

// over production here

if (last displayed popup == current clicked item)
{
return;
}

display popup;

last displayed popup = current clicked item;
}

This way 1) multiple clicks on one production item only displays ONE popup. 2) Clicking around all production items still displays ONE popup when you click on something different. This still allows you to make sure the server has actually queued up the maximum amount.