Appreciate the PR. :)
It's been a few years since I touched this particular functionality, but (IIRC) I think I opted not to expose these because modern macOS modal views should really be using the sheets API. The filesystem
module uses this API as it just works around the thorny issue of handling file selection in a synchronous manner - this is typically an operation where "stop the world" isn't the end of the world.
That said, I think the PR is probably still useful - but I'd maybe add some additional documentation to the function docs that just notes the above. I also think you're right and that ModalResponse
should probably just move out if these methods are public. on Add wrappers for starting and stopping modal windows
a month ago