Complicated VBA Conditional Formatting

First, the formatting will only highlight certain cells in a column when the
column header in Row 3 is the current month which is tied to A2 which is
=Text(now(),"mmm"). Second, only those cells in rows in which Column A has
four digit letters or numbers will be highlighted. What I need is if, for
example, "Col F Row 9" / "Col F Row 5" = 200% then the cell background will
be shaded bright green. Row 5 is static and used as the divisor for all. The
conditions are: if the percent is 100.1% or greater then background of cell
is bright green, if percent is 90-100% then light green, if percent is
80-89.9% than light yellow, if percent is 70-79.9% than pink, if 1-69.9% then
purple, if percent is 0 or blank then red. The columns that contain the data
are columns F-P. Is there any way to automate this process. Thanks in
advance. I am very new to VBA.

Col F Col G Col H Col I Col J Col k Col L
Row 3 Jan Feb Mar Apr May Jun Jly

Row 5 $500 $500 $500 $750 $750 $750 $1,000

Row 9 $1000 $750 $500 $0 $1500 $900 $500
Many Rows just like Row 9

Hi,
Try this: insert this code into a general module.

I have defined the cell A2 as a named range called "CurMonth" and the month
headers as a range "HdrMonths".

Sub CFormat()
'
Dim rng As Range, cell As Range
Dim ncol As Integer, lrow As Long
Dim pcnt As Double, divisor As Double

Thisworkbook.Worksheets("Sheet1").activate <=== Change to your w/sheet

' Find column for current Month (add 5 to start in colum F onwards)
ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5

' Find last row of data in current month column
lrow = Cells(Rows.Count, ncol).End(xlUp).Row

' Set range to cells for current month starting row 9
Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))

' Set Divisor for current month
divisor = Cells(5, ncol)

' Loop through all cells in range
For Each cell In rng
' Calculate perecentage
pcnt = (cell / divisor) * 100
cell.Select
' Set colorindex based on percentage
Select Case pcnt
Case Is > 100
Selection.Interior.ColorIndex = 4
Case Is >= 90
Selection.Interior.ColorIndex = 35
Case Is >= 80
Selection.Interior.ColorIndex = 36
Case Is >= 70
Selection.Interior.ColorIndex = 7
Case Is >= 1
Selection.Interior.ColorIndex = 54
Case Else
Selection.Interior.ColorIndex = 3
End Select
Next cell

End Sub


HTH

"Leslie" wrote:

> First, the formatting will only highlight certain cells in a column when the
> column header in Row 3 is the current month which is tied to A2 which is
> =Text(now(),"mmm"). Second, only those cells in rows in which Column A has
> four digit letters or numbers will be highlighted. What I need is if, for
> example, "Col F Row 9" / "Col F Row 5" = 200% then the cell background will
> be shaded bright green. Row 5 is static and used as the divisor for all. The
> conditions are: if the percent is 100.1% or greater then background of cell
> is bright green, if percent is 90-100% then light green, if percent is
> 80-89.9% than light yellow, if percent is 70-79.9% than pink, if 1-69.9% then
> purple, if percent is 0 or blank then red. The columns that contain the data
> are columns F-P. Is there any way to automate this process. Thanks in
> advance. I am very new to VBA.
>
> Col F Col G Col H Col I Col J Col k Col L
> Row 3 Jan Feb Mar Apr May Jun Jly
>
> Row 5 $500 $500 $500 $750 $750 $750 $1,000
>
> Row 9 $1000 $750 $500 $0 $1500 $900 $500
> Many Rows just like Row 9
>

Thanks very much. I inserted the code but rec'd the error "Run-Time error
1004 Application defined or object defined error" Any ideas. Thanks again
for your help in figuring this out.

"Toppers" wrote:

> Hi,
> Try this: insert this code into a general module.
>
> I have defined the cell A2 as a named range called "CurMonth" and the month
> headers as a range "HdrMonths".
>
> Sub CFormat()
> '
> Dim rng As Range, cell As Range
> Dim ncol As Integer, lrow As Long
> Dim pcnt As Double, divisor As Double
>
> Thisworkbook.Worksheets("Sheet1").activate <=== Change to your w/sheet
>
> ' Find column for current Month (add 5 to start in colum F onwards)
> ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
>
> ' Find last row of data in current month column
> lrow = Cells(Rows.Count, ncol).End(xlUp).Row
>
> ' Set range to cells for current month starting row 9
> Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
>
> ' Set Divisor for current month
> divisor = Cells(5, ncol)
>
> ' Loop through all cells in range
> For Each cell In rng
> ' Calculate perecentage
> pcnt = (cell / divisor) * 100
> cell.Select
> ' Set colorindex based on percentage
> Select Case pcnt
> Case Is > 100
> Selection.Interior.ColorIndex = 4
> Case Is >= 90
> Selection.Interior.ColorIndex = 35
> Case Is >= 80
> Selection.Interior.ColorIndex = 36
> Case Is >= 70
> Selection.Interior.ColorIndex = 7
> Case Is >= 1
> Selection.Interior.ColorIndex = 54
> Case Else
> Selection.Interior.ColorIndex = 3
> End Select
> Next cell
>
> End Sub
>
>
> HTH
>
> "Leslie" wrote:
>
> > First, the formatting will only highlight certain cells in a column when the
> > column header in Row 3 is the current month which is tied to A2 which is
> > =Text(now(),"mmm"). Second, only those cells in rows in which Column A has
> > four digit letters or numbers will be highlighted. What I need is if, for
> > example, "Col F Row 9" / "Col F Row 5" = 200% then the cell background will
> > be shaded bright green. Row 5 is static and used as the divisor for all. The
> > conditions are: if the percent is 100.1% or greater then background of cell
> > is bright green, if percent is 90-100% then light green, if percent is
> > 80-89.9% than light yellow, if percent is 70-79.9% than pink, if 1-69.9% then
> > purple, if percent is 0 or blank then red. The columns that contain the data
> > are columns F-P. Is there any way to automate this process. Thanks in
> > advance. I am very new to VBA.
> >
> > Col F Col G Col H Col I Col J Col k Col L
> > Row 3 Jan Feb Mar Apr May Jun Jly
> >
> > Row 5 $500 $500 $500 $750 $750 $750 $1,000
> >
> > Row 9 $1000 $750 $500 $0 $1500 $900 $500
> > Many Rows just like Row 9
> >

Ignore my last note. I pasted it to the wrong worksheet. I pasted it into
the correct worksheet and it is giving me a message "Run-Time error 9,
subscript out of range. Any ideas? Thanks again.

"Toppers" wrote:

> Hi,
> Try this: insert this code into a general module.
>
> I have defined the cell A2 as a named range called "CurMonth" and the month
> headers as a range "HdrMonths".
>
> Sub CFormat()
> '
> Dim rng As Range, cell As Range
> Dim ncol As Integer, lrow As Long
> Dim pcnt As Double, divisor As Double
>
> Thisworkbook.Worksheets("Sheet1").activate <=== Change to your w/sheet
>
> ' Find column for current Month (add 5 to start in colum F onwards)
> ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
>
> ' Find last row of data in current month column
> lrow = Cells(Rows.Count, ncol).End(xlUp).Row
>
> ' Set range to cells for current month starting row 9
> Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
>
> ' Set Divisor for current month
> divisor = Cells(5, ncol)
>
> ' Loop through all cells in range
> For Each cell In rng
> ' Calculate perecentage
> pcnt = (cell / divisor) * 100
> cell.Select
> ' Set colorindex based on percentage
> Select Case pcnt
> Case Is > 100
> Selection.Interior.ColorIndex = 4
> Case Is >= 90
> Selection.Interior.ColorIndex = 35
> Case Is >= 80
> Selection.Interior.ColorIndex = 36
> Case Is >= 70
> Selection.Interior.ColorIndex = 7
> Case Is >= 1
> Selection.Interior.ColorIndex = 54
> Case Else
> Selection.Interior.ColorIndex = 3
> End Select
> Next cell
>
> End Sub
>
>
> HTH
>
> "Leslie" wrote:
>
> > First, the formatting will only highlight certain cells in a column when the
> > column header in Row 3 is the current month which is tied to A2 which is
> > =Text(now(),"mmm"). Second, only those cells in rows in which Column A has
> > four digit letters or numbers will be highlighted. What I need is if, for
> > example, "Col F Row 9" / "Col F Row 5" = 200% then the cell background will
> > be shaded bright green. Row 5 is static and used as the divisor for all. The
> > conditions are: if the percent is 100.1% or greater then background of cell
> > is bright green, if percent is 90-100% then light green, if percent is
> > 80-89.9% than light yellow, if percent is 70-79.9% than pink, if 1-69.9% then
> > purple, if percent is 0 or blank then red. The columns that contain the data
> > are columns F-P. Is there any way to automate this process. Thanks in
> > advance. I am very new to VBA.
> >
> > Col F Col G Col H Col I Col J Col k Col L
> > Row 3 Jan Feb Mar Apr May Jun Jly
> >
> > Row 5 $500 $500 $500 $750 $750 $750 $1,000
> >
> > Row 9 $1000 $750 $500 $0 $1500 $900 $500
> > Many Rows just like Row 9
> >

Okay, I've confused myself. The run-time error is 1004 after all. I think
it is stuck at:

ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5

but since I am new to this VBA stuff I most likely am wrong. Does it have
something to do with "ncol". Thanks.

"Toppers" wrote:

> Hi,
> Try this: insert this code into a general module.
>
> I have defined the cell A2 as a named range called "CurMonth" and the month
> headers as a range "HdrMonths".
>
> Sub CFormat()
> '
> Dim rng As Range, cell As Range
> Dim ncol As Integer, lrow As Long
> Dim pcnt As Double, divisor As Double
>
> Thisworkbook.Worksheets("Sheet1").activate <=== Change to your w/sheet
>
> ' Find column for current Month (add 5 to start in colum F onwards)
> ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
>
> ' Find last row of data in current month column
> lrow = Cells(Rows.Count, ncol).End(xlUp).Row
>
> ' Set range to cells for current month starting row 9
> Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
>
> ' Set Divisor for current month
> divisor = Cells(5, ncol)
>
> ' Loop through all cells in range
> For Each cell In rng
> ' Calculate perecentage
> pcnt = (cell / divisor) * 100
> cell.Select
> ' Set colorindex based on percentage
> Select Case pcnt
> Case Is > 100
> Selection.Interior.ColorIndex = 4
> Case Is >= 90
> Selection.Interior.ColorIndex = 35
> Case Is >= 80
> Selection.Interior.ColorIndex = 36
> Case Is >= 70
> Selection.Interior.ColorIndex = 7
> Case Is >= 1
> Selection.Interior.ColorIndex = 54
> Case Else
> Selection.Interior.ColorIndex = 3
> End Select
> Next cell
>
> End Sub
>
>
> HTH
>
> "Leslie" wrote:
>
> > First, the formatting will only highlight certain cells in a column when the
> > column header in Row 3 is the current month which is tied to A2 which is
> > =Text(now(),"mmm"). Second, only those cells in rows in which Column A has
> > four digit letters or numbers will be highlighted. What I need is if, for
> > example, "Col F Row 9" / "Col F Row 5" = 200% then the cell background will
> > be shaded bright green. Row 5 is static and used as the divisor for all. The
> > conditions are: if the percent is 100.1% or greater then background of cell
> > is bright green, if percent is 90-100% then light green, if percent is
> > 80-89.9% than light yellow, if percent is 70-79.9% than pink, if 1-69.9% then
> > purple, if percent is 0 or blank then red. The columns that contain the data
> > are columns F-P. Is there any way to automate this process. Thanks in
> > advance. I am very new to VBA.
> >
> > Col F Col G Col H Col I Col J Col k Col L
> > Row 3 Jan Feb Mar Apr May Jun Jly
> >
> > Row 5 $500 $500 $500 $750 $750 $750 $1,000
> >
> > Row 9 $1000 $750 $500 $0 $1500 $900 $500
> > Many Rows just like Row 9
> >

Hi,
These are the "named" ranges I mentioned. You can try changing to:

ncol=Application.Match(Range("a2"),Range("F3:Q3"),0)+5

This avoids using named ranges.

On re-reading your note I didn't check colum A for a 4 digits/Letters; Try
the above a get the code working and I will look at adding the required test.
Can I simply test that column A has field of length 4, irrespctive of whether
it is numbers/letters or amixture?

HTH

"Leslie" wrote:

> Okay, I've confused myself. The run-time error is 1004 after all. I think
> it is stuck at:
>
> ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
>
> but since I am new to this VBA stuff I most likely am wrong. Does it have
> something to do with "ncol". Thanks.
>
> "Toppers" wrote:
>
> > Hi,
> > Try this: insert this code into a general module.
> >
> > I have defined the cell A2 as a named range called "CurMonth" and the month
> > headers as a range "HdrMonths".
> >
> > Sub CFormat()
> > '
> > Dim rng As Range, cell As Range
> > Dim ncol As Integer, lrow As Long
> > Dim pcnt As Double, divisor As Double
> >
> > Thisworkbook.Worksheets("Sheet1").activate <=== Change to your w/sheet
> >
> > ' Find column for current Month (add 5 to start in colum F onwards)
> > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> >
> > ' Find last row of data in current month column
> > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> >
> > ' Set range to cells for current month starting row 9
> > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> >
> > ' Set Divisor for current month
> > divisor = Cells(5, ncol)
> >
> > ' Loop through all cells in range
> > For Each cell In rng
> > ' Calculate perecentage
> > pcnt = (cell / divisor) * 100
> > cell.Select
> > ' Set colorindex based on percentage
> > Select Case pcnt
> > Case Is > 100
> > Selection.Interior.ColorIndex = 4
> > Case Is >= 90
> > Selection.Interior.ColorIndex = 35
> > Case Is >= 80
> > Selection.Interior.ColorIndex = 36
> > Case Is >= 70
> > Selection.Interior.ColorIndex = 7
> > Case Is >= 1
> > Selection.Interior.ColorIndex = 54
> > Case Else
> > Selection.Interior.ColorIndex = 3
> > End Select
> > Next cell
> >
> > End Sub
> >
> >
> > HTH
> >
> > "Leslie" wrote:
> >
> > > First, the formatting will only highlight certain cells in a column when the
> > > column header in Row 3 is the current month which is tied to A2 which is
> > > =Text(now(),"mmm"). Second, only those cells in rows in which Column A has
> > > four digit letters or numbers will be highlighted. What I need is if, for
> > > example, "Col F Row 9" / "Col F Row 5" = 200% then the cell background will
> > > be shaded bright green. Row 5 is static and used as the divisor for all. The
> > > conditions are: if the percent is 100.1% or greater then background of cell
> > > is bright green, if percent is 90-100% then light green, if percent is
> > > 80-89.9% than light yellow, if percent is 70-79.9% than pink, if 1-69.9% then
> > > purple, if percent is 0 or blank then red. The columns that contain the data
> > > are columns F-P. Is there any way to automate this process. Thanks in
> > > advance. I am very new to VBA.
> > >
> > > Col F Col G Col H Col I Col J Col k Col L
> > > Row 3 Jan Feb Mar Apr May Jun Jly
> > >
> > > Row 5 $500 $500 $500 $750 $750 $750 $1,000
> > >
> > > Row 9 $1000 $750 $500 $0 $1500 $900 $500
> > > Many Rows just like Row 9
> > >

Re: Col. A yes for field length of four. Also, I defined the Ranges so I
think that is okay now but I am getting a Type Mismatch error 13. Any ideas?
Thanks. Here is my code:

Sub CFormat()

Dim rng As Range, cell As Range
Dim ncol As Integer, lrow As Long
Dim pcnt As Double, divisor As Double

'Ace is sheet name
ThisWorkbook.Worksheets("Ace").Activate

' Find column for current Month (add 5 to start in colum F onwards)
ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5

' Find last row of data in current month column
lrow = Cells(Rows.Count, ncol).End(xlUp).Row

' Set range to cells for current month starting row 9
Set rng = Range(Cells(20, ncol), Cells(lrow, ncol))

' Set Divisor for current month
divisor = Cells(5, ncol)

' Loop through all cells in range
For Each cell In rng
' Calculate percentage
pcnt = (cell / divisor) * 100
cell.Select
' Set colorindex based on percentage
Select Case pcnt
Case Is > 100
Selection.Interior.ColorIndex = 4
Case Is >= 90
Selection.Interior.ColorIndex = 35
Case Is >= 80
Selection.Interior.ColorIndex = 36
Case Is >= 70
Selection.Interior.ColorIndex = 7
Case Is >= 1
Selection.Interior.ColorIndex = 54
Case Else
Selection.Interior.ColorIndex = 3
End Select
Next cell

End Sub


"Toppers" wrote:

> Hi,
> These are the "named" ranges I mentioned. You can try changing to:
>
> ncol=Application.Match(Range("a2"),Range("F3:Q3"),0)+5
>
> This avoids using named ranges.
>
> On re-reading your note I didn't check colum A for a 4 digits/Letters; Try
> the above a get the code working and I will look at adding the required test.
> Can I simply test that column A has field of length 4, irrespctive of whether
> it is numbers/letters or amixture?
>
> HTH
>
> "Leslie" wrote:
>
> > Okay, I've confused myself. The run-time error is 1004 after all. I think
> > it is stuck at:
> >
> > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> >
> > but since I am new to this VBA stuff I most likely am wrong. Does it have
> > something to do with "ncol". Thanks.
> >
> > "Toppers" wrote:
> >
> > > Hi,
> > > Try this: insert this code into a general module.
> > >
> > > I have defined the cell A2 as a named range called "CurMonth" and the month
> > > headers as a range "HdrMonths".
> > >
> > > Sub CFormat()
> > > '
> > > Dim rng As Range, cell As Range
> > > Dim ncol As Integer, lrow As Long
> > > Dim pcnt As Double, divisor As Double
> > >
> > > Thisworkbook.Worksheets("Sheet1").activate <=== Change to your w/sheet
> > >
> > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > >
> > > ' Find last row of data in current month column
> > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > >
> > > ' Set range to cells for current month starting row 9
> > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > >
> > > ' Set Divisor for current month
> > > divisor = Cells(5, ncol)
> > >
> > > ' Loop through all cells in range
> > > For Each cell In rng
> > > ' Calculate perecentage
> > > pcnt = (cell / divisor) * 100
> > > cell.Select
> > > ' Set colorindex based on percentage
> > > Select Case pcnt
> > > Case Is > 100
> > > Selection.Interior.ColorIndex = 4
> > > Case Is >= 90
> > > Selection.Interior.ColorIndex = 35
> > > Case Is >= 80
> > > Selection.Interior.ColorIndex = 36
> > > Case Is >= 70
> > > Selection.Interior.ColorIndex = 7
> > > Case Is >= 1
> > > Selection.Interior.ColorIndex = 54
> > > Case Else
> > > Selection.Interior.ColorIndex = 3
> > > End Select
> > > Next cell
> > >
> > > End Sub
> > >
> > >
> > > HTH
> > >
> > > "Leslie" wrote:
> > >
> > > > First, the formatting will only highlight certain cells in a column when the
> > > > column header in Row 3 is the current month which is tied to A2 which is
> > > > =Text(now(),"mmm"). Second, only those cells in rows in which Column A has
> > > > four digit letters or numbers will be highlighted. What I need is if, for
> > > > example, "Col F Row 9" / "Col F Row 5" = 200% then the cell background will
> > > > be shaded bright green. Row 5 is static and used as the divisor for all. The
> > > > conditions are: if the percent is 100.1% or greater then background of cell
> > > > is bright green, if percent is 90-100% then light green, if percent is
> > > > 80-89.9% than light yellow, if percent is 70-79.9% than pink, if 1-69.9% then
> > > > purple, if percent is 0 or blank then red. The columns that contain the data
> > > > are columns F-P. Is there any way to automate this process. Thanks in
> > > > advance. I am very new to VBA.
> > > >
> > > > Col F Col G Col H Col I Col J Col k Col L
> > > > Row 3 Jan Feb Mar Apr May Jun Jly
> > > >
> > > > Row 5 $500 $500 $500 $750 $750 $750 $1,000
> > > >
> > > > Row 9 $1000 $750 $500 $0 $1500 $900 $500
> > > > Many Rows just like Row 9
> > > >

I am still confused as I thought we were dividing by row 5 which I wouldn't
expect to be zero. However, I appreciate the data in row 9 (or 20) onwards
could have blanks and that does cause the type mismatch error. Again I have
put the test for column A on the data in row 9 onwards not on row 5!


