Controlling RGB LED with two buttons.

Projects General Guidance
13 
if (digitalRead(buttonPin1)== LOW);

Yeah, that will work great. Of course, it'd be even better without the ; on the end...