nbozzy
asked on
Worksheet_Change event not running when DELETE key is used
Hi, Experts:
I've seen a couple of questions like this but none were answered in a way that helped me, so sorry for any redundancy. I need the Worksheet_Change event to delete contents in some cells if a specific cell's contents are deleted. But the code won't run if the cell is changed with the Delete key....this is driving me nuts. Can someone please help? Partial code is below. Thanks!
I've seen a couple of questions like this but none were answered in a way that helped me, so sorry for any redundancy. I need the Worksheet_Change event to delete contents in some cells if a specific cell's contents are deleted. But the code won't run if the cell is changed with the Delete key....this is driving me nuts. Can someone please help? Partial code is below. Thanks!
If Not Intersect(Target, Range("A9:A21")) Is Nothing Then
'first employee
If Target.Address = "A9" Then
If Target = "" Then
Application.EnableEvents = False
Cells(Target.Row, Target.Column + 1).Value = ""
Cells(Target.Row, Target.Column + 2).Value = ""
Cells(Target.Row, Target.Column + 3).Value = ""
Cells(Target.Row, Target.Column + 4).Value = ""
Cells(Target.Row, Target.Column + 5).Value = ""
Cells(Target.Row, Target.Column + 6).Value = ""
Cells(Target.Row, Target.Column + 7).Value = ""
End If
'more code will follow
Application.EnableEvents = True
End If
End If
The following code (slightly mofifies your code) worked when I tried ( for change and delete)
Private Sub Worksheet_Change(ByVal Target As Range)
If Not Intersect(Target, Range("A9:A21")) Is Nothing Then
'first employee
If Target.Address = "$A$9" Then
If Target = "" Then
Application.EnableEvents = False
Cells(Target.Row, Target.Column + 1).Value = ""
Cells(Target.Row, Target.Column + 2).Value = ""
Cells(Target.Row, Target.Column + 3).Value = ""
Cells(Target.Row, Target.Column + 4).Value = ""
Cells(Target.Row, Target.Column + 5).Value = ""
Cells(Target.Row, Target.Column + 6).Value = ""
Cells(Target.Row, Target.Column + 7).Value = ""
End If
'more code will follow
Application.EnableEvents = True
End If
End If
End Sub
ASKER
Neither approach worked for me, so I am attaching my file (see PerDiem sheet). Two things that might have an affect: Cell A9 is merged with B9:D9, and A9 is populated with a drop-down list.
And StephenJR - your code modification was very interesting to me! Thanks for the new trick!
EXPENSE-REPORT-MASTER---022510.xls
And StephenJR - your code modification was very interesting to me! Thanks for the new trick!
EXPENSE-REPORT-MASTER---022510.xls
SOLUTION
membership
This solution is only available to members.
To access this solution, you must be a member of Experts Exchange.
ASKER
But then Center Across Selection allows the user to click in any of the 4 cells, which will easily confuse them. Suggestions, please?
No, the change event will only be triggered by A9 changing. My only suggestion is not to merge cells! You may have to rethink your layout but that's a different matter.
ASKER
The layout has been around for years and I inherited it. Is there no way to get the code to work with the cells merged?
ASKER CERTIFIED SOLUTION
membership
This solution is only available to members.
To access this solution, you must be a member of Experts Exchange.
Perhaps this works? Untested.
Private Sub Worksheet_Change(ByVal Target As Range)
'PerDiem worksheet, Expense Report Master file
'Populate dollar amounts if Job# and Employee name
'Pays $30
Application.ScreenUpdating = False
Application.DisplayAlerts = False
If Not Intersect(Target, Range("E8:K20")) Is Nothing And Target.Count = 1 Then
'first employee
If Not Intersect(Target, Range("E8:K8")) Is Nothing Then
If Cells(Target.Row + 1, "A") <> "" Then
Cells(Target.Row + 1, Target.Column) = "30"
Else
Application.EnableEvents = False
MsgBox "Please FIRST choose the Employee Name."
Cells(Target.Row, Target.Column) = ""
End If
Application.EnableEvents = True
End If
'second employee
If Not Intersect(Target, Range("E12:K12, E16:K16, E20:K20")) Is Nothing And Target.Count = 1 Then
If Cells(Target.Row + 1, "A") <> "" Then
Cells(Target.Row + 1, Target.Column) = 30
Else
Application.EnableEvents = False
MsgBox "Please FIRST choose the Employee Name."
Cells(Target.Row, Target.Column) = ""
End If
Application.EnableEvents = True
End If
End If
'Delete dollar amounts if employee name is deleted
If Not Intersect(Target, Range("A9:A21")) Is Nothing Then
'first employee
If Not Intersect(Target, Range("A9:D9")) Is Nothing Then
If Target(1, 1) = "" Then
Application.EnableEvents = False
Target.Offset(, 1).Resize(, 7).ClearContents
End If
'more code will follow
Application.EnableEvents = True
End If
End If
Application.ScreenUpdating = True
Application.DisplayAlerts = True
End Sub
ASKER
StephenJR, that worked! I'm very grateful because I learned TWO things: how to slay the savage merged cells, and why to not use them myself! Thanks a million for your persistence.
Good news. That is an important lesson in life!
ASKER
StephenJR: regarding your code (which you snuck in as I was fashioning my reply), this line didn't work: Target.Offset(, 1).Resize(, 7).ClearContents. Instead I commented it out and used my plodding first attempt, which did work. If you have an idea for this cleaner code, I'd be interested. Thanks!
Funny, it worked for me. See if this variation works.
Target(1,1).Offset(, 1).Resize(, 7).ClearContents
Target(1,1).Offset(, 1).Resize(, 7).ClearContents
ASKER
Oops -- silly me...mine didn't work because I tried to "fix" it for the merged cell logic, and I changed the offset column to 5. Putting it back to 1 worked beautifully. Thanks so much for all the new knowledge!
Open in new window