gmayor,

thanks for you help it has solved my issue, apologies about the code however I am very much a beginner learning along the way.

I have userform1 as a form for data entry into a templated letter which transfers the textbox values entered to variables and then into DoC Var fields then applying formatting.

The other userform2 is simple a box to show the listbox with the addresses and a button to import. I had to pass this as 6 separate strings as the userform1 is name , address line 1,2,3,4 and Postcode.

I realise some mistakes in the code as you pointed out I had originally used oRng to put the text right into the document but changed my mind. Purely because I wanted the user to have the visual of the address being imported otherwise it would be hidden behind the userform.

Also because I have checks on the data in the textbox not allowing most fields to be empty.

I am planning to re code with meaningful names the project started off very small but soon grew and I realised I should have done this from the start.

I have learned a lot since I started however I am always looking for tips so thank you for your time and help, if you have any feedback I'd love to hear it, I am always looking for a better way , I have taken your points onboard.