How to disable a button in the slot function?

[andre]

Disabling a button will result in a redraw. That happens when the eventloop gets a chance to run, which you don't give it because your "other things that may take several seconds" are where your application spends its time. There are several ways to fix this, but I think for a beginner processEvents() is the easiest*:

@
#include <QApplication>

void on_button1_clicked()
{
button2.setEnabled(false);
QApplication::processEvents();
//do other things that may take serveral seconds

button2.setEnabled(true);
}
@

As a side note: it is smart to give your buttons more descriptive names than just "button1" and "button2". Personally, I use names like "cmdStartLongAction" and "cmdCancel". For those, you understand what they do without reading the code.

*) Even though I'm not a big fan of it myself, and try to avoid it if possible.

[giesbert]

But take care:

 *QApplication::processEvents();*

opens the event loop and your synchronous flow is not synchronous!
It might be that user interactions / signals from other threads etc. come in here and do some (long time) operation. This is only meant as a hint, as it might bring problems to open the event loop.

If you only want the drawing to be done, you can use one of the following:

1.) this excludes all use actions.

@
#include <QApplication>

void on_button1_clicked()
{
button2.setEnabled(false);
QApplication::processEvents(QEventLoop::ExcludeUserInputEvents);
//do other things that may take serveral seconds

button2.setEnabled(true);
}
@

2.) another way is to draw the button directly:

@
#include <QApplication>

void on_button1_clicked()
{
button2.setEnabled(false);
button2.repaint(); // this does an repaint
//do other things that may take serveral seconds

}
@

[andre]

I did not say that I'm not a big fan for nothing ;-)

There are other approaches. Perhaps you should re-think if you actually want to block your entire program for a few seconds while it is processing your things. Personally, I would not like that. A UI needs to be snappy and responsive, no matter if the program is doing work or not.

[udrnrso]

[quote author="Andre" date="1301383065"]Disabling a button will result in a redraw. That happens when the eventloop gets a chance to run, which you don't give it because your "other things that may take several seconds" are where your application spends its time. There are several ways to fix this, but I think for a beginner processEvents() is the easiest*: @ #include <QApplication> void on_button1_clicked() { button2.setEnabled(false); QApplication::processEvents(); //do other things that may take serveral seconds button2.setEnabled(true); } @ As a side note: it is smart to give your buttons more descriptive names than just "button1" and "button2". Personally, I use names like "cmdStartLongAction" and "cmdCancel". For those, you understand what they do without reading the code. *) Even though I'm not a big fan of it myself, and try to avoid it if possible. [/quote]

thx for reply!

I have tried what you wrote as above ,but it didnt work, and I also tried button2.repaint(); but still
, no effect! what should I do now? Use QThread or ?

[andre]

Then, perhaps, we misunderstood what your exact problem is. Could you try to re-word your original question to try to make more clear what you want to achieve?

[udrnrso]

[quote author="Andre" date="1301388222"]Then, perhaps, we misunderstood what your exact problem is. Could you try to re-word your original question to try to make more clear what you want to achieve?[/quote]

Sorry for my poor English and thx for your patience! I have two buttons : button1 and button2 ,when I click button1, just before doing other work, disable button2, when the work done, enable button2``that is all I wanna do ! must I use thread?

[andre]

No, you don't need to use threads. In fact, given the level of Qt you seem to have, I would recommend you ignore them for the time being :)

[giesbert]

If you just don't want to react on button 2 while the work from button 1 is ongoing, you don't need to disable the button. If everything is running in main thread, there will be no button handling until the slot is finished.

Can you please provide a small, running app to check where the error is located. I'm sure, the way with repaint or processEvents works.

[andre]

Well, I guess you would want to have visual feedback that an action will not be taken, and thus, disable the button.

[udrnrso]

[quote author="Andre" date="1301409493"]Well, I guess you would want to have visual feedback that an action will not be taken, and thus, disable the button. [/quote]

Yes, that is exactly what I want.I complied and ran the app on vmware, does it matter?

[udrnrso]

[quote author="Gerolf" date="1301407400"]If you just don't want to react on button 2 while the work from button 1 is ongoing, you don't need to disable the button. If everything is running in main thread, there will be no button handling until the slot is finished. Can you please provide a small, running app to check where the error is located. I'm sure, the way with repaint or processEvents works.[/quote]
I tried processEvents on windows, it works, while on linux, that doesnt work. It confused me.
Finally , I combine repaint and processEvents as following, it works on both windows and linux now.
I have no idea why?
@
void on_button1_clicked()
{
ui->button2->setEnabled(false);
repaint(ui->button2->rect());
QApplication::processEvents(QEventLoop::ExcludeUserInputEvents);
// Do something
ui->button2->setEnabled(true);

}
@

[giesbert]

you should use repaint in a different manner:

