Some clicks not registering? [Demo Provided] [Solved]

ID:248146
 
Feb 4 2012, 9:27 am (Edited on Feb 4 2012, 10:18 am)
(See the best response by Kaiochao.)
Code:
atom
    Click(turf/l,c,p)
        world<<"Click() is being called"
        var/list/L=params2list(p)
        if(L["right"])RightClick(l,c,p)
    RightClick(l,c,p)
        world<<"RightClick() called!"
        if(usr.can_move)return//If your not in the middle of a shield bash
        flickOn('effects.dmi',"target",-17,-17)//Creates that little target effect
        walk_towards(usr,l)
        usr.resetAttack()//Cancels the attack


Problem description:
http://www.mediafire.com/?vc0viufacr0rupi - Link to the demo

As you right click to move around does anyone else feel like some clicks don't register. I do have MouseEntered running as well if that causes some weird lag that I should know about. It seems to be most prevalent when your already, and the screen is moving

P.S. I know that you can shield bash into the void.
Feb 4 2012, 9:48 am
Best response
I don't think that is how you detect right clicks.
"right" in p

Using p["right"] checks if the associated value is true, not if the index exists.
Feb 4 2012, 9:53 am
In response to Kaiochao
Did you even play the demo? Right clicking is fine. Its the occasional Click() is lost from time to time.
Feb 4 2012, 10:18 am
MOUSEDOWN IS GOD! So much better than click. Thanks Kaio.
Feb 4 2012, 1:50 pm (Edited on Feb 4 2012, 2:23 pm)
Wait what? When did anyone even mention MouseDown?
Anyway, the problem with Click(s) missing, is probably that when you DblClick(), it doesn't call Click() again.
Feb 4 2012, 2:17 pm
In response to Falacy
Falacy wrote:
Wait what? When did anyone even mention MouseDown?
Anyway, the problem with Click(s) missing, is probably that when you DblClick(), it doesn't call Click() again.

Thanks for the explaination on the reasoning behind it and kaio helped me on msn with it so I figured I'd just write it down on the forums for anyone's future usage and give him the credit he deserves.
Feb 4 2012, 2:20 pm
DblClick() does actually call Click() both times.

I mentioned MouseDown() to Tubutas through MSN, because there's no chance the Click() can be converted into MouseDrag/Drop() if you're moving and the mouse moves at all inside the click.
Feb 4 2012, 2:22 pm
In response to Kaiochao
Kaiochao wrote:
DblClick() does actually call Click() both times.

So it does
Feb 4 2012, 2:22 pm
In response to Kaiochao
Kaiochao wrote:
DblClick() does actually call Click() both times.

I mentioned MouseDown() to Tubutas through MSN, because there's no chance the Click() can be converted into MouseDrag/Drop() if you're moving and the mouse moves at all inside the click.

I still highly doubt that that was the problem because I was I did a test with a macro program and it was still losing clicks.

Unless of course the client itself was lagging, but I also doubt that because there's nothing else running in that demo.