Problems with 2 post button

Hello everyone. So im pretty new to using Arduino. Im working on a project that uses quite a few different components and thought it might be best to build smaller circuits and experiment with each component individually before I try to combine them all together.
I got most everything dialed in and working except for a button that is really giving me a hard time and I thought maybe someone here could help me out.

Bare with me cause I dont have an actual sketch of my circuit but ill try to type it out to make sense. I am using a 2 post button and as far as I can tell it doesnt matter which is positive and which is negative so I will call them pushbutton A and B

Arduino pin #3 -> pushbutton A and 1k resistor -> Led1 + -> Led1 - and Pushbutton B -> ground

Arduino pin #8 -> 1k resistor -> Led2 + -> Led2 - -> ground

Ill attach a picture and hopefully it will make sense

Pin #3 is configured as an input pin
Pin #8 is configured as an output pin

Going off of advice from a different post on this forum I am using the built in pullup resistor on arduino Pin #3
Led1 is lit up and turns off when the button is pressed which is what I want.

Led2 should be programmed to turn on after the button is pressed and that is where im having troubles. No matter what Led2 is always on ive tried changing the code for the button between HIGH and LOW and it makes no difference on LED 2.

Is this a problem with my code or the actual circuit itself?

I cant figure out how to upload my code so ill just type it out here.

int buttonPin1 = 3;
int ledPin = 8;

void setup() {

pinMode (buttonPin1, INPUT);
pinMode (ledPin, OUTPUT);
digitalWrite (buttonPin1, HIGH);
digitalWrite (ledPin, LOW)
}

void loop() {

if (digitalRead(buttonPin1) == LOW) {
digitalWrite (ledPin, HIGH)
}
}

Any help would be very appreciated and I hope you guys can make sense of everything.

Thanks,
John

Nevermind I think I figured it out

I added an Else statement to digitalWrite the Led pin to low and it seems to have solved my problem