tidied up add dialog