lets make a menu instead
refactoring etc