Actually, I think the former implementation was easier to comprehend (it was very clear that the colors were created only once the display is available. This is still true now, but it's less obvious).

I think we should instantiate the colors in ``createInterpreterPartControl``, just like it used to be (and also dispose them in ``dispose``).