Problem with millis() timing

[christop]

@Idahowalker

Especially if using a Uno, try to save memory use space.

No need for a variable that is used once and thrown away with each iteration end, then is created again at the next iteration.

if (digitalRead(button) == LOW) {
ledState1 = true;
}

I can almost guarantee you won't see any difference in object code size by using an extra automatic variable (buttonState1) here. The compiler is very good at optimizing the generated code (sometimes in ways that surprise me, and I've been programming in C for 25 years).