more work on the PersonDialog
complete the PersonDialog