Ted, There is nothing wrong with his code style, but I also suspect this piece should be
If it.Interior.Color = X Then it.EntireRow.Hidden = Not it.Hidden
In order to Toggle visibility.

IMO, we should be expose our guests to many diverse styles. I always hope that between my style of verbosity and snb's terseness, we have encompassed the entire spectrum.

The good and the bad of VBA is that the Compiler will digest the beautiful and the ugly and turn both into pretty darn efficient code.

I would really hate to have to parse one of his large projects. His style reminds me of an olde coders drinking game: Write very terse (working) code with obscure functions and see who could figure out what it did. I bet he won many free drinks.