Simpliest way for creating contextMenu for QTableWidget cells

jsulm

@Engelard I don't have a solution, but want to say that you're writing dangerous code: you're not checking whether tempItem is a valid pointer! If it is not your app will crash.
Also, shouldn't it be http://doc.qt.io/qt-5/qtablewidgetitem.html#setBackground?

VRonin

You just need to set the menu policy: http://doc.qt.io/qt-5/qwidget.html#contextMenuPolicy-prop

In MWindow constructor call setContextMenuPolicy(Qt::CustomContextMenu);

Also note what @Christian-Ehrlicher said.
Instead of ui->tableWidMemory->itemAt(pos); you'll probably need ui->tableWidMemory->itemAt(ui->tableWidMemory->mapFromGlobal(mapToGlobal(pos)));

Engelard

@jsulm said in Simpliest way for creating contextMenu for QTableWidget cells:

Also, shouldn't it be http://doc.qt.io/qt-5/qtablewidgetitem.html#setBackground?

Nope. setBackgroundColor is wroked well with leftClick slot.

Engelard

@VRonin nothing changed, still not working..

VRonin

@Engelard said in Simpliest way for creating contextMenu for QTableWidget cells:

nothing changed, still not working

~~Does on_tableWidMemory_customContextMenuRequested get executed at all?~~

Actually realised my mistake. You are asking the context menu only for the table widget so instead of setContextMenuPolicy(Qt::CustomContextMenu); you should have ui->tableWidMemory->setContextMenuPolicy(Qt::CustomContextMenu); and forget about the mapFromGlobal/mapToGlobal part

Engelard

@VRonin said in Simpliest way for creating contextMenu for QTableWidget cells:

@Engelard said in Simpliest way for creating contextMenu for QTableWidget cells:

nothing changed, still not working

~~Does on_tableWidMemory_customContextMenuRequested get executed at all?~~

Lol, no it does'nt. But after your:

Actually realised my mistake. You are asking the context menu only for the table widget so instead of setContextMenuPolicy(Qt::CustomContextMenu); you should have ui->tableWidMemory->setContextMenuPolicy(Qt::CustomContextMenu); and forget about the mapFromGlobal/mapToGlobal part

It called, but whole app crashes)

Engelard

Now it's woring, i removed whole that stuff

QTableWidgetItem *tempItem = ui->tableWidMemory->itemAt(pos);

The only thing is left now:

ui->tableWidMemory->itemAt(pos)->setBackgroundColor(Qt::green);

And

ui->tableWidMemory->setContextMenuPolicy(Qt::CustomContextMenu);

in constructor.

Tnx VRonin for help. Still can't get why no slot for rightclick for QTableWidget cells, would be way efficient and simplier for newcomers. But anyway, at least it working with two extra-lines of code.

jsulm

@Engelard said in Simpliest way for creating contextMenu for QTableWidget cells:

It called, but whole app crashes)

Where do you execute this code? It should be after ui->setupUI

Engelard

P.S. Ah. Now i get it, that new object

QTableWidgetItem *tempItem = ui->tableWidMemory->itemAt(pos);

needed for checking if user rightclick to anything in TableWidget but cells, it would crash without it.

jsulm

@Engelard I told you :-)