Sub name issue

  • Thread starter Robert Hargreaves
  • Start date
R

Robert Hargreaves

Hi everyone I'm a bit puzzled over where to put some code to make it work.

I have got a public sub in my excel file. I know how you should call this
in code but I do not know if this is the correct place for it.

I only want the contents of the code to work on one sheet and I only would
like the code to apply to columns Y, Z, AA, AB

I dont know what the declaration name should be like onactivate or onchange.

can someone help please. Patrick Malloy, I couldn't find your code you
recommeded before.

Thankyou for your help.
Rob

Here is a copy of the code in the sub

Public Sub ConditionalFill()

Dim val As Long
Dim nr1 As Long
Dim nr2 As Long
Dim nr3 As Long
Dim nr4 As Long
Dim nr5 As Long
Dim nr6 As Long
Dim nr7 As Long
Dim nr8 As Long

val = ActiveCell.Value
nr1 = Range(ActiveWorkbook.Names("NaburnMLSSTrig1a")).Value
nr2 = Range(ActiveWorkbook.Names("NaburnMLSSTrig1b")).Value
nr3 = Range(ActiveWorkbook.Names("NaburnMLSSTrig2a")).Value
nr4 = Range(ActiveWorkbook.Names("NaburnMLSSTrig2b")).Value
nr5 = Range(ActiveWorkbook.Names("NaburnMLSSTrig3a")).Value
nr6 = Range(ActiveWorkbook.Names("NaburnMLSSTrig3b")).Value
nr7 = Range(ActiveWorkbook.Names("NaburnMLSSTrig4a")).Value
nr8 = Range(ActiveWorkbook.Names("NaburnMLSSTrig4b")).Value

Select Case True
Case val > nr1 And val < nr2
ActiveCell.Interior.ColorIndex = 45
Case val > nr3 And val < nr4
ActiveCell.Interior.ColorIndex = 3
Case val > nr5 And val < nr6
ActiveCell.Interior.ColorIndex = 45
Case val > nr7 And val < nr8
ActiveCell.Interior.ColorIndex = 3
Case Else
MsgBox ("Non Apply"), vbInformation
ActiveCell.Interior.ColorIndex = xlColorIndexNone
End Select

End Sub
 
W

William Benson

I am thinking the Workbook_SheetSelectionChange event is the place, have you
tried this? sh is one of the args, you test for the sh.name = "xxx" and you
have code affecting only one sheet. Target is another arg, and it will be
the cell that was GONE TO ... so, if you only want it to act on the cell you
just finished working in, you can hold a static value called
CELL_LAST_VISITED at the end of the sub. Then do your test on either Target
or CELL_LAST_VISITED depending on your needs.
 
W

William Benson

You posted this same request message twice, but with a changed subject, even
though you had answers from other people (which you didn't acknowledge)
under your earlier the thread ... "Declaration Name" ...Why ???
 
R

Rob Hargreaves

Sorry for the repeats I am using outlook express and I havent had the
problems before but couldnt see the old post in the list and now no matter
what I refresh or reset doesnt work!!

Thanks for the other replies though I will get them somehow.

Rob
 
G

Guest

I understand; I find I need to go to page 2 and back to page 1 to get it to
really refresh - probably something to do with the page being cached.
 

Ask a Question

Want to reply to this thread or ask your own question?

You'll need to choose a username for the site, which only take a couple of moments. After that, you can post your question and our members will help you out.

Ask a Question

Top