@
void on_button1_clicked()
{
ui->button2->setEnabled(false);
ui->button2->repaint();
// Do something
ui->button2->setEnabled(true);
}
@

perhaps then it also works on linux?
You call repaint typically on the widget you want to repaint, not on yourself with a rectangle of a child. This could lead to no repaint (as your own area is hidden behind the child --> no redraw of your widget is needed).

[udrnrso]

[quote author="Gerolf" date="1301496355"]you should use repaint in a different manner: @ void on_button1_clicked() { ui->button2->setEnabled(false); ui->button2->repaint(); // Do something ui->button2->setEnabled(true); } @ perhaps then it also works on linux? You call repaint typically on the widget you want to repaint, not on yourself with a rectangle of a child. This could lead to no repaint (as your own area is hidden behind the child --> no redraw of your widget is needed).[/quote]
Thx for your answer!
Dear Gerolf , I tried your code, also, it perform the same way , unless I put a processEvents() behind ui->button2->repaint(); Is it a bug ?

[giesbert]

as per description (and my current usage) repaint should repaint emediatly. In the "docs":http://doc.qt.nokia.com/latest/qwidget.html#repaint there is stated:

[quote]
Repaints the widget directly by calling paintEvent() immediately, unless updates are disabled or the widget is hidden.

We suggest only using repaint() if you need an immediate repaint, for example during animation. In almost all circumstances update() is better, as it permits Qt to optimize for speed and minimize flicker.

Warning: If you call repaint() in a function which may itself be called from paintEvent(), you may get infinite recursion. The update() function never causes recursion.
[/quote]

[andre]

According to the documentation, repaint() should trigger an immediate repaint. If it doesn't, then indeed I think you have a bug on your hands.

[ivan.todorovich]

Couldn't it be a VMWare problem?
I mean, from my own experience with VMWare, sometimes it repaints blocks just like it were a VNC window. And if it is trying to get the window's contents to paint it on the host when the event loop is blocked it might fail.

Maybe we should try the same code on a linux host. Unfortunately I don't have it right now.

[Revu]

Hi All,
If there are some 'N' buttons that need to be disabled with the same scenario,then how that can be achieved? With the complexity that the slots are interconnected with the independent windows.Like button1 click activates an independent window that has a slot to activate another independent window.With the use of modal dialog this can be achieved only between two windows then how about the rest of the active window?

[andre]

If you have multiple active windows in your application, and you want a modal dialog that is modal relative to all of them, you need an ApplicationModal dialog box. You can make that by using QDialog::setModel(true) or by using QWidget::setWindowModality(Qt::ApplicationModal).

For disabling and enabling a whole host of widgets in one go, I would change the setup from the last page a bit. Instead of directly disabling and enabling your buttons, I would either simply use signals and slots or use a list of widgets. Using the signal/slot method, you would declare a signal like this:

in the class declaration
@
signals:
void enableButton(bool);
@

Then, of course, you need to hookup your buttons to this signal, for instance in your constructor:
@
//make sure this runs once, for instance by putting it in the constructor
connect (this, SIGNAL(enableButton(bool)), m_ui->button1, SLOT(setEnabled(bool)));
connect (this, SIGNAL(enableButton(bool)), m_ui->button2, SLOT(setEnabled(bool)));
connect (this, SIGNAL(enableButton(bool)), m_ui->button3, SLOT(setEnabled(bool)));
//...
connect (this, SIGNAL(enableButton(bool)), m_ui->buttonN, SLOT(setEnabled(bool)));
@

In your method, you then do something like this:
@
#include <QApplication>

void on_button1_clicked()
{
emit enableButton(false);
QApplication::processEvents(QEventLoop::ExcludeUserInputEvents);
//do other things that may take serveral seconds

emit enableButton(true);
}
@

[Revu]

Thank You.It works fine so far it is concerned with button1.When I use button2 I want rest of them to be inactive/disabled.But writing the same code repeatedly by making changes for button that is out of group is complicated.
Well I tried with window modality as you have mentioned but still could not get the required result.
Is it possible to check for any active window(by using isActiveWindow) before I click on a button?

Thank You

[andre]

So, simply disable all buttons like I show above, and re-enable the one button you like to keep enabled?

@
#include <QApplication>

void on_button1_clicked()
{
emit enableButton(false);
button1->setEnabled(true);
QApplication::processEvents(QEventLoop::ExcludeUserInputEvents);
//do other things that may take serveral seconds

emit enableButton(true);
}

void on_button2_clicked()
{
emit enableButton(false);
button2->setEnabled(true);
QApplication::processEvents(QEventLoop::ExcludeUserInputEvents);
//do something else that may also take serveral seconds

emit enableButton(true);
}
@

There are other solutions, of course.

I don't understand where you're going with your last question. What do you mean that you would like to check what the active window is (what do you mean by that, anyway?) when a button is clicked? That would obviously be the window that button is on then, would it not?