#11
|
||||
|
||||
![]()
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.
__________________
https://apps.slashkey.com/farmtown/p...00000691893378 Visit 's Farm ![]() DB:W20, MPS:C20 Last edited by TsunamiVince; Jun 08 2010 at 03:56 AM. |
|
|