Hopefully you can now sort it out.


"Leslie" wrote:

> It is stopping when it hits a #DIV/0 but once we have column A included in
> the code then that won't happen but there will be blanks.
>
> "Toppers" wrote:
>
> > Hi again,
> > You will get your error message if the cell is blank so I
> > have added a test for cell value being a number.
> >
> > HTH
> >
> > Sub CFormat()
> > '
> > Dim rng As Range, cell As Range
> > Dim ncol As Integer, lrow As Long
> > Dim pcnt As Double, divisor As Double
> >
> > ThisWorkbook.Worksheets("Sheet1").Activate ' <=== Change to your w/sheet
> > ' Find column for current Month (add 5 to start in colum F onwards)
> > ncol = Application.Match(Range("A2"), Range("F3:q3"), 0) + 5
> >
> > ' Find last row of data in current month column
> > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> >
> > ' Set range to cells for current month starting row 9
> > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> >
> > ' Set Divisor for current month
> > divisor = Cells(5, ncol)
> >
> > ' Loop through all cells in range
> > For Each cell In rng
> > ' Check length of cell in column A
> > If Len(cell.Offset(0, -(ncol - 1))) = 4 Then
> > ' Calculate perecentage
> > If Application.IsNumber(cell) Then ' Is this cell a number ?
> > pcnt = (cell / divisor) * 100
> > cell.Select
> > ' Set colorindex based on percentage
> > Select Case pcnt
> > Case Is > 100
> > Selection.Interior.ColorIndex = 4
> > Case Is >= 90
> > Selection.Interior.ColorIndex = 35
> > Case Is >= 80
> > Selection.Interior.ColorIndex = 36
> > Case Is >= 70
> > Selection.Interior.ColorIndex = 7
> > Case Is >= 1
> > Selection.Interior.ColorIndex = 54
> > Case Else
> > Selection.Interior.ColorIndex = 3
> > End Select
> > End If
> > End If
> > Next cell
> >
> > End Sub
> >
> >
> > "Leslie" wrote:
> >
> > > Okay, I've confused myself. The run-time error is 1004 after all. I think
> > > it is stuck at:
> > >
> > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > >
> > > but since I am new to this VBA stuff I most likely am wrong. Does it have
> > > something to do with "ncol". Thanks.
> > >
> > > "Toppers" wrote:
> > >
> > > > Hi,
> > > > Try this: insert this code into a general module.
> > > >
> > > > I have defined the cell A2 as a named range called "CurMonth" and the month
> > > > headers as a range "HdrMonths".
> > > >
> > > > Sub CFormat()
> > > > '
> > > > Dim rng As Range, cell As Range
> > > > Dim ncol As Integer, lrow As Long
> > > > Dim pcnt As Double, divisor As Double
> > > >
> > > > Thisworkbook.Worksheets("Sheet1").activate <=== Change to your w/sheet
> > > >
> > > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > >
> > > > ' Find last row of data in current month column
> > > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > > >
> > > > ' Set range to cells for current month starting row 9
> > > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > > >
> > > > ' Set Divisor for current month
> > > > divisor = Cells(5, ncol)
> > > >
> > > > ' Loop through all cells in range
> > > > For Each cell In rng
> > > > ' Calculate perecentage
> > > > pcnt = (cell / divisor) * 100
> > > > cell.Select
> > > > ' Set colorindex based on percentage
> > > > Select Case pcnt
> > > > Case Is > 100
> > > > Selection.Interior.ColorIndex = 4
> > > > Case Is >= 90
> > > > Selection.Interior.ColorIndex = 35
> > > > Case Is >= 80
> > > > Selection.Interior.ColorIndex = 36
> > > > Case Is >= 70
> > > > Selection.Interior.ColorIndex = 7
> > > > Case Is >= 1
> > > > Selection.Interior.ColorIndex = 54
> > > > Case Else
> > > > Selection.Interior.ColorIndex = 3
> > > > End Select
> > > > Next cell
> > > >
> > > > End Sub
> > > >
> > > >
> > > > HTH
> > > >
> > > > "Leslie" wrote:
> > > >
> > > > > First, the formatting will only highlight certain cells in a column when the
> > > > > column header in Row 3 is the current month which is tied to A2 which is
> > > > > =Text(now(),"mmm"). Second, only those cells in rows in which Column A has
> > > > > four digit letters or numbers will be highlighted. What I need is if, for
> > > > > example, "Col F Row 9" / "Col F Row 5" = 200% then the cell background will
> > > > > be shaded bright green. Row 5 is static and used as the divisor for all. The
> > > > > conditions are: if the percent is 100.1% or greater then background of cell
> > > > > is bright green, if percent is 90-100% then light green, if percent is
> > > > > 80-89.9% than light yellow, if percent is 70-79.9% than pink, if 1-69.9% then
> > > > > purple, if percent is 0 or blank then red. The columns that contain the data
> > > > > are columns F-P. Is there any way to automate this process. Thanks in
> > > > > advance. I am very new to VBA.
> > > > >
> > > > > Col F Col G Col H Col I Col J Col k Col L
> > > > > Row 3 Jan Feb Mar Apr May Jun Jly
> > > > >
> > > > > Row 5 $500 $500 $500 $750 $750 $750 $1,000
> > > > >
> > > > > Row 9 $1000 $750 $500 $0 $1500 $900 $500
> > > > > Many Rows just like Row 9
> > > > >

Check the END IF are all there! It works OK on my machine!

"Leslie" wrote:

> It doesn't like the "Next Cell" at the bottom of the code. Thanks.
>
> "Toppers" wrote:
>
> > Hi again,
> > You will get your error message if the cell is blank so I
> > have added a test for cell value being a number.
> >
> > HTH
> >
> > Sub CFormat()
> > '
> > Dim rng As Range, cell As Range
> > Dim ncol As Integer, lrow As Long
> > Dim pcnt As Double, divisor As Double
> >
> > ThisWorkbook.Worksheets("Sheet1").Activate ' <=== Change to your w/sheet
> > ' Find column for current Month (add 5 to start in colum F onwards)
> > ncol = Application.Match(Range("A2"), Range("F3:q3"), 0) + 5
> >
> > ' Find last row of data in current month column
> > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> >
> > ' Set range to cells for current month starting row 9
> > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> >
> > ' Set Divisor for current month
> > divisor = Cells(5, ncol)
> >
> > ' Loop through all cells in range
> > For Each cell In rng
> > ' Check length of cell in column A
> > If Len(cell.Offset(0, -(ncol - 1))) = 4 Then
> > ' Calculate perecentage
> > If Application.IsNumber(cell) Then ' Is this cell a number ?
> > pcnt = (cell / divisor) * 100
> > cell.Select
> > ' Set colorindex based on percentage
> > Select Case pcnt
> > Case Is > 100
> > Selection.Interior.ColorIndex = 4
> > Case Is >= 90
> > Selection.Interior.ColorIndex = 35
> > Case Is >= 80
> > Selection.Interior.ColorIndex = 36
> > Case Is >= 70
> > Selection.Interior.ColorIndex = 7
> > Case Is >= 1
> > Selection.Interior.ColorIndex = 54
> > Case Else
> > Selection.Interior.ColorIndex = 3
> > End Select
> > End If
> > End If
> > Next cell
> >
> > End Sub
> >
> >
> > "Leslie" wrote:
> >
> > > Okay, I've confused myself. The run-time error is 1004 after all. I think
> > > it is stuck at:
> > >
> > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > >
> > > but since I am new to this VBA stuff I most likely am wrong. Does it have
> > > something to do with "ncol". Thanks.
> > >
> > > "Toppers" wrote:
> > >
> > > > Hi,
> > > > Try this: insert this code into a general module.
> > > >
> > > > I have defined the cell A2 as a named range called "CurMonth" and the month
> > > > headers as a range "HdrMonths".
> > > >
> > > > Sub CFormat()
> > > > '
> > > > Dim rng As Range, cell As Range
> > > > Dim ncol As Integer, lrow As Long
> > > > Dim pcnt As Double, divisor As Double
> > > >
> > > > Thisworkbook.Worksheets("Sheet1").activate <=== Change to your w/sheet
> > > >
> > > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > >
> > > > ' Find last row of data in current month column
> > > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > > >
> > > > ' Set range to cells for current month starting row 9
> > > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > > >
> > > > ' Set Divisor for current month
> > > > divisor = Cells(5, ncol)
> > > >
> > > > ' Loop through all cells in range
> > > > For Each cell In rng
> > > > ' Calculate perecentage
> > > > pcnt = (cell / divisor) * 100
> > > > cell.Select
> > > > ' Set colorindex based on percentage
> > > > Select Case pcnt
> > > > Case Is > 100
> > > > Selection.Interior.ColorIndex = 4
> > > > Case Is >= 90
> > > > Selection.Interior.ColorIndex = 35
> > > > Case Is >= 80
> > > > Selection.Interior.ColorIndex = 36
> > > > Case Is >= 70
> > > > Selection.Interior.ColorIndex = 7
> > > > Case Is >= 1
> > > > Selection.Interior.ColorIndex = 54
> > > > Case Else
> > > > Selection.Interior.ColorIndex = 3
> > > > End Select
> > > > Next cell
> > > >
> > > > End Sub
> > > >
> > > >
> > > > HTH
> > > >
> > > > "Leslie" wrote:
> > > >
> > > > > First, the formatting will only highlight certain cells in a column when the
> > > > > column header in Row 3 is the current month which is tied to A2 which is
> > > > > =Text(now(),"mmm"). Second, only those cells in rows in which Column A has
> > > > > four digit letters or numbers will be highlighted. What I need is if, for
> > > > > example, "Col F Row 9" / "Col F Row 5" = 200% then the cell background will
> > > > > be shaded bright green. Row 5 is static and used as the divisor for all. The
> > > > > conditions are: if the percent is 100.1% or greater then background of cell
> > > > > is bright green, if percent is 90-100% then light green, if percent is
> > > > > 80-89.9% than light yellow, if percent is 70-79.9% than pink, if 1-69.9% then
> > > > > purple, if percent is 0 or blank then red. The columns that contain the data
> > > > > are columns F-P. Is there any way to automate this process. Thanks in
> > > > > advance. I am very new to VBA.
> > > > >
> > > > > Col F Col G Col H Col I Col J Col k Col L
> > > > > Row 3 Jan Feb Mar Apr May Jun Jly
> > > > >
> > > > > Row 5 $500 $500 $500 $750 $750 $750 $1,000
> > > > >
> > > > > Row 9 $1000 $750 $500 $0 $1500 $900 $500
> > > > > Many Rows just like Row 9
> > > > >

It doesn't like the "Next Cell" at the bottom of the code. Thanks.

"Toppers" wrote:

> Hi again,
> You will get your error message if the cell is blank so I
> have added a test for cell value being a number.
>
> HTH
>
> Sub CFormat()
> '
> Dim rng As Range, cell As Range
> Dim ncol As Integer, lrow As Long
> Dim pcnt As Double, divisor As Double
>
> ThisWorkbook.Worksheets("Sheet1").Activate ' <=== Change to your w/sheet
> ' Find column for current Month (add 5 to start in colum F onwards)
> ncol = Application.Match(Range("A2"), Range("F3:q3"), 0) + 5
>
> ' Find last row of data in current month column
> lrow = Cells(Rows.Count, ncol).End(xlUp).Row
>
> ' Set range to cells for current month starting row 9
> Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
>
> ' Set Divisor for current month
> divisor = Cells(5, ncol)
>
> ' Loop through all cells in range
> For Each cell In rng
> ' Check length of cell in column A
> If Len(cell.Offset(0, -(ncol - 1))) = 4 Then
> ' Calculate perecentage
> If Application.IsNumber(cell) Then ' Is this cell a number ?
> pcnt = (cell / divisor) * 100
> cell.Select
> ' Set colorindex based on percentage
> Select Case pcnt
> Case Is > 100
> Selection.Interior.ColorIndex = 4
> Case Is >= 90
> Selection.Interior.ColorIndex = 35
> Case Is >= 80
> Selection.Interior.ColorIndex = 36
> Case Is >= 70
> Selection.Interior.ColorIndex = 7
> Case Is >= 1
> Selection.Interior.ColorIndex = 54
> Case Else
> Selection.Interior.ColorIndex = 3
> End Select
> End If
> End If
> Next cell
>
> End Sub
>
>
> "Leslie" wrote:
>
> > Okay, I've confused myself. The run-time error is 1004 after all. I think
> > it is stuck at:
> >
> > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> >
> > but since I am new to this VBA stuff I most likely am wrong. Does it have
> > something to do with "ncol". Thanks.
> >
> > "Toppers" wrote:
> >
> > > Hi,
> > > Try this: insert this code into a general module.
> > >
> > > I have defined the cell A2 as a named range called "CurMonth" and the month
> > > headers as a range "HdrMonths".
> > >
> > > Sub CFormat()
> > > '
> > > Dim rng As Range, cell As Range
> > > Dim ncol As Integer, lrow As Long
> > > Dim pcnt As Double, divisor As Double
> > >
> > > Thisworkbook.Worksheets("Sheet1").activate <=== Change to your w/sheet
> > >
> > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > >
> > > ' Find last row of data in current month column
> > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > >
> > > ' Set range to cells for current month starting row 9
> > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > >
> > > ' Set Divisor for current month
> > > divisor = Cells(5, ncol)
> > >
> > > ' Loop through all cells in range
> > > For Each cell In rng
> > > ' Calculate perecentage
> > > pcnt = (cell / divisor) * 100
> > > cell.Select
> > > ' Set colorindex based on percentage
> > > Select Case pcnt
> > > Case Is > 100
> > > Selection.Interior.ColorIndex = 4
> > > Case Is >= 90
> > > Selection.Interior.ColorIndex = 35
> > > Case Is >= 80
> > > Selection.Interior.ColorIndex = 36
> > > Case Is >= 70
> > > Selection.Interior.ColorIndex = 7
> > > Case Is >= 1
> > > Selection.Interior.ColorIndex = 54
> > > Case Else
> > > Selection.Interior.ColorIndex = 3
> > > End Select
> > > Next cell
> > >
> > > End Sub
> > >
> > >
> > > HTH
> > >
> > > "Leslie" wrote:
> > >
> > > > First, the formatting will only highlight certain cells in a column when the
> > > > column header in Row 3 is the current month which is tied to A2 which is
> > > > =Text(now(),"mmm"). Second, only those cells in rows in which Column A has
> > > > four digit letters or numbers will be highlighted. What I need is if, for
> > > > example, "Col F Row 9" / "Col F Row 5" = 200% then the cell background will
> > > > be shaded bright green. Row 5 is static and used as the divisor for all. The
> > > > conditions are: if the percent is 100.1% or greater then background of cell
> > > > is bright green, if percent is 90-100% then light green, if percent is
> > > > 80-89.9% than light yellow, if percent is 70-79.9% than pink, if 1-69.9% then
> > > > purple, if percent is 0 or blank then red. The columns that contain the data
> > > > are columns F-P. Is there any way to automate this process. Thanks in
> > > > advance. I am very new to VBA.
> > > >
> > > > Col F Col G Col H Col I Col J Col k Col L
> > > > Row 3 Jan Feb Mar Apr May Jun Jly
> > > >
> > > > Row 5 $500 $500 $500 $750 $750 $750 $1,000
> > > >
> > > > Row 9 $1000 $750 $500 $0 $1500 $900 $500
> > > > Many Rows just like Row 9
> > > >

It is stopping when it hits a #DIV/0 but once we have column A included in
the code then that won't happen but there will be blanks.

"Toppers" wrote:

> Hi again,
> You will get your error message if the cell is blank so I
> have added a test for cell value being a number.
>
> HTH
>
> Sub CFormat()
> '
> Dim rng As Range, cell As Range
> Dim ncol As Integer, lrow As Long
> Dim pcnt As Double, divisor As Double
>
> ThisWorkbook.Worksheets("Sheet1").Activate ' <=== Change to your w/sheet
> ' Find column for current Month (add 5 to start in colum F onwards)
> ncol = Application.Match(Range("A2"), Range("F3:q3"), 0) + 5
>
> ' Find last row of data in current month column
> lrow = Cells(Rows.Count, ncol).End(xlUp).Row
>
> ' Set range to cells for current month starting row 9
> Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
>
> ' Set Divisor for current month
> divisor = Cells(5, ncol)
>
> ' Loop through all cells in range
> For Each cell In rng
> ' Check length of cell in column A
> If Len(cell.Offset(0, -(ncol - 1))) = 4 Then
> ' Calculate perecentage
> If Application.IsNumber(cell) Then ' Is this cell a number ?
> pcnt = (cell / divisor) * 100
> cell.Select
> ' Set colorindex based on percentage
> Select Case pcnt
> Case Is > 100
> Selection.Interior.ColorIndex = 4
> Case Is >= 90
> Selection.Interior.ColorIndex = 35
> Case Is >= 80
> Selection.Interior.ColorIndex = 36
> Case Is >= 70
> Selection.Interior.ColorIndex = 7
> Case Is >= 1
> Selection.Interior.ColorIndex = 54
> Case Else
> Selection.Interior.ColorIndex = 3
> End Select
> End If
> End If
> Next cell
>
> End Sub
>
>
> "Leslie" wrote:
>
> > Okay, I've confused myself. The run-time error is 1004 after all. I think
> > it is stuck at:
> >
> > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> >
> > but since I am new to this VBA stuff I most likely am wrong. Does it have
> > something to do with "ncol". Thanks.
> >
> > "Toppers" wrote:
> >
> > > Hi,
> > > Try this: insert this code into a general module.
> > >
> > > I have defined the cell A2 as a named range called "CurMonth" and the month
> > > headers as a range "HdrMonths".
> > >
> > > Sub CFormat()
> > > '
> > > Dim rng As Range, cell As Range
> > > Dim ncol As Integer, lrow As Long
> > > Dim pcnt As Double, divisor As Double
> > >
> > > Thisworkbook.Worksheets("Sheet1").activate <=== Change to your w/sheet
> > >
> > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > >
> > > ' Find last row of data in current month column
> > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > >
> > > ' Set range to cells for current month starting row 9
> > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > >
> > > ' Set Divisor for current month
> > > divisor = Cells(5, ncol)
> > >
> > > ' Loop through all cells in range
> > > For Each cell In rng
> > > ' Calculate perecentage
> > > pcnt = (cell / divisor) * 100
> > > cell.Select
> > > ' Set colorindex based on percentage
> > > Select Case pcnt
> > > Case Is > 100
> > > Selection.Interior.ColorIndex = 4
> > > Case Is >= 90
> > > Selection.Interior.ColorIndex = 35
> > > Case Is >= 80
> > > Selection.Interior.ColorIndex = 36
> > > Case Is >= 70
> > > Selection.Interior.ColorIndex = 7
> > > Case Is >= 1
> > > Selection.Interior.ColorIndex = 54
> > > Case Else
> > > Selection.Interior.ColorIndex = 3
> > > End Select
> > > Next cell
> > >
> > > End Sub
> > >
> > >
> > > HTH
> > >
> > > "Leslie" wrote:
> > >
> > > > First, the formatting will only highlight certain cells in a column when the
> > > > column header in Row 3 is the current month which is tied to A2 which is
> > > > =Text(now(),"mmm"). Second, only those cells in rows in which Column A has
> > > > four digit letters or numbers will be highlighted. What I need is if, for
> > > > example, "Col F Row 9" / "Col F Row 5" = 200% then the cell background will
> > > > be shaded bright green. Row 5 is static and used as the divisor for all. The
> > > > conditions are: if the percent is 100.1% or greater then background of cell
> > > > is bright green, if percent is 90-100% then light green, if percent is
> > > > 80-89.9% than light yellow, if percent is 70-79.9% than pink, if 1-69.9% then
> > > > purple, if percent is 0 or blank then red. The columns that contain the data
> > > > are columns F-P. Is there any way to automate this process. Thanks in
> > > > advance. I am very new to VBA.
> > > >
> > > > Col F Col G Col H Col I Col J Col k Col L
> > > > Row 3 Jan Feb Mar Apr May Jun Jly
> > > >
> > > > Row 5 $500 $500 $500 $750 $750 $750 $1,000
> > > >
> > > > Row 9 $1000 $750 $500 $0 $1500 $900 $500
> > > > Many Rows just like Row 9
> > > >

