simplify palette argument

[casperhart]

Rather than removing the palette argument entirely, I think we could restrict it to being a function, and not allow a vector of colours to be used. That way we can simplify the logic significantly, and users can use tour_aes(colour = I(...)) if they need more granular control.