Problem connecting QAction to slot

JonB

@Jakob-Clausen
Put a qDebug() into the slot lambda?

Jakob Clausen

No. Nothing happens.

JonB

@Jakob-Clausen
I have not used a checkable QAction. But I'm thinking it does not fire triggered(), it fires changed() instead? Because it no longer triggers an "action"? Check the docs/comment out the setCheckable(true).

Jakob Clausen

If I comment out setCheckable() it gives the same result.
I have also tried to connect changed(), and it does not work either.

If I use a pushbutton instread, then I can make it work

    Button* button = new Button(this, name);
    button->setCheckable(true);
    connect(button, &Button::pressed, this, [=]() {workspace->setCurrentIndex(index);});
    return addWidget(button);

Is this a bad way to do it?
I guess I manually have to uncheck the previous button that was checked.

jsulm

@Jakob-Clausen Isn't https://doc.qt.io/qt-5/qaction.html#toggled what you're looking for?

Jakob Clausen

Same result.

JonB

@Jakob-Clausen
Since it ought to work, and we are about to give up :), can you test it without that QActionGroup, just in case...? Else test it in a tiny, standalone.

KroMignon

@Jakob-Clausen said in Problem connecting QAction to slot:

Same result.

Maybe a silly question, but did you add the action to a widget?
Extract from QAction documentation:

Note that an action must be added to a widget before it can be used; this is also true when the shortcut should be global (i.e., Qt::ApplicationShortcut as Qt::ShortcutContext).

Jakob Clausen

@KroMignon Maybe so. But that was it. If I add it, then it works. Clazy still complains, and I don't know what that means. But now its working. :-)
Thank you very much.

KroMignon

@Jakob-Clausen said in Problem connecting QAction to slot:

Clazy still complains

The Clazy complain seems to be a bug in Clazy/QtCreator ==> https://bugreports.qt.io/browse/QTCREATORBUG-25165