Bring QFileDialog to the front

[Ratzz]

@JoeCFD
Can you show me your code?

[JoeCFD]

QFileDialog * dialog = new QFilieDialog( this);
dialog->setWindowTitle( "App" );
dialog->setWindowModality( Qt::ApplicationModal );
dialog->setWindowFlags( Qt::WindowStaysOnTop );
dialog->setAcceptMode(QFileDialog::AcceptSave );
....
if ( QDialog::Accepted == dialog->exec() )
{
do something
}

[JonB]

@JoeCFD
What is this, which you pass as the parent to QFileDialog( this)?
I'm not sure how that combines with setWindowModality( Qt::ApplicationModal ) anyway....
I have not had to use setWindowModality( Qt::ApplicationModal ) nor setWindowFlags( Qt::WindowStaysOnTop );, but I admit I haven't done Windows/DLLs.

I think from what you write that if you ensure you pass the main window as the parent to new QFileDialog( this) you will get you want.

[JoeCFD]

@JonB Right.Here "this" is another widget, not the mainwindow. If I pass my mainwindow as this, it is all right. However, I can not use it since this piece of code is in a DLL. The mainwindow is not available.

[JonB]

@JoeCFD
Then I think that is your problem, because IIRC QFileDialog likes the main window not a widget as its parent if it's to behave modally the way you want.

If that is true, the DLL is running in your program's user space, so when you say "The mainwindow is not available" you can always go get it. That's what I've done in my code (though nothing to do with DLLs/Windows). You might at least try that to see if it solves your problem?

[JoeCFD]

@JonB I can send a signal to the mainwindow to do the popup over there and solve the problem. But it is not convienent. Thanks anyway.

[JoeCFD]

@JoeCFD It turns out it is a virtual machine issue. On Linux within a virtual machine this problem shows up. Not sure why. No problem on a standalone Linux.

[JonB]

@JoeCFD

I can send a signal to the mainwindow to do the popup over there and solve the problem. But it is not convienent. Thanks anyway.

You could, but that wasn't what I intended, I understand that would mean re-architecting. I meant: discover it where you open the file dialog, in your DLL, without having to pass it around in code. If you're interested, I wrote:

def findMainWindow() -> typing.Union[QMainWindow, None]:
    # Global function to find the (open) QMainWindow in application
    app = QApplication.instance()
    for widget in app.topLevelWidgets():
        if isinstance(widget, QMainWindow):
            return widget
    return None

[JoeCFD]

@JonB Got it.This is something new for me. Thanks a lot.

something like the following works.

QWidget * widget{ nullptr };
for ( widget : QApplication::toLevelWidgets() )
{
if ( widget->objectName() == QString( "MyMainWindow" ) )
{
break;
}
}

[JonB]

@JoeCFD
If this helps you I am glad.

If you wanted to write the direct equivalent of Python's if isinstance(widget, QMainWindow) in C++ you should be able to use if (dynamic_cast<QMainWindow*>(widget)).

[JoeCFD]

@JonB My C++ code is good enough to solve the issue. No casting is needed. Many thanks again.