review comments
more work on closing dialogs