Sounds good. The way I'd prefer to do it generally is for the dialog subclass to be able to override what the submit button would normally do. As long as this works for you though, lgtm.