Walking looks a bit tricky.. And failsafes, pweeaze?
repeat
FindColor(x,y,KebabColor,mmx1,mmy1,mmx2,mmy2);
Wait(502+random(120));
Mouse(x,y,1,1,true);
until (FindColor(x,y,KebabColor,mmx1,mmy1,mmx2,mmy2));
What if it never finds it? Then it'll just go on and on and on.. Plus findcolor isn't that good, I suggest FindColorTolerance or FindColorSpiralTolerance (sorry if i spelt one of the two wrong)… And that goes for other parts in the script too, not just this procedure.
Now after reading it more I noticed other things…
procedure DepositLeather;
begin
Mouse(620,228,5,5,true);
Mouse(665,227,5,5,false);
ClickOption('All', 2);
Wait(1000+random(150));
end;
I suggest first finding a leather in the inventory with for example a DTM, as what if there'd be for example a item from a random event in that spot, it wouldn't bank the leathers… And just generally coordinate clicking is not good (goes to other places in the script too).
The TerminateScript; in the end isn't really needed as it'll end anyways. And once again, failsafes. I also suggest adding multiplayer to the script.
Quite good for a first script, but it still lacks some things.