Hi,
Can you divide by zero in your situation i.e. values in row 5 should
not be zero ? See my prvoius reply about blank data.

"Leslie" wrote:

> I think it stops when it hits a #DIV/0 on the spreadsheet. Could that be it.
> How do I fix that? Thanks.
>
> "Toppers" wrote:
>
> > Hi,
> >
> > Type mismatch suggests invalid data and I didn't put any check in for
> > valid data; which statement is it going wrong on?
> >
> > You can "step through" the macro by opening the code and with the cursor
> > somewhere in the code, and press the F8 button. This will go through an
> > instruction at a time. You could add statements to show the variable values-
> > an eay way is to use:
> >
> > msgbox ncol
> > msgbox lrow
> > msgbox divisor
> > msgbox cell
> >
> > placed at appropriate points in the code.
> >
> > If you alter the window size of the VB code you can look at the data as it
> > executes.
> >
> >
> > I see (in you code) your data starts at row 20 not 9.
> >
> >
> > The code below tests for column A having a field of length 4:
> >
> > Sub CFormat()
> > '
> > Dim rng As Range, cell As Range
> > Dim ncol As Integer, lrow As Long
> > Dim pcnt As Double, divisor As Double
> >
> > ThisWorkbook.Worksheets("Sheet1").Activate ' <=== Change to your w/sheet
> > ' Find column for current Month (add 5 to start in colum F onwards)
> > ncol = Application.Match(Range("A2"), Range("F3:q3"), 0) + 5
> >
> > ' Find last row of data in current month column
> > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> >
> > ' Set range to cells for current month starting row 9
> > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> >
> > ' Set Divisor for current month
> > divisor = Cells(5, ncol)
> >
> > ' Loop through all cells in range
> > For Each cell In rng
> > ' Check length of cell in column A
> > If Len(cell.Offset(0, -(ncol - 1))) = 4 Then
> > ' Calculate perecentage
> > pcnt = (cell / divisor) * 100
> > cell.Select
> > ' Set colorindex based on percentage
> > Select Case pcnt
> > Case Is > 100
> > Selection.Interior.ColorIndex = 4
> > Case Is >= 90
> > Selection.Interior.ColorIndex = 35
> > Case Is >= 80
> > Selection.Interior.ColorIndex = 36
> > Case Is >= 70
> > Selection.Interior.ColorIndex = 7
> > Case Is >= 1
> > Selection.Interior.ColorIndex = 54
> > Case Else
> > Selection.Interior.ColorIndex = 3
> > End Select
> > End If
> > Next cell
> >
> > End Sub
> >
> >
> > "Leslie" wrote:
> >
> > > Re: Col. A yes for field length of four. Also, I defined the Ranges so I
> > > think that is okay now but I am getting a Type Mismatch error 13. Any ideas?
> > > Thanks. Here is my code:
> > >
> > > Sub CFormat()
> > >
> > > Dim rng As Range, cell As Range
> > > Dim ncol As Integer, lrow As Long
> > > Dim pcnt As Double, divisor As Double
> > >
> > > 'Ace is sheet name
> > > ThisWorkbook.Worksheets("Ace").Activate
> > >
> > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > >
> > > ' Find last row of data in current month column
> > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > >
> > > ' Set range to cells for current month starting row 9
> > > Set rng = Range(Cells(20, ncol), Cells(lrow, ncol))
> > >
> > > ' Set Divisor for current month
> > > divisor = Cells(5, ncol)
> > >
> > > ' Loop through all cells in range
> > > For Each cell In rng
> > > ' Calculate percentage
> > > pcnt = (cell / divisor) * 100
> > > cell.Select
> > > ' Set colorindex based on percentage
> > > Select Case pcnt
> > > Case Is > 100
> > > Selection.Interior.ColorIndex = 4
> > > Case Is >= 90
> > > Selection.Interior.ColorIndex = 35
> > > Case Is >= 80
> > > Selection.Interior.ColorIndex = 36
> > > Case Is >= 70
> > > Selection.Interior.ColorIndex = 7
> > > Case Is >= 1
> > > Selection.Interior.ColorIndex = 54
> > > Case Else
> > > Selection.Interior.ColorIndex = 3
> > > End Select
> > > Next cell
> > >
> > > End Sub
> > >
> > >
> > > "Toppers" wrote:
> > >
> > > > Hi,
> > > > These are the "named" ranges I mentioned. You can try changing to:
> > > >
> > > > ncol=Application.Match(Range("a2"),Range("F3:Q3"),0)+5
> > > >
> > > > This avoids using named ranges.
> > > >
> > > > On re-reading your note I didn't check colum A for a 4 digits/Letters; Try
> > > > the above a get the code working and I will look at adding the required test.
> > > > Can I simply test that column A has field of length 4, irrespctive of whether
> > > > it is numbers/letters or amixture?
> > > >
> > > > HTH
> > > >
> > > > "Leslie" wrote:
> > > >
> > > > > Okay, I've confused myself. The run-time error is 1004 after all. I think
> > > > > it is stuck at:
> > > > >
> > > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > > >
> > > > > but since I am new to this VBA stuff I most likely am wrong. Does it have
> > > > > something to do with "ncol". Thanks.
> > > > >
> > > > > "Toppers" wrote:
> > > > >
> > > > > > Hi,
> > > > > > Try this: insert this code into a general module.
> > > > > >
> > > > > > I have defined the cell A2 as a named range called "CurMonth" and the month
> > > > > > headers as a range "HdrMonths".
> > > > > >
> > > > > > Sub CFormat()
> > > > > > '
> > > > > > Dim rng As Range, cell As Range
> > > > > > Dim ncol As Integer, lrow As Long
> > > > > > Dim pcnt As Double, divisor As Double
> > > > > >
> > > > > > Thisworkbook.Worksheets("Sheet1").activate <=== Change to your w/sheet
> > > > > >
> > > > > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > > > >
> > > > > > ' Find last row of data in current month column
> > > > > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > > > > >
> > > > > > ' Set range to cells for current month starting row 9
> > > > > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > > > > >
> > > > > > ' Set Divisor for current month
> > > > > > divisor = Cells(5, ncol)
> > > > > >
> > > > > > ' Loop through all cells in range
> > > > > > For Each cell In rng
> > > > > > ' Calculate perecentage
> > > > > > pcnt = (cell / divisor) * 100
> > > > > > cell.Select
> > > > > > ' Set colorindex based on percentage
> > > > > > Select Case pcnt
> > > > > > Case Is > 100
> > > > > > Selection.Interior.ColorIndex = 4
> > > > > > Case Is >= 90
> > > > > > Selection.Interior.ColorIndex = 35
> > > > > > Case Is >= 80
> > > > > > Selection.Interior.ColorIndex = 36
> > > > > > Case Is >= 70
> > > > > > Selection.Interior.ColorIndex = 7
> > > > > > Case Is >= 1
> > > > > > Selection.Interior.ColorIndex = 54
> > > > > > Case Else
> > > > > > Selection.Interior.ColorIndex = 3
> > > > > > End Select
> > > > > > Next cell
> > > > > >
> > > > > > End Sub
> > > > > >
> > > > > >
> > > > > > HTH
> > > > > >
> > > > > > "Leslie" wrote:
> > > > > >
> > > > > > > First, the formatting will only highlight certain cells in a column when the
> > > > > > > column header in Row 3 is the current month which is tied to A2 which is
> > > > > > > =Text(now(),"mmm"). Second, only those cells in rows in which Column A has
> > > > > > > four digit letters or numbers will be highlighted. What I need is if, for
> > > > > > > example, "Col F Row 9" / "Col F Row 5" = 200% then the cell background will
> > > > > > > be shaded bright green. Row 5 is static and used as the divisor for all. The
> > > > > > > conditions are: if the percent is 100.1% or greater then background of cell
> > > > > > > is bright green, if percent is 90-100% then light green, if percent is
> > > > > > > 80-89.9% than light yellow, if percent is 70-79.9% than pink, if 1-69.9% then
> > > > > > > purple, if percent is 0 or blank then red. The columns that contain the data
> > > > > > > are columns F-P. Is there any way to automate this process. Thanks in
> > > > > > > advance. I am very new to VBA.
> > > > > > >
> > > > > > > Col F Col G Col H Col I Col J Col k Col L
> > > > > > > Row 3 Jan Feb Mar Apr May Jun Jly
> > > > > > >
> > > > > > > Row 5 $500 $500 $500 $750 $750 $750 $1,000
> > > > > > >
> > > > > > > Row 9 $1000 $750 $500 $0 $1500 $900 $500
> > > > > > > Many Rows just like Row 9
> > > > > > >

I think it stops when it hits a #DIV/0 on the spreadsheet. Could that be it.
How do I fix that? Thanks.

"Toppers" wrote:

> Hi,
>
> Type mismatch suggests invalid data and I didn't put any check in for
> valid data; which statement is it going wrong on?
>
> You can "step through" the macro by opening the code and with the cursor
> somewhere in the code, and press the F8 button. This will go through an
> instruction at a time. You could add statements to show the variable values-
> an eay way is to use:
>
> msgbox ncol
> msgbox lrow
> msgbox divisor
> msgbox cell
>
> placed at appropriate points in the code.
>
> If you alter the window size of the VB code you can look at the data as it
> executes.
>
>
> I see (in you code) your data starts at row 20 not 9.
>
>
> The code below tests for column A having a field of length 4:
>
> Sub CFormat()
> '
> Dim rng As Range, cell As Range
> Dim ncol As Integer, lrow As Long
> Dim pcnt As Double, divisor As Double
>
> ThisWorkbook.Worksheets("Sheet1").Activate ' <=== Change to your w/sheet
> ' Find column for current Month (add 5 to start in colum F onwards)
> ncol = Application.Match(Range("A2"), Range("F3:q3"), 0) + 5
>
> ' Find last row of data in current month column
> lrow = Cells(Rows.Count, ncol).End(xlUp).Row
>
> ' Set range to cells for current month starting row 9
> Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
>
> ' Set Divisor for current month
> divisor = Cells(5, ncol)
>
> ' Loop through all cells in range
> For Each cell In rng
> ' Check length of cell in column A
> If Len(cell.Offset(0, -(ncol - 1))) = 4 Then
> ' Calculate perecentage
> pcnt = (cell / divisor) * 100
> cell.Select
> ' Set colorindex based on percentage
> Select Case pcnt
> Case Is > 100
> Selection.Interior.ColorIndex = 4
> Case Is >= 90
> Selection.Interior.ColorIndex = 35
> Case Is >= 80
> Selection.Interior.ColorIndex = 36
> Case Is >= 70
> Selection.Interior.ColorIndex = 7
> Case Is >= 1
> Selection.Interior.ColorIndex = 54
> Case Else
> Selection.Interior.ColorIndex = 3
> End Select
> End If
> Next cell
>
> End Sub
>
>
> "Leslie" wrote:
>
> > Re: Col. A yes for field length of four. Also, I defined the Ranges so I
> > think that is okay now but I am getting a Type Mismatch error 13. Any ideas?
> > Thanks. Here is my code:
> >
> > Sub CFormat()
> >
> > Dim rng As Range, cell As Range
> > Dim ncol As Integer, lrow As Long
> > Dim pcnt As Double, divisor As Double
> >
> > 'Ace is sheet name
> > ThisWorkbook.Worksheets("Ace").Activate
> >
> > ' Find column for current Month (add 5 to start in colum F onwards)
> > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> >
> > ' Find last row of data in current month column
> > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> >
> > ' Set range to cells for current month starting row 9
> > Set rng = Range(Cells(20, ncol), Cells(lrow, ncol))
> >
> > ' Set Divisor for current month
> > divisor = Cells(5, ncol)
> >
> > ' Loop through all cells in range
> > For Each cell In rng
> > ' Calculate percentage
> > pcnt = (cell / divisor) * 100
> > cell.Select
> > ' Set colorindex based on percentage
> > Select Case pcnt
> > Case Is > 100
> > Selection.Interior.ColorIndex = 4
> > Case Is >= 90
> > Selection.Interior.ColorIndex = 35
> > Case Is >= 80
> > Selection.Interior.ColorIndex = 36
> > Case Is >= 70
> > Selection.Interior.ColorIndex = 7
> > Case Is >= 1
> > Selection.Interior.ColorIndex = 54
> > Case Else
> > Selection.Interior.ColorIndex = 3
> > End Select
> > Next cell
> >
> > End Sub
> >
> >
> > "Toppers" wrote:
> >
> > > Hi,
> > > These are the "named" ranges I mentioned. You can try changing to:
> > >
> > > ncol=Application.Match(Range("a2"),Range("F3:Q3"),0)+5
> > >
> > > This avoids using named ranges.
> > >
> > > On re-reading your note I didn't check colum A for a 4 digits/Letters; Try
> > > the above a get the code working and I will look at adding the required test.
> > > Can I simply test that column A has field of length 4, irrespctive of whether
> > > it is numbers/letters or amixture?
> > >
> > > HTH
> > >
> > > "Leslie" wrote:
> > >
> > > > Okay, I've confused myself. The run-time error is 1004 after all. I think
> > > > it is stuck at:
> > > >
> > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > >
> > > > but since I am new to this VBA stuff I most likely am wrong. Does it have
> > > > something to do with "ncol". Thanks.
> > > >
> > > > "Toppers" wrote:
> > > >
> > > > > Hi,
> > > > > Try this: insert this code into a general module.
> > > > >
> > > > > I have defined the cell A2 as a named range called "CurMonth" and the month
> > > > > headers as a range "HdrMonths".
> > > > >
> > > > > Sub CFormat()
> > > > > '
> > > > > Dim rng As Range, cell As Range
> > > > > Dim ncol As Integer, lrow As Long
> > > > > Dim pcnt As Double, divisor As Double
> > > > >
> > > > > Thisworkbook.Worksheets("Sheet1").activate <=== Change to your w/sheet
> > > > >
> > > > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > > >
> > > > > ' Find last row of data in current month column
> > > > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > > > >
> > > > > ' Set range to cells for current month starting row 9
> > > > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > > > >
> > > > > ' Set Divisor for current month
> > > > > divisor = Cells(5, ncol)
> > > > >
> > > > > ' Loop through all cells in range
> > > > > For Each cell In rng
> > > > > ' Calculate perecentage
> > > > > pcnt = (cell / divisor) * 100
> > > > > cell.Select
> > > > > ' Set colorindex based on percentage
> > > > > Select Case pcnt
> > > > > Case Is > 100
> > > > > Selection.Interior.ColorIndex = 4
> > > > > Case Is >= 90
> > > > > Selection.Interior.ColorIndex = 35
> > > > > Case Is >= 80
> > > > > Selection.Interior.ColorIndex = 36
> > > > > Case Is >= 70
> > > > > Selection.Interior.ColorIndex = 7
> > > > > Case Is >= 1
> > > > > Selection.Interior.ColorIndex = 54
> > > > > Case Else
> > > > > Selection.Interior.ColorIndex = 3
> > > > > End Select
> > > > > Next cell
> > > > >
> > > > > End Sub
> > > > >
> > > > >
> > > > > HTH
> > > > >
> > > > > "Leslie" wrote:
> > > > >
> > > > > > First, the formatting will only highlight certain cells in a column when the
> > > > > > column header in Row 3 is the current month which is tied to A2 which is
> > > > > > =Text(now(),"mmm"). Second, only those cells in rows in which Column A has
> > > > > > four digit letters or numbers will be highlighted. What I need is if, for
> > > > > > example, "Col F Row 9" / "Col F Row 5" = 200% then the cell background will
> > > > > > be shaded bright green. Row 5 is static and used as the divisor for all. The
> > > > > > conditions are: if the percent is 100.1% or greater then background of cell
> > > > > > is bright green, if percent is 90-100% then light green, if percent is
> > > > > > 80-89.9% than light yellow, if percent is 70-79.9% than pink, if 1-69.9% then
> > > > > > purple, if percent is 0 or blank then red. The columns that contain the data
> > > > > > are columns F-P. Is there any way to automate this process. Thanks in
> > > > > > advance. I am very new to VBA.
> > > > > >
> > > > > > Col F Col G Col H Col I Col J Col k Col L
> > > > > > Row 3 Jan Feb Mar Apr May Jun Jly
> > > > > >
> > > > > > Row 5 $500 $500 $500 $750 $750 $750 $1,000
> > > > > >
> > > > > > Row 9 $1000 $750 $500 $0 $1500 $900 $500
> > > > > > Many Rows just like Row 9
> > > > > >

Hi again,
You will get your error message if the cell is blank so I
have added a test for cell value being a number.

HTH

Sub CFormat()
'
Dim rng As Range, cell As Range
Dim ncol As Integer, lrow As Long
Dim pcnt As Double, divisor As Double

ThisWorkbook.Worksheets("Sheet1").Activate ' <=== Change to your w/sheet
' Find column for current Month (add 5 to start in colum F onwards)
ncol = Application.Match(Range("A2"), Range("F3:q3"), 0) + 5

' Find last row of data in current month column
lrow = Cells(Rows.Count, ncol).End(xlUp).Row

' Set range to cells for current month starting row 9
Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))

' Set Divisor for current month
divisor = Cells(5, ncol)

' Loop through all cells in range
For Each cell In rng
' Check length of cell in column A
If Len(cell.Offset(0, -(ncol - 1))) = 4 Then
' Calculate perecentage
If Application.IsNumber(cell) Then ' Is this cell a number ?
pcnt = (cell / divisor) * 100
cell.Select
' Set colorindex based on percentage
Select Case pcnt
Case Is > 100
Selection.Interior.ColorIndex = 4
Case Is >= 90
Selection.Interior.ColorIndex = 35
Case Is >= 80
Selection.Interior.ColorIndex = 36
Case Is >= 70
Selection.Interior.ColorIndex = 7
Case Is >= 1
Selection.Interior.ColorIndex = 54
Case Else
Selection.Interior.ColorIndex = 3
End Select
End If
End If
Next cell

End Sub


"Leslie" wrote:

> Okay, I've confused myself. The run-time error is 1004 after all. I think
> it is stuck at:
>
> ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
>
> but since I am new to this VBA stuff I most likely am wrong. Does it have
> something to do with "ncol". Thanks.
>
> "Toppers" wrote:
>
> > Hi,
> > Try this: insert this code into a general module.
> >
> > I have defined the cell A2 as a named range called "CurMonth" and the month
> > headers as a range "HdrMonths".
> >
> > Sub CFormat()
> > '
> > Dim rng As Range, cell As Range
> > Dim ncol As Integer, lrow As Long
> > Dim pcnt As Double, divisor As Double
> >
> > Thisworkbook.Worksheets("Sheet1").activate <=== Change to your w/sheet
> >
> > ' Find column for current Month (add 5 to start in colum F onwards)
> > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> >
> > ' Find last row of data in current month column
> > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> >
> > ' Set range to cells for current month starting row 9
> > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> >
> > ' Set Divisor for current month
> > divisor = Cells(5, ncol)
> >
> > ' Loop through all cells in range
> > For Each cell In rng
> > ' Calculate perecentage
> > pcnt = (cell / divisor) * 100
> > cell.Select
> > ' Set colorindex based on percentage
> > Select Case pcnt
> > Case Is > 100
> > Selection.Interior.ColorIndex = 4
> > Case Is >= 90
> > Selection.Interior.ColorIndex = 35
> > Case Is >= 80
> > Selection.Interior.ColorIndex = 36
> > Case Is >= 70
> > Selection.Interior.ColorIndex = 7
> > Case Is >= 1
> > Selection.Interior.ColorIndex = 54
> > Case Else
> > Selection.Interior.ColorIndex = 3
> > End Select
> > Next cell
> >
> > End Sub
> >
> >
> > HTH
> >
> > "Leslie" wrote:
> >
> > > First, the formatting will only highlight certain cells in a column when the
> > > column header in Row 3 is the current month which is tied to A2 which is
> > > =Text(now(),"mmm"). Second, only those cells in rows in which Column A has
> > > four digit letters or numbers will be highlighted. What I need is if, for
> > > example, "Col F Row 9" / "Col F Row 5" = 200% then the cell background will
> > > be shaded bright green. Row 5 is static and used as the divisor for all. The
> > > conditions are: if the percent is 100.1% or greater then background of cell
> > > is bright green, if percent is 90-100% then light green, if percent is
> > > 80-89.9% than light yellow, if percent is 70-79.9% than pink, if 1-69.9% then
> > > purple, if percent is 0 or blank then red. The columns that contain the data
> > > are columns F-P. Is there any way to automate this process. Thanks in
> > > advance. I am very new to VBA.
> > >
> > > Col F Col G Col H Col I Col J Col k Col L
> > > Row 3 Jan Feb Mar Apr May Jun Jly
> > >
> > > Row 5 $500 $500 $500 $750 $750 $750 $1,000
> > >
> > > Row 9 $1000 $750 $500 $0 $1500 $900 $500
> > > Many Rows just like Row 9
> > >

