the right direction, but not working