That looks ok, though one would usually do the connect from outside the dialog instead of passing a reference to the mainwindow.

Since you have log output, which of those to you get?

Cheers,
_