Hi,

Type mismatch suggests invalid data and I didn't put any check in for
valid data; which statement is it going wrong on?

You can "step through" the macro by opening the code and with the cursor
somewhere in the code, and press the F8 button. This will go through an
instruction at a time. You could add statements to show the variable values-
an eay way is to use:

msgbox ncol
msgbox lrow
msgbox divisor
msgbox cell

placed at appropriate points in the code.

If you alter the window size of the VB code you can look at the data as it
executes.


I see (in you code) your data starts at row 20 not 9.


The code below tests for column A having a field of length 4:

Sub CFormat()
'
Dim rng As Range, cell As Range
Dim ncol As Integer, lrow As Long
Dim pcnt As Double, divisor As Double

ThisWorkbook.Worksheets("Sheet1").Activate ' <=== Change to your w/sheet
' Find column for current Month (add 5 to start in colum F onwards)
ncol = Application.Match(Range("A2"), Range("F3:q3"), 0) + 5

' Find last row of data in current month column
lrow = Cells(Rows.Count, ncol).End(xlUp).Row

' Set range to cells for current month starting row 9
Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))

' Set Divisor for current month
divisor = Cells(5, ncol)

' Loop through all cells in range
For Each cell In rng
' Check length of cell in column A
If Len(cell.Offset(0, -(ncol - 1))) = 4 Then
' Calculate perecentage
pcnt = (cell / divisor) * 100
cell.Select
' Set colorindex based on percentage
Select Case pcnt
Case Is > 100
Selection.Interior.ColorIndex = 4
Case Is >= 90
Selection.Interior.ColorIndex = 35
Case Is >= 80
Selection.Interior.ColorIndex = 36
Case Is >= 70
Selection.Interior.ColorIndex = 7
Case Is >= 1
Selection.Interior.ColorIndex = 54
Case Else
Selection.Interior.ColorIndex = 3
End Select
End If
Next cell

End Sub


"Leslie" wrote:

> Re: Col. A yes for field length of four. Also, I defined the Ranges so I
> think that is okay now but I am getting a Type Mismatch error 13. Any ideas?
> Thanks. Here is my code:
>
> Sub CFormat()
>
> Dim rng As Range, cell As Range
> Dim ncol As Integer, lrow As Long
> Dim pcnt As Double, divisor As Double
>
> 'Ace is sheet name
> ThisWorkbook.Worksheets("Ace").Activate
>
> ' Find column for current Month (add 5 to start in colum F onwards)
> ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
>
> ' Find last row of data in current month column
> lrow = Cells(Rows.Count, ncol).End(xlUp).Row
>
> ' Set range to cells for current month starting row 9
> Set rng = Range(Cells(20, ncol), Cells(lrow, ncol))
>
> ' Set Divisor for current month
> divisor = Cells(5, ncol)
>
> ' Loop through all cells in range
> For Each cell In rng
> ' Calculate percentage
> pcnt = (cell / divisor) * 100
> cell.Select
> ' Set colorindex based on percentage
> Select Case pcnt
> Case Is > 100
> Selection.Interior.ColorIndex = 4
> Case Is >= 90
> Selection.Interior.ColorIndex = 35
> Case Is >= 80
> Selection.Interior.ColorIndex = 36
> Case Is >= 70
> Selection.Interior.ColorIndex = 7
> Case Is >= 1
> Selection.Interior.ColorIndex = 54
> Case Else
> Selection.Interior.ColorIndex = 3
> End Select
> Next cell
>
> End Sub
>
>
> "Toppers" wrote:
>
> > Hi,
> > These are the "named" ranges I mentioned. You can try changing to:
> >
> > ncol=Application.Match(Range("a2"),Range("F3:Q3"),0)+5
> >
> > This avoids using named ranges.
> >
> > On re-reading your note I didn't check colum A for a 4 digits/Letters; Try
> > the above a get the code working and I will look at adding the required test.
> > Can I simply test that column A has field of length 4, irrespctive of whether
> > it is numbers/letters or amixture?
> >
> > HTH
> >
> > "Leslie" wrote:
> >
> > > Okay, I've confused myself. The run-time error is 1004 after all. I think
> > > it is stuck at:
> > >
> > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > >
> > > but since I am new to this VBA stuff I most likely am wrong. Does it have
> > > something to do with "ncol". Thanks.
> > >
> > > "Toppers" wrote:
> > >
> > > > Hi,
> > > > Try this: insert this code into a general module.
> > > >
> > > > I have defined the cell A2 as a named range called "CurMonth" and the month
> > > > headers as a range "HdrMonths".
> > > >
> > > > Sub CFormat()
> > > > '
> > > > Dim rng As Range, cell As Range
> > > > Dim ncol As Integer, lrow As Long
> > > > Dim pcnt As Double, divisor As Double
> > > >
> > > > Thisworkbook.Worksheets("Sheet1").activate <=== Change to your w/sheet
> > > >
> > > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > >
> > > > ' Find last row of data in current month column
> > > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > > >
> > > > ' Set range to cells for current month starting row 9
> > > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > > >
> > > > ' Set Divisor for current month
> > > > divisor = Cells(5, ncol)
> > > >
> > > > ' Loop through all cells in range
> > > > For Each cell In rng
> > > > ' Calculate perecentage
> > > > pcnt = (cell / divisor) * 100
> > > > cell.Select
> > > > ' Set colorindex based on percentage
> > > > Select Case pcnt
> > > > Case Is > 100
> > > > Selection.Interior.ColorIndex = 4
> > > > Case Is >= 90
> > > > Selection.Interior.ColorIndex = 35
> > > > Case Is >= 80
> > > > Selection.Interior.ColorIndex = 36
> > > > Case Is >= 70
> > > > Selection.Interior.ColorIndex = 7
> > > > Case Is >= 1
> > > > Selection.Interior.ColorIndex = 54
> > > > Case Else
> > > > Selection.Interior.ColorIndex = 3
> > > > End Select
> > > > Next cell
> > > >
> > > > End Sub
> > > >
> > > >
> > > > HTH
> > > >
> > > > "Leslie" wrote:
> > > >
> > > > > First, the formatting will only highlight certain cells in a column when the
> > > > > column header in Row 3 is the current month which is tied to A2 which is
> > > > > =Text(now(),"mmm"). Second, only those cells in rows in which Column A has
> > > > > four digit letters or numbers will be highlighted. What I need is if, for
> > > > > example, "Col F Row 9" / "Col F Row 5" = 200% then the cell background will
> > > > > be shaded bright green. Row 5 is static and used as the divisor for all. The
> > > > > conditions are: if the percent is 100.1% or greater then background of cell
> > > > > is bright green, if percent is 90-100% then light green, if percent is
> > > > > 80-89.9% than light yellow, if percent is 70-79.9% than pink, if 1-69.9% then
> > > > > purple, if percent is 0 or blank then red. The columns that contain the data
> > > > > are columns F-P. Is there any way to automate this process. Thanks in
> > > > > advance. I am very new to VBA.
> > > > >
> > > > > Col F Col G Col H Col I Col J Col k Col L
> > > > > Row 3 Jan Feb Mar Apr May Jun Jly
> > > > >
> > > > > Row 5 $500 $500 $500 $750 $750 $750 $1,000
> > > > >
> > > > > Row 9 $1000 $750 $500 $0 $1500 $900 $500
> > > > > Many Rows just like Row 9
> > > > >

I ran the code as listed and also received the type mismatch error.

I then checked and made sure that my month headers coincided with the range that was set.

' Find column for current Month (add 5 to start in colum F onwards)
ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5

Once I made sure that Jan was on column F, everything flowed well and the appropriate values were highlighted.

Thanks so much for all your help that was so tricky for me. May I ask one
more question. I tried it out and everything works great but I think it is
missing something. Once July comes or any month thereafter I want it to
clear out the background colors from the previous month and only have the
current month highlighted. Is that possible to do?

"Toppers" wrote:

> Check the END IF are all there! It works OK on my machine!
>
> "Leslie" wrote:
>
> > It doesn't like the "Next Cell" at the bottom of the code. Thanks.
> >
> > "Toppers" wrote:
> >
> > > Hi again,
> > > You will get your error message if the cell is blank so I
> > > have added a test for cell value being a number.
> > >
> > > HTH
> > >
> > > Sub CFormat()
> > > '
> > > Dim rng As Range, cell As Range
> > > Dim ncol As Integer, lrow As Long
> > > Dim pcnt As Double, divisor As Double
> > >
> > > ThisWorkbook.Worksheets("Sheet1").Activate ' <=== Change to your w/sheet
> > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > ncol = Application.Match(Range("A2"), Range("F3:q3"), 0) + 5
> > >
> > > ' Find last row of data in current month column
> > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > >
> > > ' Set range to cells for current month starting row 9
> > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > >
> > > ' Set Divisor for current month
> > > divisor = Cells(5, ncol)
> > >
> > > ' Loop through all cells in range
> > > For Each cell In rng
> > > ' Check length of cell in column A
> > > If Len(cell.Offset(0, -(ncol - 1))) = 4 Then
> > > ' Calculate perecentage
> > > If Application.IsNumber(cell) Then ' Is this cell a number ?
> > > pcnt = (cell / divisor) * 100
> > > cell.Select
> > > ' Set colorindex based on percentage
> > > Select Case pcnt
> > > Case Is > 100
> > > Selection.Interior.ColorIndex = 4
> > > Case Is >= 90
> > > Selection.Interior.ColorIndex = 35
> > > Case Is >= 80
> > > Selection.Interior.ColorIndex = 36
> > > Case Is >= 70
> > > Selection.Interior.ColorIndex = 7
> > > Case Is >= 1
> > > Selection.Interior.ColorIndex = 54
> > > Case Else
> > > Selection.Interior.ColorIndex = 3
> > > End Select
> > > End If
> > > End If
> > > Next cell
> > >
> > > End Sub
> > >
> > >
> > > "Leslie" wrote:
> > >
> > > > Okay, I've confused myself. The run-time error is 1004 after all. I think
> > > > it is stuck at:
> > > >
> > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > >
> > > > but since I am new to this VBA stuff I most likely am wrong. Does it have
> > > > something to do with "ncol". Thanks.
> > > >
> > > > "Toppers" wrote:
> > > >
> > > > > Hi,
> > > > > Try this: insert this code into a general module.
> > > > >
> > > > > I have defined the cell A2 as a named range called "CurMonth" and the month
> > > > > headers as a range "HdrMonths".
> > > > >
> > > > > Sub CFormat()
> > > > > '
> > > > > Dim rng As Range, cell As Range
> > > > > Dim ncol As Integer, lrow As Long
> > > > > Dim pcnt As Double, divisor As Double
> > > > >
> > > > > Thisworkbook.Worksheets("Sheet1").activate <=== Change to your w/sheet
> > > > >
> > > > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > > >
> > > > > ' Find last row of data in current month column
> > > > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > > > >
> > > > > ' Set range to cells for current month starting row 9
> > > > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > > > >
> > > > > ' Set Divisor for current month
> > > > > divisor = Cells(5, ncol)
> > > > >
> > > > > ' Loop through all cells in range
> > > > > For Each cell In rng
> > > > > ' Calculate perecentage
> > > > > pcnt = (cell / divisor) * 100
> > > > > cell.Select
> > > > > ' Set colorindex based on percentage
> > > > > Select Case pcnt
> > > > > Case Is > 100
> > > > > Selection.Interior.ColorIndex = 4
> > > > > Case Is >= 90
> > > > > Selection.Interior.ColorIndex = 35
> > > > > Case Is >= 80
> > > > > Selection.Interior.ColorIndex = 36
> > > > > Case Is >= 70
> > > > > Selection.Interior.ColorIndex = 7
> > > > > Case Is >= 1
> > > > > Selection.Interior.ColorIndex = 54
> > > > > Case Else
> > > > > Selection.Interior.ColorIndex = 3
> > > > > End Select
> > > > > Next cell
> > > > >
> > > > > End Sub
> > > > >
> > > > >
> > > > > HTH
> > > > >
> > > > > "Leslie" wrote:
> > > > >
> > > > > > First, the formatting will only highlight certain cells in a column when the
> > > > > > column header in Row 3 is the current month which is tied to A2 which is
> > > > > > =Text(now(),"mmm"). Second, only those cells in rows in which Column A has
> > > > > > four digit letters or numbers will be highlighted. What I need is if, for
> > > > > > example, "Col F Row 9" / "Col F Row 5" = 200% then the cell background will
> > > > > > be shaded bright green. Row 5 is static and used as the divisor for all. The
> > > > > > conditions are: if the percent is 100.1% or greater then background of cell
> > > > > > is bright green, if percent is 90-100% then light green, if percent is
> > > > > > 80-89.9% than light yellow, if percent is 70-79.9% than pink, if 1-69.9% then
> > > > > > purple, if percent is 0 or blank then red. The columns that contain the data
> > > > > > are columns F-P. Is there any way to automate this process. Thanks in
> > > > > > advance. I am very new to VBA.
> > > > > >
> > > > > > Col F Col G Col H Col I Col J Col k Col L
> > > > > > Row 3 Jan Feb Mar Apr May Jun Jly
> > > > > >
> > > > > > Row 5 $500 $500 $500 $750 $750 $750 $1,000
> > > > > >
> > > > > > Row 9 $1000 $750 $500 $0 $1500 $900 $500
> > > > > > Many Rows just like Row 9
> > > > > >

Hi,
The simplest way would be to clear ALL months at the beginning of
the macro and just re-populate the current month.

Range("F9:Q100").Select <=== change to 100 to whatever you think the max
roes are going to be.
Selection.Interior.ColorIndex = xlNone ' Clears the colours

Place this code at the start of the macro - after DIM statements.

HTH


"Leslie" wrote:

> Thanks so much for all your help that was so tricky for me. May I ask one
> more question. I tried it out and everything works great but I think it is
> missing something. Once July comes or any month thereafter I want it to
> clear out the background colors from the previous month and only have the
> current month highlighted. Is that possible to do?
>
> "Toppers" wrote:
>
> > Check the END IF are all there! It works OK on my machine!
> >
> > "Leslie" wrote:
> >
> > > It doesn't like the "Next Cell" at the bottom of the code. Thanks.
> > >
> > > "Toppers" wrote:
> > >
> > > > Hi again,
> > > > You will get your error message if the cell is blank so I
> > > > have added a test for cell value being a number.
> > > >
> > > > HTH
> > > >
> > > > Sub CFormat()
> > > > '
> > > > Dim rng As Range, cell As Range
> > > > Dim ncol As Integer, lrow As Long
> > > > Dim pcnt As Double, divisor As Double
> > > >
> > > > ThisWorkbook.Worksheets("Sheet1").Activate ' <=== Change to your w/sheet
> > > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > > ncol = Application.Match(Range("A2"), Range("F3:q3"), 0) + 5
> > > >
> > > > ' Find last row of data in current month column
> > > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > > >
> > > > ' Set range to cells for current month starting row 9
> > > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > > >
> > > > ' Set Divisor for current month
> > > > divisor = Cells(5, ncol)
> > > >
> > > > ' Loop through all cells in range
> > > > For Each cell In rng
> > > > ' Check length of cell in column A
> > > > If Len(cell.Offset(0, -(ncol - 1))) = 4 Then
> > > > ' Calculate perecentage
> > > > If Application.IsNumber(cell) Then ' Is this cell a number ?
> > > > pcnt = (cell / divisor) * 100
> > > > cell.Select
> > > > ' Set colorindex based on percentage
> > > > Select Case pcnt
> > > > Case Is > 100
> > > > Selection.Interior.ColorIndex = 4
> > > > Case Is >= 90
> > > > Selection.Interior.ColorIndex = 35
> > > > Case Is >= 80
> > > > Selection.Interior.ColorIndex = 36
> > > > Case Is >= 70
> > > > Selection.Interior.ColorIndex = 7
> > > > Case Is >= 1
> > > > Selection.Interior.ColorIndex = 54
> > > > Case Else
> > > > Selection.Interior.ColorIndex = 3
> > > > End Select
> > > > End If
> > > > End If
> > > > Next cell
> > > >
> > > > End Sub
> > > >
> > > >
> > > > "Leslie" wrote:
> > > >
> > > > > Okay, I've confused myself. The run-time error is 1004 after all. I think
> > > > > it is stuck at:
> > > > >
> > > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > > >
> > > > > but since I am new to this VBA stuff I most likely am wrong. Does it have
> > > > > something to do with "ncol". Thanks.
> > > > >
> > > > > "Toppers" wrote:
> > > > >
> > > > > > Hi,
> > > > > > Try this: insert this code into a general module.
> > > > > >
> > > > > > I have defined the cell A2 as a named range called "CurMonth" and the month
> > > > > > headers as a range "HdrMonths".
> > > > > >
> > > > > > Sub CFormat()
> > > > > > '
> > > > > > Dim rng As Range, cell As Range
> > > > > > Dim ncol As Integer, lrow As Long
> > > > > > Dim pcnt As Double, divisor As Double
> > > > > >
> > > > > > Thisworkbook.Worksheets("Sheet1").activate <=== Change to your w/sheet
> > > > > >
> > > > > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > > > >
> > > > > > ' Find last row of data in current month column
> > > > > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > > > > >
> > > > > > ' Set range to cells for current month starting row 9
> > > > > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > > > > >
> > > > > > ' Set Divisor for current month
> > > > > > divisor = Cells(5, ncol)
> > > > > >
> > > > > > ' Loop through all cells in range
> > > > > > For Each cell In rng
> > > > > > ' Calculate perecentage
> > > > > > pcnt = (cell / divisor) * 100
> > > > > > cell.Select
> > > > > > ' Set colorindex based on percentage
> > > > > > Select Case pcnt
> > > > > > Case Is > 100
> > > > > > Selection.Interior.ColorIndex = 4
> > > > > > Case Is >= 90
> > > > > > Selection.Interior.ColorIndex = 35
> > > > > > Case Is >= 80
> > > > > > Selection.Interior.ColorIndex = 36
> > > > > > Case Is >= 70
> > > > > > Selection.Interior.ColorIndex = 7
> > > > > > Case Is >= 1
> > > > > > Selection.Interior.ColorIndex = 54
> > > > > > Case Else
> > > > > > Selection.Interior.ColorIndex = 3
> > > > > > End Select
> > > > > > Next cell
> > > > > >
> > > > > > End Sub
> > > > > >
> > > > > >
> > > > > > HTH
> > > > > >
> > > > > > "Leslie" wrote:
> > > > > >
> > > > > > > First, the formatting will only highlight certain cells in a column when the
> > > > > > > column header in Row 3 is the current month which is tied to A2 which is
> > > > > > > =Text(now(),"mmm"). Second, only those cells in rows in which Column A has
> > > > > > > four digit letters or numbers will be highlighted. What I need is if, for
> > > > > > > example, "Col F Row 9" / "Col F Row 5" = 200% then the cell background will
> > > > > > > be shaded bright green. Row 5 is static and used as the divisor for all. The
> > > > > > > conditions are: if the percent is 100.1% or greater then background of cell
> > > > > > > is bright green, if percent is 90-100% then light green, if percent is
> > > > > > > 80-89.9% than light yellow, if percent is 70-79.9% than pink, if 1-69.9% then
> > > > > > > purple, if percent is 0 or blank then red. The columns that contain the data
> > > > > > > are columns F-P. Is there any way to automate this process. Thanks in
> > > > > > > advance. I am very new to VBA.
> > > > > > >
> > > > > > > Col F Col G Col H Col I Col J Col k Col L
> > > > > > > Row 3 Jan Feb Mar Apr May Jun Jly
> > > > > > >
> > > > > > > Row 5 $500 $500 $500 $750 $750 $750 $1,000
> > > > > > >
> > > > > > > Row 9 $1000 $750 $500 $0 $1500 $900 $500
> > > > > > > Many Rows just like Row 9
> > > > > > >

Sorry .. place after Thisworkbook. ....

"Toppers" wrote:

