need return true, so the dialog will close