Warren Moore

I would advise glancing then ignoring 'viewport' branch for now, because it really makes things a little more complicated, and having a working Metal renderer in 'master' would be immediately and readily useful as a first step.

Thanks! Can't comment on Metal usage. The ideal is to maximize the amount of stuff that people can reuse directly (aka imgui_impl_metal) in their existing codebase, so additional params are totally ok, probably better to be explicit if it provides more flexibility?

Do think imgui_impl_osx.mm is not useful then? or hadn't had time to look at it? (I'll ditch the icons and replace them with empty and optimized PNG files before squashing+merging so it doesn't get added to git repo size, they are not needed)

In my rush to get my renderer feature-complete, I haven’t had time to take a look, but I’ll do so now.

I think this is still very useful; it’s nice to have the IO stuff factored separately, and I think it’ll work nicely with the Metal renderer. My only concern is the coupling to NSOpenGLView; it doesn’t seem essential (could just be NSWindow)?

Yes this probably ought to be fixed. The whole _osx.mm codebase probably needs feedback from actual Apple users (it's not merged because unfinished and had no one looking at it, if it plugged nicely into e.g. your example I would merge it).