How to prevent the mouse click event from happening when the button is Disabled

[jsulm]

@ManiRon Are you sure you're disabling the correct button?
As I said: I can't reproduce such behaviour, there must be something in your code.
Also you can disable your style sheet and test again.
And to be sure: in the title you're talking about event, but you mean clicked() signal, correct?

[ManiRon]

@jsulm Yes clicked signal

[J.Hilk]

@ManiRon
can you create a minimal example?

In fact you should!
Or maybe @jsulm can share his for you ;)

[jsulm]

@J.Hilk @ManiRon It's as simple as

connect(ui->pushButton, &QPushButton::clicked, [this](bool){ qDebug() << "Clicked"; ui->pushButton->setDisabled(true);});

And I'm still interested in an answer to this question: Are you sure you're disabling the correct button?

[Pradeep P N]

@jsulm @J-Hilk

I think the problem here for @ManiRon is the style sheet for the QPushButton

The Button is actually disabled on Click, but due to the StyleSheet the color difference is not found.

void MainWindow::on_pushButton_clicked()
{
    qDebug() << Q_FUNC_INFO;
    ui->label->setText("Button is Disabled");
    ui->pushButton->setEnabled(false);
}

[ManiRon]

@jsulm Yes Actually i am disabling the button for 3sec only so it accepts the signal again after 3sec , which i came to know when i printed the time with seconds

[ManiRon]

@jsulm I think the problem here is the timeout that i have given , i think its less

[Pradeep P N]

@ManiRon
I just tried with Timer it works fine.
The Button is disabled for 3s.

• Check with your stylesheet.

MainWindow::MainWindow(QWidget *parent) :
    QMainWindow(parent),
    ui(new Ui::MainWindow)
{
    ui->setupUi(this);
    ui->label->setVisible(false);

    tmr = new QTimer(this);
    connect(tmr, &QTimer::timeout, [this]{
        ui->pushButton->setEnabled(true);
    });
}

void MainWindow::on_pushButton_clicked()
{
    qDebug() << Q_FUNC_INFO << QTime::currentTime().toString();

    tmr->start();
    tmr->setInterval(3000);

    ui->pushButton->setEnabled(false);
}

[Pradeep P N]

Hi @ManiRon

Try this with your setStyleSheet()

ui->pushButton->setStyleSheet("QPushButton {"
                              "background-color:#44c767;"
                              "}"
                              "QPushButton:disabled {"
                              "background-color:gray;"
                              "}");

All the best.

[ManiRon]

@Pradeep-P-N @jsulm @J-Hilk
when i created a sample i was absorbing one thing , that when i click the button i disabled it , but while i click the button when it is disabled after enabling the button the click signal gets emitted and the click function is executed by the click signal which was occuring when i clicked the button when it was disabled

[J.Hilk]

@ManiRon said in How to prevent the mouse click event from happening when the button is Disabled:

but while i click the button when it is disabled after enabling the button the click signal gets emitted and the click function is executed

If true, then that's a bug in the library!

[ManiRon]

@J.Hilk how it can be solved ?

[Pradeep P N]

@J.Hilk
i am bit confused with the statement in fact.

"but while i click the button when it is disabled after enabling the button the click signal gets emitted and the click function is executed"

• i click the button when it is disabled -> Nothing will work as Button is Disabled.
• after enabling the button the click signal gets emitted -> Yes as after 3s timer the button is enabled and can be clicked.

[J.Hilk]

@Pradeep-P-N

What I get out of @ManiRon statement is:

• Button is disabled
• Button gets pressed/clicked
• • nothing happens, as expected
• Button gets set to enabled
• • clicked signal is emitted without button interaction

Thats not normal behavior in my opinion

I'll make a quick test case

@ManiRon what Qt Version what OS btw?

[Pradeep P N]

Hmmm, Got it @J-Hilk
Thanks.

@ManiRon Can you share a sample code ?
You said you are using the QTimer.
Just wanted to check if there is anything mismatch with the code.

[J.Hilk]

@J.Hilk said in How to prevent the mouse click event from happening when the button is Disabled:

If true, then that's a bug in the library!

Nope, not true 5.12.4 MacOS, no unexpected clicked emits

int main(int argc, char *argv[])
{

    QApplication a(argc, argv);

    auto *btn = new QPushButton("Enabled");
    btn->resize(200, 50);
    btn->show();

    QObject::connect(btn, &QPushButton::clicked, [btn]()->void{
                         btn->setEnabled(false);
                         btn->setText("Disabled");
                         qDebug("Btn clicked");
                     });
    QObject::connect(btn, &QPushButton::pressed, []()->void{qDebug("Btn Pressed"); });
    QObject::connect(btn, &QPushButton::released, []()->void{qDebug("Btn Released"); });

    QTimer t_reset;
    t_reset.setInterval(3000);
    QObject::connect(&t_reset, &QTimer::timeout, btn, [btn]()->void{btn->setEnabled(true); btn->setText("Enabled");});

    QObject::connect(btn, &QPushButton::clicked, &t_reset, QOverload<>::of(&QTimer::start));

    return  a.exec();
}

[Pradeep P N]

@J-Hilk
Even i tried on 5.12.3 - Ubuntu 16.04 & the above code ( From my post ).
Also the one shared by you.
It works fine.

• No Click called related to button enable/disable status.

[jsulm]

@ManiRon Please add

qDebug() << ui->pushButton->enabled();

to your slot and see what it outputs when you click on the button while it is disabled.
And please debug you app more carefully: you say you have a timeout to activate the button again (you should have said that in your first post already)? Did you make sure the button was not yet activated? Is it the same if you disable you style sheet?

[ManiRon]

@J.Hilk QT 5.5.0 QT Creator version 3.4.2 OS Windows Vista

[ManiRon]

@jsulm Yes i made sure cause i enable the button after 3s only , and this statement is not there , it throws error (qDebug() << ui->pushButton->enabled())