Help with new script

I'm a total scripting noob. All I want to do is for the script to see if a color is in a certain area, then click that area, then do it again for a second area. Basically just an auto login for a private server.

I've put one together using the findcolor method, and even if the color isn't there at the time, it still clicks on the area. I just don't get it.

What am I doing wrong?

 

Remove your loop (repeat - until(false)) and see how far it goes.