fixed coloro function