still not working propoerly