Don't close the window when clicking "OK" QDialog
-
@Idodoqdo said in Don't close the window when clicking "OK" QDialog:
connect(ui->buttonBox, SIGNAL(accepted()), this, SLOT(SendInfo()));I am somewhat surprised this works. What type is
ui->buttonBox? It does not help that you are using the old styleSIGNAL/SLOT()macros toconnect(), which were deprecated a decade ago. -
@Idodoqdo said in Don't close the window when clicking "OK" QDialog:
connect(ui->buttonBox, SIGNAL(accepted()), this, SLOT(SendInfo()));I am somewhat surprised this works. What type is
ui->buttonBox? It does not help that you are using the old styleSIGNAL/SLOT()macros toconnect(), which were deprecated a decade ago. -
@JonB
rewrote like this:connect(ui->buttonBox, &QDialogButtonBox::accepted, this, &AuthorizationView::SendInfo); -
@Idodoqdo
That is much better/clearer/safer. It will also tell you if anything is wrong at compile-time. Basically for your purposes you should continue to doconnect()s using this syntax all the time.@JonB said in Don't close the window when clicking "OK" QDialog:
That is much better/clearer/safer. It will also tell you if anything is wrong at compile-time. Basically for your purposes you should continue to do connect()s using this syntax all the time.
thx
-
Does using a QDialogButtonBox within a ui file automatically connect its signals with the surrounding QDialog? I have created a QDialog and a QDialogButtonBox with code instead and there I need to connect the signals myself in order to have the buttons of the QDialogButtonBox do anything at all.
-
Does using a QDialogButtonBox within a ui file automatically connect its signals with the surrounding QDialog? I have created a QDialog and a QDialogButtonBox with code instead and there I need to connect the signals myself in order to have the buttons of the QDialogButtonBox do anything at all.
@SimonSchroeder
Is this not what the OP'sconnect(ui->buttonBox, &QDialogButtonBox::accepted, ...)already does? -
@SimonSchroeder
Is this not what the OP'sconnect(ui->buttonBox, &QDialogButtonBox::accepted, ...)already does?@JonB said in Don't close the window when clicking "OK" QDialog:
Is this not what the OP's connect(ui->buttonBox, &QDialogButtonBox::accepted, ...) already does?
Usually, you would connect to QDialog::accept. So, somewhere I would expect that the dialog is closed on purpose if there is no automatic connection between QDialogButtonBox::accepted and QDialog::accept. Why else would it close?
-
@JonB said in Don't close the window when clicking "OK" QDialog:
Is this not what the OP's connect(ui->buttonBox, &QDialogButtonBox::accepted, ...) already does?
Usually, you would connect to QDialog::accept. So, somewhere I would expect that the dialog is closed on purpose if there is no automatic connection between QDialogButtonBox::accepted and QDialog::accept. Why else would it close?
@SimonSchroeder
Ah, I think I understand you now. I haven't used aQDialogButtonBox. You don't mind theQDialogButtonBox::acceptedsignal having a slot, but you would expect that to accept the dialog and then the dialog'sacceptedto do the action, right? Likeconnect(ui->buttonBox, &QDialogButtonBox::accepted, this, &QDialog::accept); connect(this, &QDialog::accepted, this, &AuthorizationView::SendInfo);Is this what you had in mind?
But I think there is a wrinkle. The OP's question is "Don't close the window when clicking "OK" QDialog", which the above would do? I think they want the authorization to happen upon accepting/clicking the "OK" button in the dialog button box but before the dialog gets accepted. Then they verify the data and only accept the overall dialog if that has succeeded. I think that is why they do not attach
QDialogButtonBox::acceptedtoQDialog::accept? -
@SimonSchroeder
Ah, I think I understand you now. I haven't used aQDialogButtonBox. You don't mind theQDialogButtonBox::acceptedsignal having a slot, but you would expect that to accept the dialog and then the dialog'sacceptedto do the action, right? Likeconnect(ui->buttonBox, &QDialogButtonBox::accepted, this, &QDialog::accept); connect(this, &QDialog::accepted, this, &AuthorizationView::SendInfo);Is this what you had in mind?
But I think there is a wrinkle. The OP's question is "Don't close the window when clicking "OK" QDialog", which the above would do? I think they want the authorization to happen upon accepting/clicking the "OK" button in the dialog button box but before the dialog gets accepted. Then they verify the data and only accept the overall dialog if that has succeeded. I think that is why they do not attach
QDialogButtonBox::acceptedtoQDialog::accept?@JonB I'm not saying you should connect those (certainly not if you don't want the dialog to close upon clicking 'OK'). I was saying that either there is some auto-connect because of the ui-file (which I don't know) or somewhere in the code the dialog is closed on purpose. If the OP thinks he's not closing the dialog himself on purpose, then the auto-connect would be one of my guesses to check.
-
@JonB I'm not saying you should connect those (certainly not if you don't want the dialog to close upon clicking 'OK'). I was saying that either there is some auto-connect because of the ui-file (which I don't know) or somewhere in the code the dialog is closed on purpose. If the OP thinks he's not closing the dialog himself on purpose, then the auto-connect would be one of my guesses to check.
@SimonSchroeder said in Don't close the window when clicking "OK" QDialog:
or somewhere in the code the dialog is closed on purpose. If the OP thinks he's not closing the dialog himself on purpose
In the OP's
void AuthorizationView::ResultAuthorization(bool result)he hasif (result) { this->close();His code emits a signal with the name & password, the authorizer presumably calls above method with success/failure parameter for authorization. If success it closes the dialog, but if failure it puts up a message box but stays inside the dialog, presumably for the user to try again. That is why the button box's "OK"/accepted should not be connected directly and unconditionally to the dialog's accepted.