> Hi,
> The simplest way would be to clear ALL months at the beginning of
> the macro and just re-populate the current month.
>
> Range("F9:Q100").Select <=== change to 100 to whatever you think the max
> roes are going to be.
> Selection.Interior.ColorIndex = xlNone ' Clears the colours
>
> Place this code at the start of the macro - after DIM statements.
>
> HTH
>
>
> "Leslie" wrote:
>
> > Thanks so much for all your help that was so tricky for me. May I ask one
> > more question. I tried it out and everything works great but I think it is
> > missing something. Once July comes or any month thereafter I want it to
> > clear out the background colors from the previous month and only have the
> > current month highlighted. Is that possible to do?
> >
> > "Toppers" wrote:
> >
> > > Check the END IF are all there! It works OK on my machine!
> > >
> > > "Leslie" wrote:
> > >
> > > > It doesn't like the "Next Cell" at the bottom of the code. Thanks.
> > > >
> > > > "Toppers" wrote:
> > > >
> > > > > Hi again,
> > > > > You will get your error message if the cell is blank so I
> > > > > have added a test for cell value being a number.
> > > > >
> > > > > HTH
> > > > >
> > > > > Sub CFormat()
> > > > > '
> > > > > Dim rng As Range, cell As Range
> > > > > Dim ncol As Integer, lrow As Long
> > > > > Dim pcnt As Double, divisor As Double
> > > > >
> > > > > ThisWorkbook.Worksheets("Sheet1").Activate ' <=== Change to your w/sheet
> > > > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > > > ncol = Application.Match(Range("A2"), Range("F3:q3"), 0) + 5
> > > > >
> > > > > ' Find last row of data in current month column
> > > > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > > > >
> > > > > ' Set range to cells for current month starting row 9
> > > > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > > > >
> > > > > ' Set Divisor for current month
> > > > > divisor = Cells(5, ncol)
> > > > >
> > > > > ' Loop through all cells in range
> > > > > For Each cell In rng
> > > > > ' Check length of cell in column A
> > > > > If Len(cell.Offset(0, -(ncol - 1))) = 4 Then
> > > > > ' Calculate perecentage
> > > > > If Application.IsNumber(cell) Then ' Is this cell a number ?
> > > > > pcnt = (cell / divisor) * 100
> > > > > cell.Select
> > > > > ' Set colorindex based on percentage
> > > > > Select Case pcnt
> > > > > Case Is > 100
> > > > > Selection.Interior.ColorIndex = 4
> > > > > Case Is >= 90
> > > > > Selection.Interior.ColorIndex = 35
> > > > > Case Is >= 80
> > > > > Selection.Interior.ColorIndex = 36
> > > > > Case Is >= 70
> > > > > Selection.Interior.ColorIndex = 7
> > > > > Case Is >= 1
> > > > > Selection.Interior.ColorIndex = 54
> > > > > Case Else
> > > > > Selection.Interior.ColorIndex = 3
> > > > > End Select
> > > > > End If
> > > > > End If
> > > > > Next cell
> > > > >
> > > > > End Sub
> > > > >
> > > > >
> > > > > "Leslie" wrote:
> > > > >
> > > > > > Okay, I've confused myself. The run-time error is 1004 after all. I think
> > > > > > it is stuck at:
> > > > > >
> > > > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > > > >
> > > > > > but since I am new to this VBA stuff I most likely am wrong. Does it have
> > > > > > something to do with "ncol". Thanks.
> > > > > >
> > > > > > "Toppers" wrote:
> > > > > >
> > > > > > > Hi,
> > > > > > > Try this: insert this code into a general module.
> > > > > > >
> > > > > > > I have defined the cell A2 as a named range called "CurMonth" and the month
> > > > > > > headers as a range "HdrMonths".
> > > > > > >
> > > > > > > Sub CFormat()
> > > > > > > '
> > > > > > > Dim rng As Range, cell As Range
> > > > > > > Dim ncol As Integer, lrow As Long
> > > > > > > Dim pcnt As Double, divisor As Double
> > > > > > >
> > > > > > > Thisworkbook.Worksheets("Sheet1").activate <=== Change to your w/sheet
> > > > > > >
> > > > > > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > > > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > > > > >
> > > > > > > ' Find last row of data in current month column
> > > > > > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > > > > > >
> > > > > > > ' Set range to cells for current month starting row 9
> > > > > > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > > > > > >
> > > > > > > ' Set Divisor for current month
> > > > > > > divisor = Cells(5, ncol)
> > > > > > >
> > > > > > > ' Loop through all cells in range
> > > > > > > For Each cell In rng
> > > > > > > ' Calculate perecentage
> > > > > > > pcnt = (cell / divisor) * 100
> > > > > > > cell.Select
> > > > > > > ' Set colorindex based on percentage
> > > > > > > Select Case pcnt
> > > > > > > Case Is > 100
> > > > > > > Selection.Interior.ColorIndex = 4
> > > > > > > Case Is >= 90
> > > > > > > Selection.Interior.ColorIndex = 35
> > > > > > > Case Is >= 80
> > > > > > > Selection.Interior.ColorIndex = 36
> > > > > > > Case Is >= 70
> > > > > > > Selection.Interior.ColorIndex = 7
> > > > > > > Case Is >= 1
> > > > > > > Selection.Interior.ColorIndex = 54
> > > > > > > Case Else
> > > > > > > Selection.Interior.ColorIndex = 3
> > > > > > > End Select
> > > > > > > Next cell
> > > > > > >
> > > > > > > End Sub
> > > > > > >
> > > > > > >
> > > > > > > HTH
> > > > > > >
> > > > > > > "Leslie" wrote:
> > > > > > >
> > > > > > > > First, the formatting will only highlight certain cells in a column when the
> > > > > > > > column header in Row 3 is the current month which is tied to A2 which is
> > > > > > > > =Text(now(),"mmm"). Second, only those cells in rows in which Column A has
> > > > > > > > four digit letters or numbers will be highlighted. What I need is if, for
> > > > > > > > example, "Col F Row 9" / "Col F Row 5" = 200% then the cell background will
> > > > > > > > be shaded bright green. Row 5 is static and used as the divisor for all. The
> > > > > > > > conditions are: if the percent is 100.1% or greater then background of cell
> > > > > > > > is bright green, if percent is 90-100% then light green, if percent is
> > > > > > > > 80-89.9% than light yellow, if percent is 70-79.9% than pink, if 1-69.9% then
> > > > > > > > purple, if percent is 0 or blank then red. The columns that contain the data
> > > > > > > > are columns F-P. Is there any way to automate this process. Thanks in
> > > > > > > > advance. I am very new to VBA.
> > > > > > > >
> > > > > > > > Col F Col G Col H Col I Col J Col k Col L
> > > > > > > > Row 3 Jan Feb Mar Apr May Jun Jly
> > > > > > > >
> > > > > > > > Row 5 $500 $500 $500 $750 $750 $750 $1,000
> > > > > > > >
> > > > > > > > Row 9 $1000 $750 $500 $0 $1500 $900 $500
> > > > > > > > Many Rows just like Row 9
> > > > > > > >

Thanks its working great. The only thing that is not working is if the cell
is empty or blank it should also have a background color of red. I've been
trying to figure this out but no luck so far. Thanks again for all your
help.

"Toppers" wrote:

> Hi,
> The simplest way would be to clear ALL months at the beginning of
> the macro and just re-populate the current month.
>
> Range("F9:Q100").Select <=== change to 100 to whatever you think the max
> roes are going to be.
> Selection.Interior.ColorIndex = xlNone ' Clears the colours
>
> Place this code at the start of the macro - after DIM statements.
>
> HTH
>
>
> "Leslie" wrote:
>
> > Thanks so much for all your help that was so tricky for me. May I ask one
> > more question. I tried it out and everything works great but I think it is
> > missing something. Once July comes or any month thereafter I want it to
> > clear out the background colors from the previous month and only have the
> > current month highlighted. Is that possible to do?
> >
> > "Toppers" wrote:
> >
> > > Check the END IF are all there! It works OK on my machine!
> > >
> > > "Leslie" wrote:
> > >
> > > > It doesn't like the "Next Cell" at the bottom of the code. Thanks.
> > > >
> > > > "Toppers" wrote:
> > > >
> > > > > Hi again,
> > > > > You will get your error message if the cell is blank so I
> > > > > have added a test for cell value being a number.
> > > > >
> > > > > HTH
> > > > >
> > > > > Sub CFormat()
> > > > > '
> > > > > Dim rng As Range, cell As Range
> > > > > Dim ncol As Integer, lrow As Long
> > > > > Dim pcnt As Double, divisor As Double
> > > > >
> > > > > ThisWorkbook.Worksheets("Sheet1").Activate ' <=== Change to your w/sheet
> > > > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > > > ncol = Application.Match(Range("A2"), Range("F3:q3"), 0) + 5
> > > > >
> > > > > ' Find last row of data in current month column
> > > > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > > > >
> > > > > ' Set range to cells for current month starting row 9
> > > > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > > > >
> > > > > ' Set Divisor for current month
> > > > > divisor = Cells(5, ncol)
> > > > >
> > > > > ' Loop through all cells in range
> > > > > For Each cell In rng
> > > > > ' Check length of cell in column A
> > > > > If Len(cell.Offset(0, -(ncol - 1))) = 4 Then
> > > > > ' Calculate perecentage
> > > > > If Application.IsNumber(cell) Then ' Is this cell a number ?
> > > > > pcnt = (cell / divisor) * 100
> > > > > cell.Select
> > > > > ' Set colorindex based on percentage
> > > > > Select Case pcnt
> > > > > Case Is > 100
> > > > > Selection.Interior.ColorIndex = 4
> > > > > Case Is >= 90
> > > > > Selection.Interior.ColorIndex = 35
> > > > > Case Is >= 80
> > > > > Selection.Interior.ColorIndex = 36
> > > > > Case Is >= 70
> > > > > Selection.Interior.ColorIndex = 7
> > > > > Case Is >= 1
> > > > > Selection.Interior.ColorIndex = 54
> > > > > Case Else
> > > > > Selection.Interior.ColorIndex = 3
> > > > > End Select
> > > > > End If
> > > > > End If
> > > > > Next cell
> > > > >
> > > > > End Sub
> > > > >
> > > > >
> > > > > "Leslie" wrote:
> > > > >
> > > > > > Okay, I've confused myself. The run-time error is 1004 after all. I think
> > > > > > it is stuck at:
> > > > > >
> > > > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > > > >
> > > > > > but since I am new to this VBA stuff I most likely am wrong. Does it have
> > > > > > something to do with "ncol". Thanks.
> > > > > >
> > > > > > "Toppers" wrote:
> > > > > >
> > > > > > > Hi,
> > > > > > > Try this: insert this code into a general module.
> > > > > > >
> > > > > > > I have defined the cell A2 as a named range called "CurMonth" and the month
> > > > > > > headers as a range "HdrMonths".
> > > > > > >
> > > > > > > Sub CFormat()
> > > > > > > '
> > > > > > > Dim rng As Range, cell As Range
> > > > > > > Dim ncol As Integer, lrow As Long
> > > > > > > Dim pcnt As Double, divisor As Double
> > > > > > >
> > > > > > > Thisworkbook.Worksheets("Sheet1").activate <=== Change to your w/sheet
> > > > > > >
> > > > > > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > > > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > > > > >
> > > > > > > ' Find last row of data in current month column
> > > > > > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > > > > > >
> > > > > > > ' Set range to cells for current month starting row 9
> > > > > > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > > > > > >
> > > > > > > ' Set Divisor for current month
> > > > > > > divisor = Cells(5, ncol)
> > > > > > >
> > > > > > > ' Loop through all cells in range
> > > > > > > For Each cell In rng
> > > > > > > ' Calculate perecentage
> > > > > > > pcnt = (cell / divisor) * 100
> > > > > > > cell.Select
> > > > > > > ' Set colorindex based on percentage
> > > > > > > Select Case pcnt
> > > > > > > Case Is > 100
> > > > > > > Selection.Interior.ColorIndex = 4
> > > > > > > Case Is >= 90
> > > > > > > Selection.Interior.ColorIndex = 35
> > > > > > > Case Is >= 80
> > > > > > > Selection.Interior.ColorIndex = 36
> > > > > > > Case Is >= 70
> > > > > > > Selection.Interior.ColorIndex = 7
> > > > > > > Case Is >= 1
> > > > > > > Selection.Interior.ColorIndex = 54
> > > > > > > Case Else
> > > > > > > Selection.Interior.ColorIndex = 3
> > > > > > > End Select
> > > > > > > Next cell
> > > > > > >
> > > > > > > End Sub
> > > > > > >
> > > > > > >
> > > > > > > HTH
> > > > > > >
> > > > > > > "Leslie" wrote:
> > > > > > >
> > > > > > > > First, the formatting will only highlight certain cells in a column when the
> > > > > > > > column header in Row 3 is the current month which is tied to A2 which is
> > > > > > > > =Text(now(),"mmm"). Second, only those cells in rows in which Column A has
> > > > > > > > four digit letters or numbers will be highlighted. What I need is if, for
> > > > > > > > example, "Col F Row 9" / "Col F Row 5" = 200% then the cell background will
> > > > > > > > be shaded bright green. Row 5 is static and used as the divisor for all. The
> > > > > > > > conditions are: if the percent is 100.1% or greater then background of cell
> > > > > > > > is bright green, if percent is 90-100% then light green, if percent is
> > > > > > > > 80-89.9% than light yellow, if percent is 70-79.9% than pink, if 1-69.9% then
> > > > > > > > purple, if percent is 0 or blank then red. The columns that contain the data
> > > > > > > > are columns F-P. Is there any way to automate this process. Thanks in
> > > > > > > > advance. I am very new to VBA.
> > > > > > > >
> > > > > > > > Col F Col G Col H Col I Col J Col k Col L
> > > > > > > > Row 3 Jan Feb Mar Apr May Jun Jly
> > > > > > > >
> > > > > > > > Row 5 $500 $500 $500 $750 $750 $750 $1,000
> > > > > > > >
> > > > > > > > Row 9 $1000 $750 $500 $0 $1500 $900 $500
> > > > > > > > Many Rows just like Row 9
> > > > > > > >

Add these lines after End Select

Else
cell.Select
Selection.Interior.ColorIndex = 3


i.e.

End Select
* Else
* cell.Select
* Selection.Interior.ColorIndex = 3
End If

"Leslie" wrote:

> Thanks its working great. The only thing that is not working is if the cell
> is empty or blank it should also have a background color of red. I've been
> trying to figure this out but no luck so far. Thanks again for all your
> help.
>
> "Toppers" wrote:
>
> > Hi,
> > The simplest way would be to clear ALL months at the beginning of
> > the macro and just re-populate the current month.
> >
> > Range("F9:Q100").Select <=== change to 100 to whatever you think the max
> > roes are going to be.
> > Selection.Interior.ColorIndex = xlNone ' Clears the colours
> >
> > Place this code at the start of the macro - after DIM statements.
> >
> > HTH
> >
> >
> > "Leslie" wrote:
> >
> > > Thanks so much for all your help that was so tricky for me. May I ask one
> > > more question. I tried it out and everything works great but I think it is
> > > missing something. Once July comes or any month thereafter I want it to
> > > clear out the background colors from the previous month and only have the
> > > current month highlighted. Is that possible to do?
> > >
> > > "Toppers" wrote:
> > >
> > > > Check the END IF are all there! It works OK on my machine!
> > > >
> > > > "Leslie" wrote:
> > > >
> > > > > It doesn't like the "Next Cell" at the bottom of the code. Thanks.
> > > > >
> > > > > "Toppers" wrote:
> > > > >
> > > > > > Hi again,
> > > > > > You will get your error message if the cell is blank so I
> > > > > > have added a test for cell value being a number.
> > > > > >
> > > > > > HTH
> > > > > >
> > > > > > Sub CFormat()
> > > > > > '
> > > > > > Dim rng As Range, cell As Range
> > > > > > Dim ncol As Integer, lrow As Long
> > > > > > Dim pcnt As Double, divisor As Double
> > > > > >
> > > > > > ThisWorkbook.Worksheets("Sheet1").Activate ' <=== Change to your w/sheet
> > > > > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > > > > ncol = Application.Match(Range("A2"), Range("F3:q3"), 0) + 5
> > > > > >
> > > > > > ' Find last row of data in current month column
> > > > > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > > > > >
> > > > > > ' Set range to cells for current month starting row 9
> > > > > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > > > > >
> > > > > > ' Set Divisor for current month
> > > > > > divisor = Cells(5, ncol)
> > > > > >
> > > > > > ' Loop through all cells in range
> > > > > > For Each cell In rng
> > > > > > ' Check length of cell in column A
> > > > > > If Len(cell.Offset(0, -(ncol - 1))) = 4 Then
> > > > > > ' Calculate perecentage
> > > > > > If Application.IsNumber(cell) Then ' Is this cell a number ?
> > > > > > pcnt = (cell / divisor) * 100
> > > > > > cell.Select
> > > > > > ' Set colorindex based on percentage
> > > > > > Select Case pcnt
> > > > > > Case Is > 100
> > > > > > Selection.Interior.ColorIndex = 4
> > > > > > Case Is >= 90
> > > > > > Selection.Interior.ColorIndex = 35
> > > > > > Case Is >= 80
> > > > > > Selection.Interior.ColorIndex = 36
> > > > > > Case Is >= 70
> > > > > > Selection.Interior.ColorIndex = 7
> > > > > > Case Is >= 1
> > > > > > Selection.Interior.ColorIndex = 54
> > > > > > Case Else
> > > > > > Selection.Interior.ColorIndex = 3
> > > > > > End Select
> > > > > > End If
> > > > > > End If
> > > > > > Next cell
> > > > > >
> > > > > > End Sub
> > > > > >
> > > > > >
> > > > > > "Leslie" wrote:
> > > > > >
> > > > > > > Okay, I've confused myself. The run-time error is 1004 after all. I think
> > > > > > > it is stuck at:
> > > > > > >
> > > > > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > > > > >
> > > > > > > but since I am new to this VBA stuff I most likely am wrong. Does it have
> > > > > > > something to do with "ncol". Thanks.
> > > > > > >
> > > > > > > "Toppers" wrote:
> > > > > > >
> > > > > > > > Hi,
> > > > > > > > Try this: insert this code into a general module.
> > > > > > > >
> > > > > > > > I have defined the cell A2 as a named range called "CurMonth" and the month
> > > > > > > > headers as a range "HdrMonths".
> > > > > > > >
> > > > > > > > Sub CFormat()
> > > > > > > > '
> > > > > > > > Dim rng As Range, cell As Range
> > > > > > > > Dim ncol As Integer, lrow As Long
> > > > > > > > Dim pcnt As Double, divisor As Double
> > > > > > > >
> > > > > > > > Thisworkbook.Worksheets("Sheet1").activate <=== Change to your w/sheet
> > > > > > > >
> > > > > > > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > > > > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > > > > > >
> > > > > > > > ' Find last row of data in current month column
> > > > > > > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > > > > > > >
> > > > > > > > ' Set range to cells for current month starting row 9
> > > > > > > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > > > > > > >
> > > > > > > > ' Set Divisor for current month
> > > > > > > > divisor = Cells(5, ncol)
> > > > > > > >
> > > > > > > > ' Loop through all cells in range
> > > > > > > > For Each cell In rng
> > > > > > > > ' Calculate perecentage
> > > > > > > > pcnt = (cell / divisor) * 100
> > > > > > > > cell.Select
> > > > > > > > ' Set colorindex based on percentage
> > > > > > > > Select Case pcnt
> > > > > > > > Case Is > 100
> > > > > > > > Selection.Interior.ColorIndex = 4
> > > > > > > > Case Is >= 90
> > > > > > > > Selection.Interior.ColorIndex = 35
> > > > > > > > Case Is >= 80
> > > > > > > > Selection.Interior.ColorIndex = 36
> > > > > > > > Case Is >= 70
> > > > > > > > Selection.Interior.ColorIndex = 7
> > > > > > > > Case Is >= 1
> > > > > > > > Selection.Interior.ColorIndex = 54
> > > > > > > > Case Else
> > > > > > > > Selection.Interior.ColorIndex = 3
> > > > > > > > End Select
> > > > > > > > Next cell
> > > > > > > >
> > > > > > > > End Sub
> > > > > > > >
> > > > > > > >
> > > > > > > > HTH
> > > > > > > >
> > > > > > > > "Leslie" wrote:
> > > > > > > >
> > > > > > > > > First, the formatting will only highlight certain cells in a column when the
> > > > > > > > > column header in Row 3 is the current month which is tied to A2 which is
> > > > > > > > > =Text(now(),"mmm"). Second, only those cells in rows in which Column A has
> > > > > > > > > four digit letters or numbers will be highlighted. What I need is if, for
> > > > > > > > > example, "Col F Row 9" / "Col F Row 5" = 200% then the cell background will
> > > > > > > > > be shaded bright green. Row 5 is static and used as the divisor for all. The
> > > > > > > > > conditions are: if the percent is 100.1% or greater then background of cell
> > > > > > > > > is bright green, if percent is 90-100% then light green, if percent is
> > > > > > > > > 80-89.9% than light yellow, if percent is 70-79.9% than pink, if 1-69.9% then
> > > > > > > > > purple, if percent is 0 or blank then red. The columns that contain the data
> > > > > > > > > are columns F-P. Is there any way to automate this process. Thanks in
> > > > > > > > > advance. I am very new to VBA.
> > > > > > > > >
> > > > > > > > > Col F Col G Col H Col I Col J Col k Col L
> > > > > > > > > Row 3 Jan Feb Mar Apr May Jun Jly
> > > > > > > > >
> > > > > > > > > Row 5 $500 $500 $500 $750 $750 $750 $1,000
> > > > > > > > >
> > > > > > > > > Row 9 $1000 $750 $500 $0 $1500 $900 $500
> > > > > > > > > Many Rows just like Row 9
> > > > > > > > >

