Close QDialog by code

mrjj

Maybe also
http://doc.qt.io/qt-5/qdialog.html#reject

Sikarjan

Thanks for all the hints so far but none worked.

void InsertImageDialog::on_pushButton_clicked()
{
    QString imgSrc = QFileDialog::getOpenFileName(this, tr("Select image"), mPath, tr("Image Files (*.png *.jpg *.gif);; All files (*)"));

    if(imgSrc.isEmpty()){
        this->reject(); // I also tried ui and ui->buttonBox
        return;
    }
}

This is not working. The QDialig is not closed. I don't understand it. According to the documentation it should work.

void QDialog::rejected()

This signal is emitted when the dialog has been rejected either by the user or by calling reject() or done() with the QDialog::Rejected argument.

When the QFileDialog is canceled the imgSrc string is empty, I double checked that. The code inside the statement should be executed. What could I possibly do wrong???

SGaist

Hi,

How are you using your dialog ?

Sikarjan

Hi,

I call the dialog as a modal dialog:

void MainWindow::on_actionImage_triggered()
{
    CodeEditor *mEditor = checkForEditor();
    InsertImageDialog insertImage;
    insertImage.setModal(true);
    connect(&insertImage, SIGNAL(insertImage(QString,int,int,QString,QString)), mEditor, SLOT(insertImage(QString,int,int,QString,QString)));
    insertImage.exec();
}

SGaist

Can you share the complete code of your dialog ?

Sikarjan

Hi,

while trying to build minimal example I found the issue but not the solution. I am calling the on_pushButton_clicked() at the end of the constructor because selecting a file is the first thing the user needs to do. I guess the constructor is not finished while the user picks a file and therefore the reject is not available at the point.

How can I call the pushButton event directly after the dialog is opened?

mrjj

@Sikarjan said in Close QDialog by code:

How can I call the pushButton event directly after the dialog is opened?

What about letting the user press the Select Image button ?

If you really want to popup automatically, you can use a timer.
QTimer::singleShot(5000, this, SLOT(your_slot_function()));

there is also
showEvent(QShowEvent * event )
( you must implement in InsertImageDialog )

Sikarjan

Thanks @mrjj the QTimer solution works but it is not very nice. First the Dialog opens and then with some delay the select file menu. There is always a delay, even if you put the time to 0 ms. Also the select file window is positioned oddly. It is not centered over the dialog.

I did not try the other solution suggested.

mrjj

@Sikarjan
Well the whole idea of popping the image dialog over
another dialog is a bit unusual so that might also be the reason its not working super well. :)

showEvent might be triggered multiple times so make sure u handle that.

I wonder if u just can do it like

QString imgSrc = QFileDialog::getOpenFileName(this, tr("Select image"), mPath, tr("Image Files (*.png *.jpg .gif);; All files ()"));
if( ! imgSrc.isEmpty()) { // if NOT empty
TheOTherdialog dia;
if ( dia.exec() ) {
// do code
}
}

So ask image name.
If selected something
pop the dialog and do X
if ok to dialog, process

Sikarjan

I see. I should open the dialog only when I have a valid file selected. Sure, that would be possible. It would be a bit more code since I have to do the select and check twice, once in the mainWindow and once in the dialog.

Thanks for the idea, did not think about that. =)