VBA to delete rows in filtered table

Hi Experts!

Attached is some code I'm using to Filter one column in a Table and then delete the visible rows.  I had difficulty with Selection.EntireRow.Delete but have replaced it with RngToDelete.Delete.

My problem is that the code stops and asks if I want to "Delete Entire Sheet Row?".  I click "Yes" and it's fine.  How can I have the code eliminate that step as well so it runs from start to finish without stopping?

This is very likely a no brainer for all of you. My attempt at code is below.  Thank you!

Sub Filter_Sales1()
'
    Dim RngToDelete As Range
    Range("TblFTE").Select
   
    ActiveSheet.ListObjects("TblFTE").Range.AutoFilter Field:=10, Criteria1:= _
        Array("Business Development", "G&A", "Marketing", "R&D", "Solutions", "Support"), _
        Operator:=xlFilterValues
       
    Set RngToDelete = Selection.SpecialCells(xlCellTypeVisible)
     
    RngToDelete.Delete
       
    Range("TblFTE[[#Headers],[EE ID]]").Select
    ActiveSheet.ShowAllData
End Sub
LVL 1
xllvrAsked:
Who is Participating?

[Product update] Infrastructure Analysis Tool is now available with Business Accounts.Learn More

x
I wear a lot of hats...

"The solutions and answers provided on Experts Exchange have been extremely helpful to me over the last few years. I wear a lot of hats - Developer, Database Administrator, Help Desk, etc., so I know a lot of things but not a lot about one thing. Experts Exchange gives me answers from people who do know a lot about one thing, in a easy to use platform." -Todd S.

dlmilleCommented:
try this (see line 12 modification):

Sub Filter_Sales1()
'
    Dim RngToDelete As Range
    Range("TblFTE").Select
    
    ActiveSheet.ListObjects("TblFTE").Range.AutoFilter Field:=10, Criteria1:= _
        Array("Business Development", "G&A", "Marketing", "R&D", "Solutions", "Support"), _
        Operator:=xlFilterValues
        
    Set RngToDelete = Selection.SpecialCells(xlCellTypeVisible)
     
    RngToDelete.EntireRow.Delete
        
    Range("TblFTE[[#Headers],[EE ID]]").Select
    ActiveSheet.ShowAllData
End Sub

Open in new window


Also, when you want to avoid Excel prompting and you don't know exactly how to do the statement so you don't get the prompt (and some don't have - e.g., for deleting sheets, you would get prompted), then use:

Application.DisplayAlerts = False

before the command that would cause the prompt

Then,

Application.DisplayAlerts = True 'continue as normal

Experts Exchange Solution brought to you by

Your issues matter to us.

Facing a tech roadblock? Get the help and guidance you need from experienced professionals who care. Ask your question anytime, anywhere, with no hassle.

Start your 7-day free trial
xllvrAuthor Commented:
Hey dlmille!  Nice to see you here.

Thanks for posting.  The code hung up on line 11 when I changed line 12.  Thoughts?
xllvrAuthor Commented:
Hang on a second.  Just tried your suggestion with Application.DisplayAlerts=False along with the code I already had and it seems to work.
JavaScript Best Practices

Save hours in development time and avoid common mistakes by learning the best practices to use for JavaScript.

dlmilleCommented:
I'm not sure what you mean by "hung up" and I don't see a line 11.

Yes, displayalerts would also work, but it shouldn't be needed if you delete using the entirerow property.

Dave
xllvrAuthor Commented:
Apologies...Line 10.  The code halted at that point.  I originally used EntireRow and had problems with it.  I have read quite a few posts that showed the very same problem with using EntireRow in the code so I replaced it with what others used successfully.  Using RngToDelete.Delete worked fine except I didn't want to have to stop and click "Yes" to Delete Entire Sheet Row to continue the code.  

Using RngToDelete.Delete with the Display Alert message seems to be working.  Thanks for sending that along as a potential solution!
dlmilleCommented:
Perhaps it was choking on EntireRow because it was a table?  Yes, any time there is an Excel Prompt you want to circumvent (almost anytime, there are a few exceptions), you can override and take the default "YES" with Application.DisplayAlerts = False.

Dave
xllvrAuthor Commented:
That's the only conclusion I could come up with as well.  Grateful for the prompt tip, Dave!

-Nancy
It's more than this solution.Get answers and train to solve all your tech problems - anytime, anywhere.Try it for free Edge Out The Competitionfor your dream job with proven skills and certifications.Get started today Stand Outas the employee with proven skills.Start learning today for free Move Your Career Forwardwith certification training in the latest technologies.Start your trial today
Microsoft Excel

From novice to tech pro — start learning today.