user input needs to be fixed