Yeah! Its working BUT, another But, Very oddly it colored all the blank cells
up to row 102 and then just stopped. I can't imagine why that is.

"Toppers" wrote:

> Add these lines after End Select
>
> Else
> cell.Select
> Selection.Interior.ColorIndex = 3
>
>
> i.e.
>
> End Select
> * Else
> * cell.Select
> * Selection.Interior.ColorIndex = 3
> End If
>
> "Leslie" wrote:
>
> > Thanks its working great. The only thing that is not working is if the cell
> > is empty or blank it should also have a background color of red. I've been
> > trying to figure this out but no luck so far. Thanks again for all your
> > help.
> >
> > "Toppers" wrote:
> >
> > > Hi,
> > > The simplest way would be to clear ALL months at the beginning of
> > > the macro and just re-populate the current month.
> > >
> > > Range("F9:Q100").Select <=== change to 100 to whatever you think the max
> > > roes are going to be.
> > > Selection.Interior.ColorIndex = xlNone ' Clears the colours
> > >
> > > Place this code at the start of the macro - after DIM statements.
> > >
> > > HTH
> > >
> > >
> > > "Leslie" wrote:
> > >
> > > > Thanks so much for all your help that was so tricky for me. May I ask one
> > > > more question. I tried it out and everything works great but I think it is
> > > > missing something. Once July comes or any month thereafter I want it to
> > > > clear out the background colors from the previous month and only have the
> > > > current month highlighted. Is that possible to do?
> > > >
> > > > "Toppers" wrote:
> > > >
> > > > > Check the END IF are all there! It works OK on my machine!
> > > > >
> > > > > "Leslie" wrote:
> > > > >
> > > > > > It doesn't like the "Next Cell" at the bottom of the code. Thanks.
> > > > > >
> > > > > > "Toppers" wrote:
> > > > > >
> > > > > > > Hi again,
> > > > > > > You will get your error message if the cell is blank so I
> > > > > > > have added a test for cell value being a number.
> > > > > > >
> > > > > > > HTH
> > > > > > >
> > > > > > > Sub CFormat()
> > > > > > > '
> > > > > > > Dim rng As Range, cell As Range
> > > > > > > Dim ncol As Integer, lrow As Long
> > > > > > > Dim pcnt As Double, divisor As Double
> > > > > > >
> > > > > > > ThisWorkbook.Worksheets("Sheet1").Activate ' <=== Change to your w/sheet
> > > > > > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > > > > > ncol = Application.Match(Range("A2"), Range("F3:q3"), 0) + 5
> > > > > > >
> > > > > > > ' Find last row of data in current month column
> > > > > > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > > > > > >
> > > > > > > ' Set range to cells for current month starting row 9
> > > > > > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > > > > > >
> > > > > > > ' Set Divisor for current month
> > > > > > > divisor = Cells(5, ncol)
> > > > > > >
> > > > > > > ' Loop through all cells in range
> > > > > > > For Each cell In rng
> > > > > > > ' Check length of cell in column A
> > > > > > > If Len(cell.Offset(0, -(ncol - 1))) = 4 Then
> > > > > > > ' Calculate perecentage
> > > > > > > If Application.IsNumber(cell) Then ' Is this cell a number ?
> > > > > > > pcnt = (cell / divisor) * 100
> > > > > > > cell.Select
> > > > > > > ' Set colorindex based on percentage
> > > > > > > Select Case pcnt
> > > > > > > Case Is > 100
> > > > > > > Selection.Interior.ColorIndex = 4
> > > > > > > Case Is >= 90
> > > > > > > Selection.Interior.ColorIndex = 35
> > > > > > > Case Is >= 80
> > > > > > > Selection.Interior.ColorIndex = 36
> > > > > > > Case Is >= 70
> > > > > > > Selection.Interior.ColorIndex = 7
> > > > > > > Case Is >= 1
> > > > > > > Selection.Interior.ColorIndex = 54
> > > > > > > Case Else
> > > > > > > Selection.Interior.ColorIndex = 3
> > > > > > > End Select
> > > > > > > End If
> > > > > > > End If
> > > > > > > Next cell
> > > > > > >
> > > > > > > End Sub
> > > > > > >
> > > > > > >
> > > > > > > "Leslie" wrote:
> > > > > > >
> > > > > > > > Okay, I've confused myself. The run-time error is 1004 after all. I think
> > > > > > > > it is stuck at:
> > > > > > > >
> > > > > > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > > > > > >
> > > > > > > > but since I am new to this VBA stuff I most likely am wrong. Does it have
> > > > > > > > something to do with "ncol". Thanks.
> > > > > > > >
> > > > > > > > "Toppers" wrote:
> > > > > > > >
> > > > > > > > > Hi,
> > > > > > > > > Try this: insert this code into a general module.
> > > > > > > > >
> > > > > > > > > I have defined the cell A2 as a named range called "CurMonth" and the month
> > > > > > > > > headers as a range "HdrMonths".
> > > > > > > > >
> > > > > > > > > Sub CFormat()
> > > > > > > > > '
> > > > > > > > > Dim rng As Range, cell As Range
> > > > > > > > > Dim ncol As Integer, lrow As Long
> > > > > > > > > Dim pcnt As Double, divisor As Double
> > > > > > > > >
> > > > > > > > > Thisworkbook.Worksheets("Sheet1").activate <=== Change to your w/sheet
> > > > > > > > >
> > > > > > > > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > > > > > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > > > > > > >
> > > > > > > > > ' Find last row of data in current month column
> > > > > > > > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > > > > > > > >
> > > > > > > > > ' Set range to cells for current month starting row 9
> > > > > > > > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > > > > > > > >
> > > > > > > > > ' Set Divisor for current month
> > > > > > > > > divisor = Cells(5, ncol)
> > > > > > > > >
> > > > > > > > > ' Loop through all cells in range
> > > > > > > > > For Each cell In rng
> > > > > > > > > ' Calculate perecentage
> > > > > > > > > pcnt = (cell / divisor) * 100
> > > > > > > > > cell.Select
> > > > > > > > > ' Set colorindex based on percentage
> > > > > > > > > Select Case pcnt
> > > > > > > > > Case Is > 100
> > > > > > > > > Selection.Interior.ColorIndex = 4
> > > > > > > > > Case Is >= 90
> > > > > > > > > Selection.Interior.ColorIndex = 35
> > > > > > > > > Case Is >= 80
> > > > > > > > > Selection.Interior.ColorIndex = 36
> > > > > > > > > Case Is >= 70
> > > > > > > > > Selection.Interior.ColorIndex = 7
> > > > > > > > > Case Is >= 1
> > > > > > > > > Selection.Interior.ColorIndex = 54
> > > > > > > > > Case Else
> > > > > > > > > Selection.Interior.ColorIndex = 3
> > > > > > > > > End Select
> > > > > > > > > Next cell
> > > > > > > > >
> > > > > > > > > End Sub
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > HTH
> > > > > > > > >
> > > > > > > > > "Leslie" wrote:
> > > > > > > > >
> > > > > > > > > > First, the formatting will only highlight certain cells in a column when the
> > > > > > > > > > column header in Row 3 is the current month which is tied to A2 which is
> > > > > > > > > > =Text(now(),"mmm"). Second, only those cells in rows in which Column A has
> > > > > > > > > > four digit letters or numbers will be highlighted. What I need is if, for
> > > > > > > > > > example, "Col F Row 9" / "Col F Row 5" = 200% then the cell background will
> > > > > > > > > > be shaded bright green. Row 5 is static and used as the divisor for all. The
> > > > > > > > > > conditions are: if the percent is 100.1% or greater then background of cell
> > > > > > > > > > is bright green, if percent is 90-100% then light green, if percent is
> > > > > > > > > > 80-89.9% than light yellow, if percent is 70-79.9% than pink, if 1-69.9% then
> > > > > > > > > > purple, if percent is 0 or blank then red. The columns that contain the data
> > > > > > > > > > are columns F-P. Is there any way to automate this process. Thanks in
> > > > > > > > > > advance. I am very new to VBA.
> > > > > > > > > >
> > > > > > > > > > Col F Col G Col H Col I Col J Col k Col L
> > > > > > > > > > Row 3 Jan Feb Mar Apr May Jun Jly
> > > > > > > > > >
> > > > > > > > > > Row 5 $500 $500 $500 $750 $750 $750 $1,000
> > > > > > > > > >
> > > > > > > > > > Row 9 $1000 $750 $500 $0 $1500 $900 $500
> > > > > > > > > > Many Rows just like Row 9
> > > > > > > > > >

Becaues (I am "guessing") row 102 is the last non-blank cell in the current
month; this what lrowl calculates. If you want to use column A as the
delimeteri .e column A is always the longest, then change ncol in the lrow
statement to 1 i.e.

lrow = Cells(Rows.Count, 1).End(xlUp).Row.

You could then modify the code to clear the colours as below and move AFTER
the lrow line.

Range("F9:Q" & lrow).Select
Selection.Interior.ColorIndex = xlNone


"Leslie" wrote:

> Yeah! Its working BUT, another But, Very oddly it colored all the blank cells
> up to row 102 and then just stopped. I can't imagine why that is.
>
> "Toppers" wrote:
>
> > Add these lines after End Select
> >
> > Else
> > cell.Select
> > Selection.Interior.ColorIndex = 3
> >
> >
> > i.e.
> >
> > End Select
> > * Else
> > * cell.Select
> > * Selection.Interior.ColorIndex = 3
> > End If
> >
> > "Leslie" wrote:
> >
> > > Thanks its working great. The only thing that is not working is if the cell
> > > is empty or blank it should also have a background color of red. I've been
> > > trying to figure this out but no luck so far. Thanks again for all your
> > > help.
> > >
> > > "Toppers" wrote:
> > >
> > > > Hi,
> > > > The simplest way would be to clear ALL months at the beginning of
> > > > the macro and just re-populate the current month.
> > > >
> > > > Range("F9:Q100").Select <=== change to 100 to whatever you think the max
> > > > roes are going to be.
> > > > Selection.Interior.ColorIndex = xlNone ' Clears the colours
> > > >
> > > > Place this code at the start of the macro - after DIM statements.
> > > >
> > > > HTH
> > > >
> > > >
> > > > "Leslie" wrote:
> > > >
> > > > > Thanks so much for all your help that was so tricky for me. May I ask one
> > > > > more question. I tried it out and everything works great but I think it is
> > > > > missing something. Once July comes or any month thereafter I want it to
> > > > > clear out the background colors from the previous month and only have the
> > > > > current month highlighted. Is that possible to do?
> > > > >
> > > > > "Toppers" wrote:
> > > > >
> > > > > > Check the END IF are all there! It works OK on my machine!
> > > > > >
> > > > > > "Leslie" wrote:
> > > > > >
> > > > > > > It doesn't like the "Next Cell" at the bottom of the code. Thanks.
> > > > > > >
> > > > > > > "Toppers" wrote:
> > > > > > >
> > > > > > > > Hi again,
> > > > > > > > You will get your error message if the cell is blank so I
> > > > > > > > have added a test for cell value being a number.
> > > > > > > >
> > > > > > > > HTH
> > > > > > > >
> > > > > > > > Sub CFormat()
> > > > > > > > '
> > > > > > > > Dim rng As Range, cell As Range
> > > > > > > > Dim ncol As Integer, lrow As Long
> > > > > > > > Dim pcnt As Double, divisor As Double
> > > > > > > >
> > > > > > > > ThisWorkbook.Worksheets("Sheet1").Activate ' <=== Change to your w/sheet
> > > > > > > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > > > > > > ncol = Application.Match(Range("A2"), Range("F3:q3"), 0) + 5
> > > > > > > >
> > > > > > > > ' Find last row of data in current month column
> > > > > > > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > > > > > > >
> > > > > > > > ' Set range to cells for current month starting row 9
> > > > > > > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > > > > > > >
> > > > > > > > ' Set Divisor for current month
> > > > > > > > divisor = Cells(5, ncol)
> > > > > > > >
> > > > > > > > ' Loop through all cells in range
> > > > > > > > For Each cell In rng
> > > > > > > > ' Check length of cell in column A
> > > > > > > > If Len(cell.Offset(0, -(ncol - 1))) = 4 Then
> > > > > > > > ' Calculate perecentage
> > > > > > > > If Application.IsNumber(cell) Then ' Is this cell a number ?
> > > > > > > > pcnt = (cell / divisor) * 100
> > > > > > > > cell.Select
> > > > > > > > ' Set colorindex based on percentage
> > > > > > > > Select Case pcnt
> > > > > > > > Case Is > 100
> > > > > > > > Selection.Interior.ColorIndex = 4
> > > > > > > > Case Is >= 90
> > > > > > > > Selection.Interior.ColorIndex = 35
> > > > > > > > Case Is >= 80
> > > > > > > > Selection.Interior.ColorIndex = 36
> > > > > > > > Case Is >= 70
> > > > > > > > Selection.Interior.ColorIndex = 7
> > > > > > > > Case Is >= 1
> > > > > > > > Selection.Interior.ColorIndex = 54
> > > > > > > > Case Else
> > > > > > > > Selection.Interior.ColorIndex = 3
> > > > > > > > End Select
> > > > > > > > End If
> > > > > > > > End If
> > > > > > > > Next cell
> > > > > > > >
> > > > > > > > End Sub
> > > > > > > >
> > > > > > > >
> > > > > > > > "Leslie" wrote:
> > > > > > > >
> > > > > > > > > Okay, I've confused myself. The run-time error is 1004 after all. I think
> > > > > > > > > it is stuck at:
> > > > > > > > >
> > > > > > > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > > > > > > >
> > > > > > > > > but since I am new to this VBA stuff I most likely am wrong. Does it have
> > > > > > > > > something to do with "ncol". Thanks.
> > > > > > > > >
> > > > > > > > > "Toppers" wrote:
> > > > > > > > >
> > > > > > > > > > Hi,
> > > > > > > > > > Try this: insert this code into a general module.
> > > > > > > > > >
> > > > > > > > > > I have defined the cell A2 as a named range called "CurMonth" and the month
> > > > > > > > > > headers as a range "HdrMonths".
> > > > > > > > > >
> > > > > > > > > > Sub CFormat()
> > > > > > > > > > '
> > > > > > > > > > Dim rng As Range, cell As Range
> > > > > > > > > > Dim ncol As Integer, lrow As Long
> > > > > > > > > > Dim pcnt As Double, divisor As Double
> > > > > > > > > >
> > > > > > > > > > Thisworkbook.Worksheets("Sheet1").activate <=== Change to your w/sheet
> > > > > > > > > >
> > > > > > > > > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > > > > > > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > > > > > > > >
> > > > > > > > > > ' Find last row of data in current month column
> > > > > > > > > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > > > > > > > > >
> > > > > > > > > > ' Set range to cells for current month starting row 9
> > > > > > > > > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > > > > > > > > >
> > > > > > > > > > ' Set Divisor for current month
> > > > > > > > > > divisor = Cells(5, ncol)
> > > > > > > > > >
> > > > > > > > > > ' Loop through all cells in range
> > > > > > > > > > For Each cell In rng
> > > > > > > > > > ' Calculate perecentage
> > > > > > > > > > pcnt = (cell / divisor) * 100
> > > > > > > > > > cell.Select
> > > > > > > > > > ' Set colorindex based on percentage
> > > > > > > > > > Select Case pcnt
> > > > > > > > > > Case Is > 100
> > > > > > > > > > Selection.Interior.ColorIndex = 4
> > > > > > > > > > Case Is >= 90
> > > > > > > > > > Selection.Interior.ColorIndex = 35
> > > > > > > > > > Case Is >= 80
> > > > > > > > > > Selection.Interior.ColorIndex = 36
> > > > > > > > > > Case Is >= 70
> > > > > > > > > > Selection.Interior.ColorIndex = 7
> > > > > > > > > > Case Is >= 1
> > > > > > > > > > Selection.Interior.ColorIndex = 54
> > > > > > > > > > Case Else
> > > > > > > > > > Selection.Interior.ColorIndex = 3
> > > > > > > > > > End Select
> > > > > > > > > > Next cell
> > > > > > > > > >
> > > > > > > > > > End Sub
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > > HTH
> > > > > > > > > >
> > > > > > > > > > "Leslie" wrote:
> > > > > > > > > >
> > > > > > > > > > > First, the formatting will only highlight certain cells in a column when the
> > > > > > > > > > > column header in Row 3 is the current month which is tied to A2 which is
> > > > > > > > > > > =Text(now(),"mmm"). Second, only those cells in rows in which Column A has
> > > > > > > > > > > four digit letters or numbers will be highlighted. What I need is if, for
> > > > > > > > > > > example, "Col F Row 9" / "Col F Row 5" = 200% then the cell background will
> > > > > > > > > > > be shaded bright green. Row 5 is static and used as the divisor for all. The
> > > > > > > > > > > conditions are: if the percent is 100.1% or greater then background of cell
> > > > > > > > > > > is bright green, if percent is 90-100% then light green, if percent is
> > > > > > > > > > > 80-89.9% than light yellow, if percent is 70-79.9% than pink, if 1-69.9% then
> > > > > > > > > > > purple, if percent is 0 or blank then red. The columns that contain the data
> > > > > > > > > > > are columns F-P. Is there any way to automate this process. Thanks in
> > > > > > > > > > > advance. I am very new to VBA.
> > > > > > > > > > >
> > > > > > > > > > > Col F Col G Col H Col I Col J Col k Col L
> > > > > > > > > > > Row 3 Jan Feb Mar Apr May Jun Jly
> > > > > > > > > > >
> > > > > > > > > > > Row 5 $500 $500 $500 $750 $750 $750 $1,000
> > > > > > > > > > >
> > > > > > > > > > > Row 9 $1000 $750 $500 $0 $1500 $900 $500
> > > > > > > > > > > Many Rows just like Row 9
> > > > > > > > > > >

Thanks again. That did fix it even though row 102 was not the last non-blank
cell in the current month. Regarding the clearing color with the code we are
using it clears out other coloring I have in certain rows. Is there a way to
limit the clearing of colors to just what we coded?

"Toppers" wrote:

