Make the current worksheet the active sheet.

JohnZ1156

Board Regular
Joined
Apr 10, 2021
Messages
160
Office Version
  1. 2021
Platform
  1. Windows
I have a macro that I would like to be able to run on different worksheets. I placed a "button" on Sheet1 and assigned this macro:
Problem is, I'd like to place the same button on Sheet2, Sheet3, etc. and run the same macro, except on the current sheet, not always on Sheet1.
I know the "fix" is somewhere in the line Set ws = Worksheets("Sheet1") but I don't know how to "fix" it.

VBA Code:
Option Explicit
Sub L_to_R()
    Application.ScreenUpdating = False
    Dim ws As Worksheet, c As Range
    Set ws = Worksheets("Sheet1")
    For Each c In ws.Range("A2", Cells(Rows.Count, "A").End(xlUp))
        If IsDate(c) Then c.Offset(, 1).Resize(, 5).Sort _
        Key1:=c, Order1:=xlAscending, Orientation:=xlLeftToRight
        ws.Sort.SortFields.Clear
    Next c
    Application.ScreenUpdating = True
End Sub
 

Excel Facts

Move date out one month or year
Use =EDATE(A2,1) for one month later. Use EDATE(A2,12) for one year later.
You need:

Code:
Set ws = activesheet
 
Upvote 0
Thanks Rory
After I made that change, now I get this:
1700833572223.png
 
Upvote 0
I think row number has to be declared with Offset. i.e. - Also you forgot the End If statement
VBA Code:
c.Offset(0,1).Resize(0,5)
 
Upvote 0
VBA Code:
Sub L_to_R()
    Application.ScreenUpdating = False
    Dim ws As Worksheet, c As Range
    Set ws = ActiveSheet
    For Each c In ws.Range("A2", Cells(Rows.Count, "A").End(xlUp))
        If IsDate(c) Then
            c.Offset(0, 1).Resize(0, 5).Sort _
            Key1:=c, Order1:=xlAscending, Orientation:=xlLeftToRight
            ws.Sort.SortFields.Clear
        End If
    Next c
    Application.ScreenUpdating = True
End Sub
 
Upvote 0
You are trying to sort a range by a cell that is not part of the range.
 
Upvote 0
I created a "dummy" worksheet to try to figure out why the macro wasn't working.

I originally had my data formatted as Excel Tables. When I tried to run the above macro, I got that VBA error message. Once I converted the tables back to normal ranges, the macro worked. Go figure! I don't understand why!

After experimenting around, I finally did get the same macro to work on multiple sheets. I placed a "button" on each sheet and assigned the same macro to each button. When I selected the sheet and pressed the button, it ran the macro on that sheet.

If someone can explain to me why this macro would not work on data formatted as an Excel Table, but did work when the same data was not formatted as an Excel Table, I'd love to understand.

In fairness, I don't believe I ever said that my data was formatted as an Excel Table.
Regardless, Thank you all for your help and suggestions.
 
Upvote 0
I think row number has to be declared with Offset. i.e.
Just to say it doesn't (although some frown upon it being omitted)

VBA Code:
Sub Skybot_test()
Debug.Print "With row omitted the address is " & Range("D4").Offset(, 4).Address
Debug.Print "With row included the address is " & Range("D4").Offset(0, 4).Address
End Sub

c.Offset(0, 1).Resize(0, 5).Sort
Resize on the other hand won't accept a 0
VBA Code:
Sub Skybot_test()
Debug.Print "With row omitted the address is " & Range("D4").Resize(, 5).Address
Debug.Print "With row included the address is " & Range("D4").Resize(0, 5).Address
End Sub
 
Last edited:
Upvote 0

Forum statistics

Threads
1,215,069
Messages
6,122,959
Members
449,096
Latest member
Anshu121

We've detected that you are using an adblocker.

We have a great community of people providing Excel help here, but the hosting costs are enormous. You can help keep this site running by allowing ads on MrExcel.com.
Allow Ads at MrExcel

Which adblocker are you using?

Disable AdBlock

Follow these easy steps to disable AdBlock

1)Click on the icon in the browser’s toolbar.
2)Click on the icon in the browser’s toolbar.
2)Click on the "Pause on this site" option.
Go back

Disable AdBlock Plus

Follow these easy steps to disable AdBlock Plus

1)Click on the icon in the browser’s toolbar.
2)Click on the toggle to disable it for "mrexcel.com".
Go back

Disable uBlock Origin

Follow these easy steps to disable uBlock Origin

1)Click on the icon in the browser’s toolbar.
2)Click on the "Power" button.
3)Click on the "Refresh" button.
Go back

Disable uBlock

Follow these easy steps to disable uBlock

1)Click on the icon in the browser’s toolbar.
2)Click on the "Power" button.
3)Click on the "Refresh" button.
Go back
Back
Top