To start with .... As I read it you should be refering to the
passed combo box in the procedure setcolor instead of trying to look at
MyComboBox1 all the time. SetColour doesn't know the object MyComboBox1 from
Eve!!

To refer to the combo box then can we assume you are in the combo box


In sub SetColour try refering to cbox.BackColor and so on and see if that
helps.