That code is verbatim from my web site
(
www.cpearson.com/Excel/BrowseFolder.htm), and it works for me. I just tried
it in both Excel 2003 and Excel 2007 on both Windows XP Pro SP2 and Windows
Vista Ultimate. It works fine. Put your cursor inside the OpenMyBrowser sub
and then press the F8 key to step through the code line by line. Does the
browse folder dialog display? Do you get an error?
Just for diagnostics, change
ID = SHBrowseForFolderA(BrowseInfo)
to
ID = SHBrowseForFolderA(BrowseInfo)
MsgBox "Last DLL Error: " & CStr(Err.LastDLLError)
If the MsgBox displays anything but 0, then the call to SHBrowseForFolderA
failed and you can decode the Err.LastDLLError using other code on my web
site (
http://www.cpearson.com/Excel/FormatMessage.aspx).
By the way, the line of code
Dim FName, SUBDIR, ROOTDIR As String
probably isn't doing what you think it is. It does NOT declare all these
variables as Strings. It declares FName and SUBDIR as Variants and only
ROOTDIR as a String. It is the same as
Dim FName As Variant, SUBDIR As Variant, ROOTDIR As String
However, this should not cause any problems.
Also, you should almost NEVER use the "End" statement. It is a poor
programming practice that can cause unexpected problems.
And on a final note, your computer's clock is wrong. Please fix it.
--
Cordially,
Chip Pearson
Microsoft MVP - Excel
Pearson Software Consulting
www.cpearson.com
(email on the web site)