Issues with millis timing with a button press as well as seven segment display

[LarryD]

BTW

@rabiesherpes

    case State1:
      Serial.println("hi there 2");
      if (millis() - previousTime >= 2000)
      {
        analogWrite(red, 255);
        digitalWrite(green, 255);  
        Serial.println ("State 2  Button: Not Pressed  Traffic Light: Green  Pedestrian Light: Red");
        previousTime = millis();
        state = State2;
      }

digitalWrite(green, 255);
is this the proper syntax ?