Simpliest way for creating contextMenu for QTableWidget cells
-
@SGaist Ah now i get it. You misswrite in first post. Not indexAt but itemAt.
P.S. there is no need to making connect, i just use all stuff inside customContextMenuRequested SLOT.
P.P.S. everything seems to be well, compiles well, but not working i rly can't understand why:
void MWindow::on_tableWidMemory_customContextMenuRequested(const QPoint &pos) { QTableWidgetItem *tempItem = ui->tableWidMemory->itemAt(pos); ui->tableWidMemory->item(tempItem->row(), tempItem->column())->setBackgroundColor(Qt::green); }Also tried:
ui->tableWidMemory->itemAt(pos.x(), pos.y())->setBackgroundColor(Qt::green);Still nothing....
@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? -
wrote on 25 Sept 2018, 08:15 last edited by
You just need to set the menu policy: http://doc.qt.io/qt-5/qwidget.html#contextMenuPolicy-prop
In
MWindowconstructor callsetContextMenuPolicy(Qt::CustomContextMenu);Also note what @Christian-Ehrlicher said.
Instead ofui->tableWidMemory->itemAt(pos);you'll probably needui->tableWidMemory->itemAt(ui->tableWidMemory->mapFromGlobal(mapToGlobal(pos))); -
@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?wrote on 25 Sept 2018, 08:36 last edited by@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.
-
You just need to set the menu policy: http://doc.qt.io/qt-5/qwidget.html#contextMenuPolicy-prop
In
MWindowconstructor callsetContextMenuPolicy(Qt::CustomContextMenu);Also note what @Christian-Ehrlicher said.
Instead ofui->tableWidMemory->itemAt(pos);you'll probably needui->tableWidMemory->itemAt(ui->tableWidMemory->mapFromGlobal(mapToGlobal(pos))); -
wrote on 25 Sept 2018, 08:43 last edited by VRonin
@Engelard said in Simpliest way for creating contextMenu for QTableWidget cells:
nothing changed, still not working
Doeson_tableWidMemory_customContextMenuRequestedget 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 haveui->tableWidMemory->setContextMenuPolicy(Qt::CustomContextMenu);and forget about themapFromGlobal/mapToGlobalpart -
@Engelard said in Simpliest way for creating contextMenu for QTableWidget cells:
nothing changed, still not working
Doeson_tableWidMemory_customContextMenuRequestedget 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 haveui->tableWidMemory->setContextMenuPolicy(Qt::CustomContextMenu);and forget about themapFromGlobal/mapToGlobalpartwrote on 25 Sept 2018, 08:47 last edited by@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
Doeson_tableWidMemory_customContextMenuRequestedget 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 haveui->tableWidMemory->setContextMenuPolicy(Qt::CustomContextMenu);and forget about themapFromGlobal/mapToGlobalpartIt called, but whole app crashes)
-
wrote on 25 Sept 2018, 08:52 last edited by 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.
-
@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
Doeson_tableWidMemory_customContextMenuRequestedget 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 haveui->tableWidMemory->setContextMenuPolicy(Qt::CustomContextMenu);and forget about themapFromGlobal/mapToGlobalpartIt called, but whole app crashes)
@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
-
wrote on 25 Sept 2018, 09:02 last edited by
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.
-
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.
@Engelard I told you :-)
-
J JonB referenced this topic on 22 Apr 2023, 16:20
17/17