Can't add menu to QSystemTrayIcon

Sucharek

Hi, I can't seem to add a menu to QSystemTrayIcon.
I've followed this tutorial, which showed me how to get the icon to the system tray, but now I can't add a context menu.
Here's what I have:

tray = new QSystemTrayIcon(this); //tray declared in .h file as a class
tray->setVisible(true);
tray->setIcon(QIcon(":/icons/clock.png"));
tray->showMessage("Screen on time", "Screen on time service enabled");
QMenu trayMenu;
trayMenu.addSection("menu");
tray->setContextMenu(trayMenu);

JonB

@Sucharek said in Can't add menu to QSystemTrayIcon:

QMenu trayMenu;

Not my area, and I have no idea whether the whole concept works/is supposed to work. But if this your actual code (which who knows, since it's QSystemTrayIcon::setContextMenu(QMenu *menu) and you are passing a QMenu not a QMenu * so it should not compile), the QMenu trayMenu will go out of scope and be destroyed?

Sucharek

Hi @JonB, when I try doing QMenu *trayMenu = nullptr;, it will crash.

JonB

@Sucharek
Of course it will! Why should passing it a nullptr for its QMenu be acceptable?
C/C++ basics: consider what you need to do about the scope/lifetime of the QMenu you pass to it.

Sucharek

Oh, yeah, got it now, thanks.
Just need to give QMenu proper configuration now, because it just 2 pixels lol.

JonB

@Sucharek
I imagine you made the QMenu trayMenu or QMenu *trayMenu = new QMenu a member variable of some class, which persists for its desired lifetime. Note that in Qt you often have to consider this lifetime declaration for various things which need to persist; we see lots of posts where user has only created object as a method local variable and does not realise it gets destroyed on scope closure, so you are not the first to have come a cropper on this! :)

Sucharek

@JonB yeah, that's exactly what I've done.
Thanks for your help :)