4 leds and a button

millis() returns an unsigned long, not an int

Why do you have stuff not depending on time in the if?