colour logic is about where I want it