They are n-channel FETs and need to go between GND and the cathodes of the LEDs. At the moment they are between 4.3 V and the anodes of the LEDs, effectively blocking current to them.