Nothing much wrong with the code, other than it not being well indented. Indentation makes code much easier to read.
Code slightly rewritten below. Only real thing I changed is rather than having two loops I've just condensed them into one So each control is stamped to the worksheet then cleared before it moves to the next control.
Also, you don't really need the clear button. I've added a couple of lines that will quickly close and reopen the form which will set it back to it's original state, i.e. cleared and with the current month in the first control.
This means you don't actually need to clear the control in the loop, so if you scrap the clear button you can remove the second command in the loop.
Or if you prefer to keep the clear button the Unload Me and the UserForm3.Show lines.
BSB
Bookmarks