Quote Originally Posted by Phantom1 View Post
However, what is the point of ButtonClicked? It does not seem to do anything to unload the form. One's own code must be added.
From the code of that form, it appears that inputbox form is hidden during cmdButton_Click. The routine that calls the frmInputBox.Display should also unload frmInputBox in my opinion.
Quote Originally Posted by Phantom1
The Index of the CommandButtons could be used directly instead of having the variables btnOK and btnCancel and checking their integers.
I think this is for convenience. You see Ok & Cancel buttons on the form, it makes sense to return a vbOk or vbCancel value instead of 0 or 1 for example. Just my opinion

For Martinliss. Suggest adding this to the Form_Unload event. Should a user unload the form by clicking the (X) close button, it will be registered as a cancel action
Code:
If ButtonClicked = 0 Then ButtonClicked = vbCancel