still not working how I want it to