> Becaues (I am "guessing") row 102 is the last non-blank cell in the current
> month; this what lrowl calculates. If you want to use column A as the
> delimeteri .e column A is always the longest, then change ncol in the lrow
> statement to 1 i.e.
>
> lrow = Cells(Rows.Count, 1).End(xlUp).Row.
>
> You could then modify the code to clear the colours as below and move AFTER
> the lrow line.
>
> Range("F9:Q" & lrow).Select
> Selection.Interior.ColorIndex = xlNone
>
>
> "Leslie" wrote:
>
> > Yeah! Its working BUT, another But, Very oddly it colored all the blank cells
> > up to row 102 and then just stopped. I can't imagine why that is.
> >
> > "Toppers" wrote:
> >
> > > Add these lines after End Select
> > >
> > > Else
> > > cell.Select
> > > Selection.Interior.ColorIndex = 3
> > >
> > >
> > > i.e.
> > >
> > > End Select
> > > * Else
> > > * cell.Select
> > > * Selection.Interior.ColorIndex = 3
> > > End If
> > >
> > > "Leslie" wrote:
> > >
> > > > Thanks its working great. The only thing that is not working is if the cell
> > > > is empty or blank it should also have a background color of red. I've been
> > > > trying to figure this out but no luck so far. Thanks again for all your
> > > > help.
> > > >
> > > > "Toppers" wrote:
> > > >
> > > > > Hi,
> > > > > The simplest way would be to clear ALL months at the beginning of
> > > > > the macro and just re-populate the current month.
> > > > >
> > > > > Range("F9:Q100").Select <=== change to 100 to whatever you think the max
> > > > > roes are going to be.
> > > > > Selection.Interior.ColorIndex = xlNone ' Clears the colours
> > > > >
> > > > > Place this code at the start of the macro - after DIM statements.
> > > > >
> > > > > HTH
> > > > >
> > > > >
> > > > > "Leslie" wrote:
> > > > >
> > > > > > Thanks so much for all your help that was so tricky for me. May I ask one
> > > > > > more question. I tried it out and everything works great but I think it is
> > > > > > missing something. Once July comes or any month thereafter I want it to
> > > > > > clear out the background colors from the previous month and only have the
> > > > > > current month highlighted. Is that possible to do?
> > > > > >
> > > > > > "Toppers" wrote:
> > > > > >
> > > > > > > Check the END IF are all there! It works OK on my machine!
> > > > > > >
> > > > > > > "Leslie" wrote:
> > > > > > >
> > > > > > > > It doesn't like the "Next Cell" at the bottom of the code. Thanks.
> > > > > > > >
> > > > > > > > "Toppers" wrote:
> > > > > > > >
> > > > > > > > > Hi again,
> > > > > > > > > You will get your error message if the cell is blank so I
> > > > > > > > > have added a test for cell value being a number.
> > > > > > > > >
> > > > > > > > > HTH
> > > > > > > > >
> > > > > > > > > Sub CFormat()
> > > > > > > > > '
> > > > > > > > > Dim rng As Range, cell As Range
> > > > > > > > > Dim ncol As Integer, lrow As Long
> > > > > > > > > Dim pcnt As Double, divisor As Double
> > > > > > > > >
> > > > > > > > > ThisWorkbook.Worksheets("Sheet1").Activate ' <=== Change to your w/sheet
> > > > > > > > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > > > > > > > ncol = Application.Match(Range("A2"), Range("F3:q3"), 0) + 5
> > > > > > > > >
> > > > > > > > > ' Find last row of data in current month column
> > > > > > > > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > > > > > > > >
> > > > > > > > > ' Set range to cells for current month starting row 9
> > > > > > > > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > > > > > > > >
> > > > > > > > > ' Set Divisor for current month
> > > > > > > > > divisor = Cells(5, ncol)
> > > > > > > > >
> > > > > > > > > ' Loop through all cells in range
> > > > > > > > > For Each cell In rng
> > > > > > > > > ' Check length of cell in column A
> > > > > > > > > If Len(cell.Offset(0, -(ncol - 1))) = 4 Then
> > > > > > > > > ' Calculate perecentage
> > > > > > > > > If Application.IsNumber(cell) Then ' Is this cell a number ?
> > > > > > > > > pcnt = (cell / divisor) * 100
> > > > > > > > > cell.Select
> > > > > > > > > ' Set colorindex based on percentage
> > > > > > > > > Select Case pcnt
> > > > > > > > > Case Is > 100
> > > > > > > > > Selection.Interior.ColorIndex = 4
> > > > > > > > > Case Is >= 90
> > > > > > > > > Selection.Interior.ColorIndex = 35
> > > > > > > > > Case Is >= 80
> > > > > > > > > Selection.Interior.ColorIndex = 36
> > > > > > > > > Case Is >= 70
> > > > > > > > > Selection.Interior.ColorIndex = 7
> > > > > > > > > Case Is >= 1
> > > > > > > > > Selection.Interior.ColorIndex = 54
> > > > > > > > > Case Else
> > > > > > > > > Selection.Interior.ColorIndex = 3
> > > > > > > > > End Select
> > > > > > > > > End If
> > > > > > > > > End If
> > > > > > > > > Next cell
> > > > > > > > >
> > > > > > > > > End Sub
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > "Leslie" wrote:
> > > > > > > > >
> > > > > > > > > > Okay, I've confused myself. The run-time error is 1004 after all. I think
> > > > > > > > > > it is stuck at:
> > > > > > > > > >
> > > > > > > > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > > > > > > > >
> > > > > > > > > > but since I am new to this VBA stuff I most likely am wrong. Does it have
> > > > > > > > > > something to do with "ncol". Thanks.
> > > > > > > > > >
> > > > > > > > > > "Toppers" wrote:
> > > > > > > > > >
> > > > > > > > > > > Hi,
> > > > > > > > > > > Try this: insert this code into a general module.
> > > > > > > > > > >
> > > > > > > > > > > I have defined the cell A2 as a named range called "CurMonth" and the month
> > > > > > > > > > > headers as a range "HdrMonths".
> > > > > > > > > > >
> > > > > > > > > > > Sub CFormat()
> > > > > > > > > > > '
> > > > > > > > > > > Dim rng As Range, cell As Range
> > > > > > > > > > > Dim ncol As Integer, lrow As Long
> > > > > > > > > > > Dim pcnt As Double, divisor As Double
> > > > > > > > > > >
> > > > > > > > > > > Thisworkbook.Worksheets("Sheet1").activate <=== Change to your w/sheet
> > > > > > > > > > >
> > > > > > > > > > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > > > > > > > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > > > > > > > > >
> > > > > > > > > > > ' Find last row of data in current month column
> > > > > > > > > > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > > > > > > > > > >
> > > > > > > > > > > ' Set range to cells for current month starting row 9
> > > > > > > > > > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > > > > > > > > > >
> > > > > > > > > > > ' Set Divisor for current month
> > > > > > > > > > > divisor = Cells(5, ncol)
> > > > > > > > > > >
> > > > > > > > > > > ' Loop through all cells in range
> > > > > > > > > > > For Each cell In rng
> > > > > > > > > > > ' Calculate perecentage
> > > > > > > > > > > pcnt = (cell / divisor) * 100
> > > > > > > > > > > cell.Select
> > > > > > > > > > > ' Set colorindex based on percentage
> > > > > > > > > > > Select Case pcnt
> > > > > > > > > > > Case Is > 100
> > > > > > > > > > > Selection.Interior.ColorIndex = 4
> > > > > > > > > > > Case Is >= 90
> > > > > > > > > > > Selection.Interior.ColorIndex = 35
> > > > > > > > > > > Case Is >= 80
> > > > > > > > > > > Selection.Interior.ColorIndex = 36
> > > > > > > > > > > Case Is >= 70
> > > > > > > > > > > Selection.Interior.ColorIndex = 7
> > > > > > > > > > > Case Is >= 1
> > > > > > > > > > > Selection.Interior.ColorIndex = 54
> > > > > > > > > > > Case Else
> > > > > > > > > > > Selection.Interior.ColorIndex = 3
> > > > > > > > > > > End Select
> > > > > > > > > > > Next cell
> > > > > > > > > > >
> > > > > > > > > > > End Sub
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > HTH
> > > > > > > > > > >
> > > > > > > > > > > "Leslie" wrote:
> > > > > > > > > > >
> > > > > > > > > > > > First, the formatting will only highlight certain cells in a column when the
> > > > > > > > > > > > column header in Row 3 is the current month which is tied to A2 which is
> > > > > > > > > > > > =Text(now(),"mmm"). Second, only those cells in rows in which Column A has
> > > > > > > > > > > > four digit letters or numbers will be highlighted. What I need is if, for
> > > > > > > > > > > > example, "Col F Row 9" / "Col F Row 5" = 200% then the cell background will
> > > > > > > > > > > > be shaded bright green. Row 5 is static and used as the divisor for all. The
> > > > > > > > > > > > conditions are: if the percent is 100.1% or greater then background of cell
> > > > > > > > > > > > is bright green, if percent is 90-100% then light green, if percent is
> > > > > > > > > > > > 80-89.9% than light yellow, if percent is 70-79.9% than pink, if 1-69.9% then
> > > > > > > > > > > > purple, if percent is 0 or blank then red. The columns that contain the data
> > > > > > > > > > > > are columns F-P. Is there any way to automate this process. Thanks in
> > > > > > > > > > > > advance. I am very new to VBA.
> > > > > > > > > > > >
> > > > > > > > > > > > Col F Col G Col H Col I Col J Col k Col L
> > > > > > > > > > > > Row 3 Jan Feb Mar Apr May Jun Jly
> > > > > > > > > > > >
> > > > > > > > > > > > Row 5 $500 $500 $500 $750 $750 $750 $1,000
> > > > > > > > > > > >
> > > > > > > > > > > > Row 9 $1000 $750 $500 $0 $1500 $900 $500
> > > > > > > > > > > > Many Rows just like Row 9
> > > > > > > > > > > >

One more thing. How is the coding executed? Is it when I open the
worksheet or workbook or do I have to press F5 all the time? Thanks.

"Leslie" wrote:

> Yeah! Its working BUT, another But, Very oddly it colored all the blank cells
> up to row 102 and then just stopped. I can't imagine why that is.
>
> "Toppers" wrote:
>
> > Add these lines after End Select
> >
> > Else
> > cell.Select
> > Selection.Interior.ColorIndex = 3
> >
> >
> > i.e.
> >
> > End Select
> > * Else
> > * cell.Select
> > * Selection.Interior.ColorIndex = 3
> > End If
> >
> > "Leslie" wrote:
> >
> > > Thanks its working great. The only thing that is not working is if the cell
> > > is empty or blank it should also have a background color of red. I've been
> > > trying to figure this out but no luck so far. Thanks again for all your
> > > help.
> > >
> > > "Toppers" wrote:
> > >
> > > > Hi,
> > > > The simplest way would be to clear ALL months at the beginning of
> > > > the macro and just re-populate the current month.
> > > >
> > > > Range("F9:Q100").Select <=== change to 100 to whatever you think the max
> > > > roes are going to be.
> > > > Selection.Interior.ColorIndex = xlNone ' Clears the colours
> > > >
> > > > Place this code at the start of the macro - after DIM statements.
> > > >
> > > > HTH
> > > >
> > > >
> > > > "Leslie" wrote:
> > > >
> > > > > Thanks so much for all your help that was so tricky for me. May I ask one
> > > > > more question. I tried it out and everything works great but I think it is
> > > > > missing something. Once July comes or any month thereafter I want it to
> > > > > clear out the background colors from the previous month and only have the
> > > > > current month highlighted. Is that possible to do?
> > > > >
> > > > > "Toppers" wrote:
> > > > >
> > > > > > Check the END IF are all there! It works OK on my machine!
> > > > > >
> > > > > > "Leslie" wrote:
> > > > > >
> > > > > > > It doesn't like the "Next Cell" at the bottom of the code. Thanks.
> > > > > > >
> > > > > > > "Toppers" wrote:
> > > > > > >
> > > > > > > > Hi again,
> > > > > > > > You will get your error message if the cell is blank so I
> > > > > > > > have added a test for cell value being a number.
> > > > > > > >
> > > > > > > > HTH
> > > > > > > >
> > > > > > > > Sub CFormat()
> > > > > > > > '
> > > > > > > > Dim rng As Range, cell As Range
> > > > > > > > Dim ncol As Integer, lrow As Long
> > > > > > > > Dim pcnt As Double, divisor As Double
> > > > > > > >
> > > > > > > > ThisWorkbook.Worksheets("Sheet1").Activate ' <=== Change to your w/sheet
> > > > > > > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > > > > > > ncol = Application.Match(Range("A2"), Range("F3:q3"), 0) + 5
> > > > > > > >
> > > > > > > > ' Find last row of data in current month column
> > > > > > > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > > > > > > >
> > > > > > > > ' Set range to cells for current month starting row 9
> > > > > > > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > > > > > > >
> > > > > > > > ' Set Divisor for current month
> > > > > > > > divisor = Cells(5, ncol)
> > > > > > > >
> > > > > > > > ' Loop through all cells in range
> > > > > > > > For Each cell In rng
> > > > > > > > ' Check length of cell in column A
> > > > > > > > If Len(cell.Offset(0, -(ncol - 1))) = 4 Then
> > > > > > > > ' Calculate perecentage
> > > > > > > > If Application.IsNumber(cell) Then ' Is this cell a number ?
> > > > > > > > pcnt = (cell / divisor) * 100
> > > > > > > > cell.Select
> > > > > > > > ' Set colorindex based on percentage
> > > > > > > > Select Case pcnt
> > > > > > > > Case Is > 100
> > > > > > > > Selection.Interior.ColorIndex = 4
> > > > > > > > Case Is >= 90
> > > > > > > > Selection.Interior.ColorIndex = 35
> > > > > > > > Case Is >= 80
> > > > > > > > Selection.Interior.ColorIndex = 36
> > > > > > > > Case Is >= 70
> > > > > > > > Selection.Interior.ColorIndex = 7
> > > > > > > > Case Is >= 1
> > > > > > > > Selection.Interior.ColorIndex = 54
> > > > > > > > Case Else
> > > > > > > > Selection.Interior.ColorIndex = 3
> > > > > > > > End Select
> > > > > > > > End If
> > > > > > > > End If
> > > > > > > > Next cell
> > > > > > > >
> > > > > > > > End Sub
> > > > > > > >
> > > > > > > >
> > > > > > > > "Leslie" wrote:
> > > > > > > >
> > > > > > > > > Okay, I've confused myself. The run-time error is 1004 after all. I think
> > > > > > > > > it is stuck at:
> > > > > > > > >
> > > > > > > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > > > > > > >
> > > > > > > > > but since I am new to this VBA stuff I most likely am wrong. Does it have
> > > > > > > > > something to do with "ncol". Thanks.
> > > > > > > > >
> > > > > > > > > "Toppers" wrote:
> > > > > > > > >
> > > > > > > > > > Hi,
> > > > > > > > > > Try this: insert this code into a general module.
> > > > > > > > > >
> > > > > > > > > > I have defined the cell A2 as a named range called "CurMonth" and the month
> > > > > > > > > > headers as a range "HdrMonths".
> > > > > > > > > >
> > > > > > > > > > Sub CFormat()
> > > > > > > > > > '
> > > > > > > > > > Dim rng As Range, cell As Range
> > > > > > > > > > Dim ncol As Integer, lrow As Long
> > > > > > > > > > Dim pcnt As Double, divisor As Double
> > > > > > > > > >
> > > > > > > > > > Thisworkbook.Worksheets("Sheet1").activate <=== Change to your w/sheet
> > > > > > > > > >
> > > > > > > > > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > > > > > > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > > > > > > > >
> > > > > > > > > > ' Find last row of data in current month column
> > > > > > > > > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > > > > > > > > >
> > > > > > > > > > ' Set range to cells for current month starting row 9
> > > > > > > > > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > > > > > > > > >
> > > > > > > > > > ' Set Divisor for current month
> > > > > > > > > > divisor = Cells(5, ncol)
> > > > > > > > > >
> > > > > > > > > > ' Loop through all cells in range
> > > > > > > > > > For Each cell In rng
> > > > > > > > > > ' Calculate perecentage
> > > > > > > > > > pcnt = (cell / divisor) * 100
> > > > > > > > > > cell.Select
> > > > > > > > > > ' Set colorindex based on percentage
> > > > > > > > > > Select Case pcnt
> > > > > > > > > > Case Is > 100
> > > > > > > > > > Selection.Interior.ColorIndex = 4
> > > > > > > > > > Case Is >= 90
> > > > > > > > > > Selection.Interior.ColorIndex = 35
> > > > > > > > > > Case Is >= 80
> > > > > > > > > > Selection.Interior.ColorIndex = 36
> > > > > > > > > > Case Is >= 70
> > > > > > > > > > Selection.Interior.ColorIndex = 7
> > > > > > > > > > Case Is >= 1
> > > > > > > > > > Selection.Interior.ColorIndex = 54
> > > > > > > > > > Case Else
> > > > > > > > > > Selection.Interior.ColorIndex = 3
> > > > > > > > > > End Select
> > > > > > > > > > Next cell
> > > > > > > > > >
> > > > > > > > > > End Sub
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > > HTH
> > > > > > > > > >
> > > > > > > > > > "Leslie" wrote:
> > > > > > > > > >
> > > > > > > > > > > First, the formatting will only highlight certain cells in a column when the
> > > > > > > > > > > column header in Row 3 is the current month which is tied to A2 which is
> > > > > > > > > > > =Text(now(),"mmm"). Second, only those cells in rows in which Column A has
> > > > > > > > > > > four digit letters or numbers will be highlighted. What I need is if, for
> > > > > > > > > > > example, "Col F Row 9" / "Col F Row 5" = 200% then the cell background will
> > > > > > > > > > > be shaded bright green. Row 5 is static and used as the divisor for all. The
> > > > > > > > > > > conditions are: if the percent is 100.1% or greater then background of cell
> > > > > > > > > > > is bright green, if percent is 90-100% then light green, if percent is
> > > > > > > > > > > 80-89.9% than light yellow, if percent is 70-79.9% than pink, if 1-69.9% then
> > > > > > > > > > > purple, if percent is 0 or blank then red. The columns that contain the data
> > > > > > > > > > > are columns F-P. Is there any way to automate this process. Thanks in
> > > > > > > > > > > advance. I am very new to VBA.
> > > > > > > > > > >
> > > > > > > > > > > Col F Col G Col H Col I Col J Col k Col L
> > > > > > > > > > > Row 3 Jan Feb Mar Apr May Jun Jly
> > > > > > > > > > >
> > > > > > > > > > > Row 5 $500 $500 $500 $750 $750 $750 $1,000
> > > > > > > > > > >
> > > > > > > > > > > Row 9 $1000 $750 $500 $0 $1500 $900 $500
> > > > > > > > > > > Many Rows just like Row 9
> > > > > > > > > > >

If the other colours you refer to are in the range of data we are analysing,
i.e. F9 (F20) to Qnnn, then it will require testing EVERY cell to see if it
is a colour we used and then change it to blank.


"Leslie" wrote:

