Hi Charles,
been fiddling and tweaking and this code works
Code:
Sub ShowForm1(control As IRibbonControl)
iUserForm1.Show vbModeless
End Sub
Sub HideForm1(control As IRibbonControl)
iUserForm1.Hide
End Sub
Now I have a suspicion that it clashed with another userform I had in Normal Template.
The macros for showing the form were the same - as it was in a different template i thought this would be ok
They were both named
Sub ShowForm()
Sub HideForm()
So for the iUserform I renamed it to
Sub ShowForm1()
Sub Hideform1()
it seems to be picking up the user form now.
Let me do some more testing..but i am on the right track i think

thanks
dan