AutoFilter Date Criteria

Hi, I do not know what your criteria are but you have forgotten to define the worksheet you're clicking

Please Login or Register  to view this content.

The bold text is what you were missing

The bold red text is incorrect, you are addressing a column and this should be a row, but the criteria, well you can continue with it, the macro error is not there anymore

The two Autofilter lines are to clear the previous autofilter, You have AutoFilter.False and that's wrong

I'm having a hard time trying to figure out what it is you want to do.

First of all, it's a good idea to start every module off with Option Explicit. This will force you to define all your variables. So when you misspell one, the editor will let you know.

For example Column is not defined so it is nothing and the code will fail there even if everything else was correct.

Next, this macro will fire if you double click anywhere on the sheet. You need to limit it to just the range of interest. I am going to assume that these are cells B4:D8. So add this code in right after the declaration:

If Intersect (Target, Range("$B$4:$D$8") ) is Nothing then Exit Sub

This line of code checks to see if the double-clicked cell is in the range of interest and if it isn't (is nothing) then exit the subroutine.

I am going to disagree with Hans about having to explicitly declare the sheet in this instance. It is almost always a good idea to explicitly state where you think you are in the workbook and for that matter which workbook you are in.

However, there are times when this is not necessary. If you only have one workbook open, then obviously it is the active workbook.

If you launch a macro from a button on a sheet, then you can safely assume that the sheet is the active sheet.

Likewise any code that gets launched from a sheet rather than a module such as a double-click event, then it is also safe to assume that the sheet is the active sheet.

However, Hans is on the right track. You do make reference to another sheet and his advice is valid except that he was pointing at the wrong sheet. I would modify his and your code thusly:

Please Login or Register  to view this content.

There is no need to select something to work on it. Generally speaking, in code, when you see:

Please Login or Register  to view this content.

You can replace it with:

Please Login or Register  to view this content.

As for the rest of the code, I can’t figure out what you are doing. How do the 4 Weeks, 3 Weeks and 2 Weeks relate to the dates? What is it that you want to filter? Do you want to filter on the Area name on the row with the double-clicked cell?

I imagined that much.
And is it working now?

True, you're not using the row variable, and column is unknown or incorrect

Please Login or Register  to view this content.

I get an empty filter but maybe it works on your file with your date settings

Could you explain a little more of what your really trying to achieve?
You criteria are all wrong.
Add a Debug.Print in you macro so you can see what you are filtering.

Nice. Looks good and you restricted the double-click area. I was going to suggest that.

You're welcome