> Thanks again. That did fix it even though row 102 was not the last non-blank
> cell in the current month. Regarding the clearing color with the code we are
> using it clears out other coloring I have in certain rows. Is there a way to
> limit the clearing of colors to just what we coded?
>
> "Toppers" wrote:
>
> > Becaues (I am "guessing") row 102 is the last non-blank cell in the current
> > month; this what lrowl calculates. If you want to use column A as the
> > delimeteri .e column A is always the longest, then change ncol in the lrow
> > statement to 1 i.e.
> >
> > lrow = Cells(Rows.Count, 1).End(xlUp).Row.
> >
> > You could then modify the code to clear the colours as below and move AFTER
> > the lrow line.
> >
> > Range("F9:Q" & lrow).Select
> > Selection.Interior.ColorIndex = xlNone
> >
> >
> > "Leslie" wrote:
> >
> > > Yeah! Its working BUT, another But, Very oddly it colored all the blank cells
> > > up to row 102 and then just stopped. I can't imagine why that is.
> > >
> > > "Toppers" wrote:
> > >
> > > > Add these lines after End Select
> > > >
> > > > Else
> > > > cell.Select
> > > > Selection.Interior.ColorIndex = 3
> > > >
> > > >
> > > > i.e.
> > > >
> > > > End Select
> > > > * Else
> > > > * cell.Select
> > > > * Selection.Interior.ColorIndex = 3
> > > > End If
> > > >
> > > > "Leslie" wrote:
> > > >
> > > > > Thanks its working great. The only thing that is not working is if the cell
> > > > > is empty or blank it should also have a background color of red. I've been
> > > > > trying to figure this out but no luck so far. Thanks again for all your
> > > > > help.
> > > > >
> > > > > "Toppers" wrote:
> > > > >
> > > > > > Hi,
> > > > > > The simplest way would be to clear ALL months at the beginning of
> > > > > > the macro and just re-populate the current month.
> > > > > >
> > > > > > Range("F9:Q100").Select <=== change to 100 to whatever you think the max
> > > > > > roes are going to be.
> > > > > > Selection.Interior.ColorIndex = xlNone ' Clears the colours
> > > > > >
> > > > > > Place this code at the start of the macro - after DIM statements.
> > > > > >
> > > > > > HTH
> > > > > >
> > > > > >
> > > > > > "Leslie" wrote:
> > > > > >
> > > > > > > Thanks so much for all your help that was so tricky for me. May I ask one
> > > > > > > more question. I tried it out and everything works great but I think it is
> > > > > > > missing something. Once July comes or any month thereafter I want it to
> > > > > > > clear out the background colors from the previous month and only have the
> > > > > > > current month highlighted. Is that possible to do?
> > > > > > >
> > > > > > > "Toppers" wrote:
> > > > > > >
> > > > > > > > Check the END IF are all there! It works OK on my machine!
> > > > > > > >
> > > > > > > > "Leslie" wrote:
> > > > > > > >
> > > > > > > > > It doesn't like the "Next Cell" at the bottom of the code. Thanks.
> > > > > > > > >
> > > > > > > > > "Toppers" wrote:
> > > > > > > > >
> > > > > > > > > > Hi again,
> > > > > > > > > > You will get your error message if the cell is blank so I
> > > > > > > > > > have added a test for cell value being a number.
> > > > > > > > > >
> > > > > > > > > > HTH
> > > > > > > > > >
> > > > > > > > > > Sub CFormat()
> > > > > > > > > > '
> > > > > > > > > > Dim rng As Range, cell As Range
> > > > > > > > > > Dim ncol As Integer, lrow As Long
> > > > > > > > > > Dim pcnt As Double, divisor As Double
> > > > > > > > > >
> > > > > > > > > > ThisWorkbook.Worksheets("Sheet1").Activate ' <=== Change to your w/sheet
> > > > > > > > > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > > > > > > > > ncol = Application.Match(Range("A2"), Range("F3:q3"), 0) + 5
> > > > > > > > > >
> > > > > > > > > > ' Find last row of data in current month column
> > > > > > > > > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > > > > > > > > >
> > > > > > > > > > ' Set range to cells for current month starting row 9
> > > > > > > > > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > > > > > > > > >
> > > > > > > > > > ' Set Divisor for current month
> > > > > > > > > > divisor = Cells(5, ncol)
> > > > > > > > > >
> > > > > > > > > > ' Loop through all cells in range
> > > > > > > > > > For Each cell In rng
> > > > > > > > > > ' Check length of cell in column A
> > > > > > > > > > If Len(cell.Offset(0, -(ncol - 1))) = 4 Then
> > > > > > > > > > ' Calculate perecentage
> > > > > > > > > > If Application.IsNumber(cell) Then ' Is this cell a number ?
> > > > > > > > > > pcnt = (cell / divisor) * 100
> > > > > > > > > > cell.Select
> > > > > > > > > > ' Set colorindex based on percentage
> > > > > > > > > > Select Case pcnt
> > > > > > > > > > Case Is > 100
> > > > > > > > > > Selection.Interior.ColorIndex = 4
> > > > > > > > > > Case Is >= 90
> > > > > > > > > > Selection.Interior.ColorIndex = 35
> > > > > > > > > > Case Is >= 80
> > > > > > > > > > Selection.Interior.ColorIndex = 36
> > > > > > > > > > Case Is >= 70
> > > > > > > > > > Selection.Interior.ColorIndex = 7
> > > > > > > > > > Case Is >= 1
> > > > > > > > > > Selection.Interior.ColorIndex = 54
> > > > > > > > > > Case Else
> > > > > > > > > > Selection.Interior.ColorIndex = 3
> > > > > > > > > > End Select
> > > > > > > > > > End If
> > > > > > > > > > End If
> > > > > > > > > > Next cell
> > > > > > > > > >
> > > > > > > > > > End Sub
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > > "Leslie" wrote:
> > > > > > > > > >
> > > > > > > > > > > Okay, I've confused myself. The run-time error is 1004 after all. I think
> > > > > > > > > > > it is stuck at:
> > > > > > > > > > >
> > > > > > > > > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > > > > > > > > >
> > > > > > > > > > > but since I am new to this VBA stuff I most likely am wrong. Does it have
> > > > > > > > > > > something to do with "ncol". Thanks.
> > > > > > > > > > >
> > > > > > > > > > > "Toppers" wrote:
> > > > > > > > > > >
> > > > > > > > > > > > Hi,
> > > > > > > > > > > > Try this: insert this code into a general module.
> > > > > > > > > > > >
> > > > > > > > > > > > I have defined the cell A2 as a named range called "CurMonth" and the month
> > > > > > > > > > > > headers as a range "HdrMonths".
> > > > > > > > > > > >
> > > > > > > > > > > > Sub CFormat()
> > > > > > > > > > > > '
> > > > > > > > > > > > Dim rng As Range, cell As Range
> > > > > > > > > > > > Dim ncol As Integer, lrow As Long
> > > > > > > > > > > > Dim pcnt As Double, divisor As Double
> > > > > > > > > > > >
> > > > > > > > > > > > Thisworkbook.Worksheets("Sheet1").activate <=== Change to your w/sheet
> > > > > > > > > > > >
> > > > > > > > > > > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > > > > > > > > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > > > > > > > > > >
> > > > > > > > > > > > ' Find last row of data in current month column
> > > > > > > > > > > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > > > > > > > > > > >
> > > > > > > > > > > > ' Set range to cells for current month starting row 9
> > > > > > > > > > > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > > > > > > > > > > >
> > > > > > > > > > > > ' Set Divisor for current month
> > > > > > > > > > > > divisor = Cells(5, ncol)
> > > > > > > > > > > >
> > > > > > > > > > > > ' Loop through all cells in range
> > > > > > > > > > > > For Each cell In rng
> > > > > > > > > > > > ' Calculate perecentage
> > > > > > > > > > > > pcnt = (cell / divisor) * 100
> > > > > > > > > > > > cell.Select
> > > > > > > > > > > > ' Set colorindex based on percentage
> > > > > > > > > > > > Select Case pcnt
> > > > > > > > > > > > Case Is > 100
> > > > > > > > > > > > Selection.Interior.ColorIndex = 4
> > > > > > > > > > > > Case Is >= 90
> > > > > > > > > > > > Selection.Interior.ColorIndex = 35
> > > > > > > > > > > > Case Is >= 80
> > > > > > > > > > > > Selection.Interior.ColorIndex = 36
> > > > > > > > > > > > Case Is >= 70
> > > > > > > > > > > > Selection.Interior.ColorIndex = 7
> > > > > > > > > > > > Case Is >= 1
> > > > > > > > > > > > Selection.Interior.ColorIndex = 54
> > > > > > > > > > > > Case Else
> > > > > > > > > > > > Selection.Interior.ColorIndex = 3
> > > > > > > > > > > > End Select
> > > > > > > > > > > > Next cell
> > > > > > > > > > > >
> > > > > > > > > > > > End Sub
> > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > > > HTH
> > > > > > > > > > > >
> > > > > > > > > > > > "Leslie" wrote:
> > > > > > > > > > > >
> > > > > > > > > > > > > First, the formatting will only highlight certain cells in a column when the
> > > > > > > > > > > > > column header in Row 3 is the current month which is tied to A2 which is
> > > > > > > > > > > > > =Text(now(),"mmm"). Second, only those cells in rows in which Column A has
> > > > > > > > > > > > > four digit letters or numbers will be highlighted. What I need is if, for
> > > > > > > > > > > > > example, "Col F Row 9" / "Col F Row 5" = 200% then the cell background will
> > > > > > > > > > > > > be shaded bright green. Row 5 is static and used as the divisor for all. The
> > > > > > > > > > > > > conditions are: if the percent is 100.1% or greater then background of cell
> > > > > > > > > > > > > is bright green, if percent is 90-100% then light green, if percent is
> > > > > > > > > > > > > 80-89.9% than light yellow, if percent is 70-79.9% than pink, if 1-69.9% then
> > > > > > > > > > > > > purple, if percent is 0 or blank then red. The columns that contain the data
> > > > > > > > > > > > > are columns F-P. Is there any way to automate this process. Thanks in
> > > > > > > > > > > > > advance. I am very new to VBA.
> > > > > > > > > > > > >
> > > > > > > > > > > > > Col F Col G Col H Col I Col J Col k Col L
> > > > > > > > > > > > > Row 3 Jan Feb Mar Apr May Jun Jly
> > > > > > > > > > > > >
> > > > > > > > > > > > > Row 5 $500 $500 $500 $750 $750 $750 $1,000
> > > > > > > > > > > > >
> > > > > > > > > > > > > Row 9 $1000 $750 $500 $0 $1500 $900 $500
> > > > > > > > > > > > > Many Rows just like Row 9
> > > > > > > > > > > > >

You can run it automatically each time the workbook is open. Put this code
in the ThisWorkbook sheet (module):

Sub Workbook_Open
Call cFormat
end sub

OR (and I'll leave to work it out!) you could add a button to the form which
calls the macro. HINT_ click the Forms toolbar and select button control. In
the Assign Macro form select CFormat then OK.

FYI

Attached code clears ONLY colurs used in this worksheet:

Sub CFormat()
'
Dim rng As Range, cell As Range
Dim ncol As Integer, lrow As Long
Dim pcnt As Double, divisor As Double

ThisWorkbook.Worksheets("Sheet1").Activate ' <=== Change to your w/sheet

' Find column for current Month (add 5 to start in colum F onwards)
ncol = Application.Match(Range("A2"), Range("F3:q3"), 0) + 5

' Find last row of data in column A
lrow = Cells(Rows.Count, 1).End(xlUp).Row
' Clear colours used in this macro
Set rng = Range("F9:Q1" & lrow)
For Each cell In rng
Select Case cell.Interior.ColorIndex
Case Is = 4, 35, 36, 7, 54, 3
cell.Interior.ColorIndex = xlNone
End Select
Next cell

' Set range to cells for current month starting row 9
Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))

' Set Divisor for current month
divisor = Cells(5, ncol)

' Loop through all cells in range
For Each cell In rng
' Check length of cell in column A
If Len(cell.Offset(0, -(ncol - 1))) = 4 Then
' Calculate perecentage
If Application.IsNumber(cell) Then ' Is this cell a number ?
pcnt = (cell / divisor) * 100
cell.Select
' Set colorindex based on percentage
Select Case pcnt
Case Is > 100
Selection.Interior.ColorIndex = 4
Case Is >= 90
Selection.Interior.ColorIndex = 35
Case Is >= 80
Selection.Interior.ColorIndex = 36
Case Is >= 70
Selection.Interior.ColorIndex = 7
Case Is >= 1
Selection.Interior.ColorIndex = 54
Case Else
Selection.Interior.ColorIndex = 3
End Select
Else
cell.Select
Selection.Interior.ColorIndex = 3
End If
End If
Next cell

End Sub

"Leslie" wrote:

> One more thing. How is the coding executed? Is it when I open the
> worksheet or workbook or do I have to press F5 all the time? Thanks.
>
> "Leslie" wrote:
>
> > Yeah! Its working BUT, another But, Very oddly it colored all the blank cells
> > up to row 102 and then just stopped. I can't imagine why that is.
> >
> > "Toppers" wrote:
> >
> > > Add these lines after End Select
> > >
> > > Else
> > > cell.Select
> > > Selection.Interior.ColorIndex = 3
> > >
> > >
> > > i.e.
> > >
> > > End Select
> > > * Else
> > > * cell.Select
> > > * Selection.Interior.ColorIndex = 3
> > > End If
> > >
> > > "Leslie" wrote:
> > >
> > > > Thanks its working great. The only thing that is not working is if the cell
> > > > is empty or blank it should also have a background color of red. I've been
> > > > trying to figure this out but no luck so far. Thanks again for all your
> > > > help.
> > > >
> > > > "Toppers" wrote:
> > > >
> > > > > Hi,
> > > > > The simplest way would be to clear ALL months at the beginning of
> > > > > the macro and just re-populate the current month.
> > > > >
> > > > > Range("F9:Q100").Select <=== change to 100 to whatever you think the max
> > > > > roes are going to be.
> > > > > Selection.Interior.ColorIndex = xlNone ' Clears the colours
> > > > >
> > > > > Place this code at the start of the macro - after DIM statements.
> > > > >
> > > > > HTH
> > > > >
> > > > >
> > > > > "Leslie" wrote:
> > > > >
> > > > > > Thanks so much for all your help that was so tricky for me. May I ask one
> > > > > > more question. I tried it out and everything works great but I think it is
> > > > > > missing something. Once July comes or any month thereafter I want it to
> > > > > > clear out the background colors from the previous month and only have the
> > > > > > current month highlighted. Is that possible to do?
> > > > > >
> > > > > > "Toppers" wrote:
> > > > > >
> > > > > > > Check the END IF are all there! It works OK on my machine!
> > > > > > >
> > > > > > > "Leslie" wrote:
> > > > > > >
> > > > > > > > It doesn't like the "Next Cell" at the bottom of the code. Thanks.
> > > > > > > >
> > > > > > > > "Toppers" wrote:
> > > > > > > >
> > > > > > > > > Hi again,
> > > > > > > > > You will get your error message if the cell is blank so I
> > > > > > > > > have added a test for cell value being a number.
> > > > > > > > >
> > > > > > > > > HTH
> > > > > > > > >
> > > > > > > > > Sub CFormat()
> > > > > > > > > '
> > > > > > > > > Dim rng As Range, cell As Range
> > > > > > > > > Dim ncol As Integer, lrow As Long
> > > > > > > > > Dim pcnt As Double, divisor As Double
> > > > > > > > >
> > > > > > > > > ThisWorkbook.Worksheets("Sheet1").Activate ' <=== Change to your w/sheet
> > > > > > > > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > > > > > > > ncol = Application.Match(Range("A2"), Range("F3:q3"), 0) + 5
> > > > > > > > >
> > > > > > > > > ' Find last row of data in current month column
> > > > > > > > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > > > > > > > >
> > > > > > > > > ' Set range to cells for current month starting row 9
> > > > > > > > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > > > > > > > >
> > > > > > > > > ' Set Divisor for current month
> > > > > > > > > divisor = Cells(5, ncol)
> > > > > > > > >
> > > > > > > > > ' Loop through all cells in range
> > > > > > > > > For Each cell In rng
> > > > > > > > > ' Check length of cell in column A
> > > > > > > > > If Len(cell.Offset(0, -(ncol - 1))) = 4 Then
> > > > > > > > > ' Calculate perecentage
> > > > > > > > > If Application.IsNumber(cell) Then ' Is this cell a number ?
> > > > > > > > > pcnt = (cell / divisor) * 100
> > > > > > > > > cell.Select
> > > > > > > > > ' Set colorindex based on percentage
> > > > > > > > > Select Case pcnt
> > > > > > > > > Case Is > 100
> > > > > > > > > Selection.Interior.ColorIndex = 4
> > > > > > > > > Case Is >= 90
> > > > > > > > > Selection.Interior.ColorIndex = 35
> > > > > > > > > Case Is >= 80
> > > > > > > > > Selection.Interior.ColorIndex = 36
> > > > > > > > > Case Is >= 70
> > > > > > > > > Selection.Interior.ColorIndex = 7
> > > > > > > > > Case Is >= 1
> > > > > > > > > Selection.Interior.ColorIndex = 54
> > > > > > > > > Case Else
> > > > > > > > > Selection.Interior.ColorIndex = 3
> > > > > > > > > End Select
> > > > > > > > > End If
> > > > > > > > > End If
> > > > > > > > > Next cell
> > > > > > > > >
> > > > > > > > > End Sub
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > "Leslie" wrote:
> > > > > > > > >
> > > > > > > > > > Okay, I've confused myself. The run-time error is 1004 after all. I think
> > > > > > > > > > it is stuck at:
> > > > > > > > > >
> > > > > > > > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > > > > > > > >
> > > > > > > > > > but since I am new to this VBA stuff I most likely am wrong. Does it have
> > > > > > > > > > something to do with "ncol". Thanks.
> > > > > > > > > >
> > > > > > > > > > "Toppers" wrote:
> > > > > > > > > >
> > > > > > > > > > > Hi,
> > > > > > > > > > > Try this: insert this code into a general module.
> > > > > > > > > > >
> > > > > > > > > > > I have defined the cell A2 as a named range called "CurMonth" and the month
> > > > > > > > > > > headers as a range "HdrMonths".
> > > > > > > > > > >
> > > > > > > > > > > Sub CFormat()
> > > > > > > > > > > '
> > > > > > > > > > > Dim rng As Range, cell As Range
> > > > > > > > > > > Dim ncol As Integer, lrow As Long
> > > > > > > > > > > Dim pcnt As Double, divisor As Double
> > > > > > > > > > >
> > > > > > > > > > > Thisworkbook.Worksheets("Sheet1").activate <=== Change to your w/sheet
> > > > > > > > > > >
> > > > > > > > > > > ' Find column for current Month (add 5 to start in colum F onwards)
> > > > > > > > > > > ncol = Application.Match(Range("CurMonth"), Range("HdrMonths"), 0) + 5
> > > > > > > > > > >
> > > > > > > > > > > ' Find last row of data in current month column
> > > > > > > > > > > lrow = Cells(Rows.Count, ncol).End(xlUp).Row
> > > > > > > > > > >
> > > > > > > > > > > ' Set range to cells for current month starting row 9
> > > > > > > > > > > Set rng = Range(Cells(9, ncol), Cells(lrow, ncol))
> > > > > > > > > > >
> > > > > > > > > > > ' Set Divisor for current month
> > > > > > > > > > > divisor = Cells(5, ncol)
> > > > > > > > > > >
> > > > > > > > > > > ' Loop through all cells in range
> > > > > > > > > > > For Each cell In rng
> > > > > > > > > > > ' Calculate perecentage
> > > > > > > > > > > pcnt = (cell / divisor) * 100
> > > > > > > > > > > cell.Select
> > > > > > > > > > > ' Set colorindex based on percentage
> > > > > > > > > > > Select Case pcnt
> > > > > > > > > > > Case Is > 100
> > > > > > > > > > > Selection.Interior.ColorIndex = 4
> > > > > > > > > > > Case Is >= 90
> > > > > > > > > > > Selection.Interior.ColorIndex = 35
> > > > > > > > > > > Case Is >= 80
> > > > > > > > > > > Selection.Interior.ColorIndex = 36
> > > > > > > > > > > Case Is >= 70
> > > > > > > > > > > Selection.Interior.ColorIndex = 7
> > > > > > > > > > > Case Is >= 1
> > > > > > > > > > > Selection.Interior.ColorIndex = 54
> > > > > > > > > > > Case Else
> > > > > > > > > > > Selection.Interior.ColorIndex = 3
> > > > > > > > > > > End Select
> > > > > > > > > > > Next cell
> > > > > > > > > > >
> > > > > > > > > > > End Sub
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > HTH
> > > > > > > > > > >
> > > > > > > > > > > "Leslie" wrote:
> > > > > > > > > > >
> > > > > > > > > > > > First, the formatting will only highlight certain cells in a column when the
> > > > > > > > > > > > column header in Row 3 is the current month which is tied to A2 which is
> > > > > > > > > > > > =Text(now(),"mmm"). Second, only those cells in rows in which Column A has
> > > > > > > > > > > > four digit letters or numbers will be highlighted. What I need is if, for
> > > > > > > > > > > > example, "Col F Row 9" / "Col F Row 5" = 200% then the cell background will
> > > > > > > > > > > > be shaded bright green. Row 5 is static and used as the divisor for all. The
> > > > > > > > > > > > conditions are: if the percent is 100.1% or greater then background of cell
> > > > > > > > > > > > is bright green, if percent is 90-100% then light green, if percent is
> > > > > > > > > > > > 80-89.9% than light yellow, if percent is 70-79.9% than pink, if 1-69.9% then
> > > > > > > > > > > > purple, if percent is 0 or blank then red. The columns that contain the data
> > > > > > > > > > > > are columns F-P. Is there any way to automate this process. Thanks in
> > > > > > > > > > > > advance. I am very new to VBA.
> > > > > > > > > > > >
> > > > > > > > > > > > Col F Col G Col H Col I Col J Col k Col L
> > > > > > > > > > > > Row 3 Jan Feb Mar Apr May Jun Jly
> > > > > > > > > > > >
> > > > > > > > > > > > Row 5 $500 $500 $500 $750 $750 $750 $1,000
> > > > > > > > > > > >
> > > > > > > > > > > > Row 9 $1000 $750 $500 $0 $1500 $900 $500
> > > > > > > > > > > > Many Rows just like Row 9
> > > > > > > > > > > >