move color generation to new funcion