i don't see that. i see it flickering (200ms) repeatedly without pressing a button.
button switches are normally connected between the pin and ground and the pin configured as INPUT_PULLUP so that there is an internal pullup resistor pulling the pin HIGH. pressing the button connects it to ground making the pin LOW
you also have two conditionals toggling the LED. shouldn't there just be one?