It might have gotten pasted into a funny spot. I looks like you have an "email sub" right above it that doesn't need to be there. Just copy and paste the code above into its own module. When that is complete assign the button to this macro: "Workbook_To_PDF" not to the function. Do you want the user to pick the email address from outlook? Or will it come from a cell etc on the sheet?