Reduce Redundant Code Help Please

[Hummuna]

Hello I was wondering if someone could help me reduce redundant code. My code is for a drop down menu to navagite through tabs, each tab is to have it's own ActiveX Combobox. My problem is I don't want to write the code to populate that list 160 times for each combobox on each sheet.

Here is what I got so far for 10 tabs

Please Login or Register  to view this content.

[JosephP]

maybe

Please Login or Register  to view this content.

[Hummuna]

Ok I am very new so bare with me.

My "cmbSheet" is what I named my Activex Combobox

How would I add what you wrote to my code? I got an Object Defined error.

[JosephP]

my code replaces yours completely. if you get an error I need to know on which line.
if it helps my code assumes there is a combobox called cmbSheet on every sheet in the workbook.

[Hummuna]

Oh I think i have it, it is working but I think because I don't have all my sheets with the combobox yet when the loop gets to a sheet that doesn't have one I get the error

Run time 1004

Application defined or object-defined error

The code does work for the sheets I have my combobox on.


Will get back to you, THANK YOU SO MUCH!

[Hummuna]

I am getting this error upon opening, code seems to work but I am unsure why I am getting the error.

I highlighted the code that it says in error.

Run Time Error 1004

Unable to get the OLEObjects propertyn of the Worksheet class

Please Login or Register  to view this content.

THis is the code i have on the worksheet

Please Login or Register  to view this content.

[JosephP]

are you sure all of your sheets have a control with that name on them?