Not that it matters much here, but VBA expects an explicit declaration for each variable. So in the code:
Dim Beige, Brown, Chartreuse, DarkBlue, DarkGreen As Long
Only DarkGreen is declared as long - the others are undeclared and will be assigned Variant by default.
Catches everyone out as...