For a first Script it is quite good, well done

.
But why do you do:
[SCAR]y +random(0)[/SCAR]
You could just leave it y.
You also have to consider if the Bot doesn't find a Color, E.g because of lagg.
[SCAR]if(FindColor(....)) then
begin
//found it do stuff...
end else
begin
//could not find that color!
//lets wait or do anything to fix...
end;
[/SCAR]
Thats just a suggestion, but other than those your Script is good

.