Make the code run only when shape is selected not after every 1 second.


New Member
Jun 13, 2016
this code is written to update the shape of the color whenever i drag n drop it according to its position. it is called once in workbook_open() event
But it is running all the time from opening the workbook till closing it.
Is there any method to make it run only when shape is selected.

Sub UpdateColor()
Dim ShapeXValue As Double
Dim datetRunTme As Date
Const TimeInterval As String = "00:00:01"

 'create error if selection does not involve a shape
On Error GoTo NoShapeSelected
 'amend colour based on selection.left
ShapeXValue = Selection.Left
Dim colour As Long
Select Case ShapeXValue
    Case Is < Range("G4", "G4").Left: colour = RGB(255, 128, 0)
    Case Is < Range("N4", "N4").Left: colour = RGB(0, 153, 0)
    Case Else: colour = RGB(219, 38, 10)
End Select

Selection.ShapeRange.Fill.ForeColor.RGB = colour

 'configure routine to run again
datetRunTme = Now + TimeValue(TimeInterval)
Application.OnTime earliesttime:=datetRunTme, procedure:="UpdateColor", schedule:=True

End Sub

Forum statistics

Latest member

Some videos you may like

This Week's Hot Topics

  • populate from drop list with multiple tables
    Hi All, i have a drop list that displays data, what i want is when i select one of those from the list to populate text from different tables on...
  • Find list of words from sheet2 in sheet1 before a comma and extract text vba
    Hi Friends, Trying to find the solution on my task. But did not find suitable one to the need. Here is my query and sample file with details...
  • Dynamic Formula entry - VBA code sought
    Hello, really hope one of you experts can help with this - i've spent hours on this and getting no-where. .I have a set of data (more rows than...
  • Listbox Header
    Have a named range called "AccidentsHeader" Within my code I have: [CODE]Private Sub CommandButton1_Click() ListBox1.RowSource =...
  • Complex Heat Map using conditional formatting
    Good day excel world. I have a concern. Below link have a list of countries that carries each country unique data. [URL...
  • Conditional formatting
    Hi good morning, hope you can help me please, I have cells P4:P54 and if this cell is equal to 1 then i want row O to say "Fully Utilised" and to...