Why is the action from a context menu on a popup dialog not triggered?

[SGaist]

Very important detail indeed. Your dialog is modal to the widget you pass as parent thus it will block all inputs on it. See the WindowModality enum.

[Jakob]

@SGaist Thanx for answering - I thought I understand this, but the signal I expected to be arriving in the dialog, hence not the underlying (blocked) widget, and then be picked up by the handler, which is also implemented in the dialog itself. As far as I can tell all 'action' is happening, and restricted to the dialog itself, and not to the underlying, blocked widget.

Or am I misunderstanding something here? The funny thing, when the same dialog is created and displayed like this without the Qt::Popup flag, these triggered signals do arrive.

[SGaist]

My bad, I've mixed your widget setup with another one. You understood right.

Do you experience the same if you make a dialog instance on the heap with the dialog->setAttribute(Qt::WA_DeleteOnClose); and call open on it rather than exec ?

[Jakob]

@SGaist Unfortunately that doesn't improve things.

In the meantime I attached a debugger, and put a breakpoint in the QAction::activate implementation, which seems to the function that should handle all triggered actions. I never hit this breakpoint when I click the 'Copy as text' menu item, nor when I click the default constructed 'Select all' menu item.

It seems almost as if the connection is not made for some reason, or destroyed before it is handled.

[Jakob]

@SGaist I did some more investigation inside the debugger.

• the QMenu::mouseReleaseEvent() method is called
• However, the d->currentAction is nullptr, and for that reason the handler never executes d->activeAction()

[EDIT] Diving further, it seems in QMenuPrivate::setSyncAction() the currentAction is not set, while it is set when not passing the Qt::Popup flag.

[SGaist]

On which OS are you currently ?

[Jakob]

@SGaist The OS is windows 7, with qt 5.6.0.

[Jakob]

@SGaist I managed to create a compilable example that reproduces the problem. Hopefully that helps.

When using the line commented with OKE the line that prints some text to qInfo() is executed - when using the other line, that function is never executed (because the action is never 'activated')

#include <QAction>
#include <QApplication>
#include <QFrame>
#include <QMenu>
#include <QtDebug>
#include <QTextEdit>
#include <QVBoxLayout>
#include <QWidget>
#include <QWindow>

class Popup : public QFrame
{
public:
    Popup()
        : QFrame(nullptr) // OKE
        //: QFrame(nullptr, Qt::Popup) // NOT OKE
        
    {
        Setup();
    }

    void SetInitialText(const QString& t)
    {
        txt->setText(t);
    }

private:
    void ShowContextMenu(const QPoint& pt)
    {
        auto menu = txt->createStandardContextMenu();
        auto custom = menu->addAction("Formatted copy");
        connect(custom, &QAction::triggered, this, &Popup::CustomActionTriggered);
        menu->exec(this->mapToGlobal(pt));
        delete menu;
    }

    void CustomActionTriggered(bool)
    {
        qInfo() << "Custom action triggered";
    }

    void Setup()
    {
        layout = new QVBoxLayout(this);
        txt = new QTextEdit(this);
        txt->setContextMenuPolicy(Qt::CustomContextMenu);
        txt->setFrameShape(QFrame::Box);
        txt->setFrameShadow(QFrame::Plain);
        txt->setLineWrapMode(QTextEdit::NoWrap);
        txt->setReadOnly(true);
        txt->setTextInteractionFlags(Qt::TextSelectableByMouse);
        connect(txt, &QTextEdit::customContextMenuRequested,
            this, &Popup::ShowContextMenu);
        layout->addWidget(txt);
    }

    QVBoxLayout* layout;
    QTextEdit* txt;
};

void CreatePopup()
{
    auto popup = new Popup();
    popup->setAttribute(Qt::WA_DeleteOnClose);
    popup->SetInitialText("Some text to display");
    popup->show();
}

int main(int argc, char** argv)
{
    QApplication app(argc, argv);

    QWidget window;
    QAction* showDialog = new QAction("Show dialog", &window);
    QObject::connect(showDialog, &QAction::triggered, []{ CreatePopup(); });
    window.addAction(showDialog);
    window.setContextMenuPolicy(Qt::ActionsContextMenu);

    window.show();

    return app.exec();
}

[Jakob]

I also created a bug report for this, as I'm getting more and more convinced it really is a bug, see https://bugreports.qt.io/browse/QTBUG-54820

[SGaist]

Thanks !

[Jakob]

@SGaist According to comments in the bug report the posted code should work in qt 5.6.1, and I have been able to verify this is indeed the case. So I guess I should consider this question as 'answered' then. Thanx for the help