Just 2 minor nitpicks, otherwise the code looks good (haven't tested yet):
+ Window *TopLevelWindowModel::createWindow(unityapi::MirSurfaceInterface *surface)
-> const qualifier, it's a getter
+ readonly property bool decorated
-> this could be rewritten to be a direct binding, instead of a function
« Back to merge proposal
Just 2 minor nitpicks, otherwise the code looks good (haven't tested yet):
+ Window *TopLevelWindow Model:: createWindow( unityapi: :MirSurfaceInte rface *surface)
-> const qualifier, it's a getter
+ readonly property bool decorated
-> this could be rewritten to be a direct binding, instead of a function