Closed RaymondWise closed 9 years ago
Looks good. I will merge it in. Couple of things I see to comment on:
Lock
and UnlockAllSheetswere changed to
ThisWorkbook.Sheets. Does this work? Or should it be
ActiveWorkbook.Sheets? I always thought
ThisWorkbook` referred to the workbook where the code was running regardless of what was active. I will test it to verify.Colorize
ranges.RangeEnd
and RangeEnd_Boundary
are called anywhere, but the idea is that they provide helper functions when you want to form a Range
using End
. They exist to replace code like Set rng = Range(Range("A1").Offset(1,1), Range("A1").Offset(1,1).End(xlDown))
. The idea is that you only have to mention the Range
once which is helpful if it is complicated. The Boundary
is supposed to do the same thing while also restricting the final Range
to stay within the Boundary of the CurrentRegion
. There was some edge case where I wanted to use RangeEnd
but knew it was possible for it to only be a single row tall which breaks End
so then it does the Intersect
to keep it from going to the edge of the Worksheet
. The Boundary
one is probably not that useful generally. They are not intended to be UDFs though (or maybe they could be used as a UDF since they return a Range
?). I guess the redundant typing
comment on this Function
wasn't helpful? :)
More of the same - variable renaming and some simplification - not much done with charts as they aren't my specialty.
Noted a few more "static" folders that you may want to remove.