Merging multiple Worksheet Change events into one Worksheet_Change, Hide Cols via Yes/No

[xlyfe]

I'm having trouble merging these various worksheet change events into one Worksheet_Change. It's very basic. It's supposed to hide a group of columns if a cell has a yes or no in it. It's not working for me tho. Anybody know what I'm doing wrong here?

Please Login or Register  to view this content.

[Kaper]

I'd expect such code would do what you want (do not unhide unnecesserily too many columns):

Please Login or Register  to view this content.

[xlyfe]

I'd expect such code would do what you want (do not unhide unnecesserily too many columns):

Please Login or Register  to view this content.

That did it! This is the first time I've tried to combine this many change events into one. I didn't think of adding the sub If statements within each event.

Thank you so much Kaper! Another lesson learned on this never ending journey of Excel knowledge. The older I get, the more I fall in love with this program.

May your life be filled with a million good karma's!

Take Care!
Mike from California

[Arkadi]

FWIW,

Please Login or Register  to view this content.

[xlyfe]

FWIW,

Please Login or Register  to view this content.

Thank you for your reply Arkadi. Your method seems much easier for me to recreate in the future. It's amazing to me how there are basically 99 different ways to approach every problem in excel with VBA.

[Arkadi]

Indeed.... truth is there is no need for the Select Case version over another approach, it just seemed cleaner and I was redoing most of it anyway. The real point of my post was the removal of selecting ranges/columns and then hiding the selection. Unless you truly need the range selected (which is not usually the case), it is much more efficient just to address the range directly. All the selects would only slow down the code (Which is not an issue here in a short macro, but larger tasks make it really noticeable).

Happy coding

p.s. Thanks for the rep!