Misplaced Context Menu

[kandalf]

Hi guys, I've been playing around with a context menu on a QListWidget and I can't make it work.
I have a slot connected to the customContextMenuRequested(const QPoint pos&) which creates a QMenu and opens it in "pos" (theoretically).
The thing is that I can't show the menu on the right place. It shows itself way too far from mouse, even when I debug the itemAt(pos)->text() and can see the right item is selected.
This is my code:

void AdminWindow::contextMenuRequested( const QPoint &pos )
{
	if (tablesList->itemAt(pos))
	{
		QMenu popupMenu(tablesList);
		connect( popupMenu.addAction("&Edit Table"), SIGNAL(triggered()), this, SLOT(editActionTriggered()) );
		connect( popupMenu.addAction("&Drop Table"), SIGNAL(triggered()), this, SLOT(dropActionTriggered()) );
	// 	qWarning(tablesList->itemAt(pos)->text().toAscii());
 
		popupMenu.exec(pos + QPoint(tablesList->horizontalScrollBar()->value(), tablesList->verticalScrollBar()->value()));
	}
}

void AdminWindow::contextMenuRequested( const QPoint &pos )
{
	if (tablesList->itemAt(pos))
	{
		QMenu popupMenu(tablesList);
		connect( popupMenu.addAction("&Edit Table"), SIGNAL(triggered()), this, SLOT(editActionTriggered()) );
		connect( popupMenu.addAction("&Drop Table"), SIGNAL(triggered()), this, SLOT(dropActionTriggered()) );
	// 	qWarning(tablesList->itemAt(pos)->text().toAscii());
		
		popupMenu.exec(pos + QPoint(tablesList->horizontalScrollBar()->value(), tablesList->verticalScrollBar()->value()));
	}
}

To copy to clipboard, switch view to plain text mode 

I tried adding values from scrollbars as I saw in this forum, but no luck.
I also tried with mapToParent() and mapToGlobal(), same results.
Any ideas?

Thanks a lot in advance.

[jrideout]

Try

popupMenu.exec( mapToGlobal( pos ) );

popupMenu.exec( mapToGlobal( pos ) );

To copy to clipboard, switch view to plain text mode 

[kandalf]

Try

popupMenu.exec( mapToGlobal( pos ) );

popupMenu.exec( mapToGlobal( pos ) );

To copy to clipboard, switch view to plain text mode 

Sorry, already tried, but no luck.

Thanx anyway.

[fullmetalcoder]

Sorry, already tried, but no luck.

Then try the contrary :

if pos is in global coordinates :

popupMenu.exec( tablesList->mapFromGlobal(pos) );

popupMenu.exec( tablesList->mapFromGlobal(pos) );

To copy to clipboard, switch view to plain text mode 

if pos is in local coordinates :

popupMenu.exec( tablesList->mapFromGlobal(mapToGlobal(pos)) );

popupMenu.exec( tablesList->mapFromGlobal(mapToGlobal(pos)) );

To copy to clipboard, switch view to plain text mode 

Hope this helps.

[kandalf]

Ok guys, I couldn't make it work. I reimplemented contextMenuEvent method and now it works as it should.

My code ended up like this:

TableListWidget.h
#include <QListWidget>
 
class TableListWidget : public QListWidget
{
	Q_OBJECT
	public:
		TableListWidget(QWidget *);
 
		void contextMenuEvent(QContextMenuEvent *);
};
 
TableListWidget.cpp
...
void TableListWidget::contextMenuEvent( QContextMenuEvent *event)
{
	QMenu popupMenu(this);
		connect( popupMenu.addAction("&Edit Table"), SIGNAL(triggered()), this, SLOT(editActionTriggered()) );
		connect( popupMenu.addAction("&Drop Table"), SIGNAL(triggered()), this, SLOT(dropActionTriggered()) );
	// 	qWarning(tablesList->itemAt(pos)->text().toAscii());
 
// 		popupMenu.exec(pos + QPoint(tablesList->horizontalScrollBar()->value(), tablesList->verticalScrollBar()->value()));
		popupMenu.exec(this->mapToGlobal(event->pos()));
}
...

TableListWidget.h
#include <QListWidget>
		
class TableListWidget : public QListWidget
{
	Q_OBJECT
	public:
		TableListWidget(QWidget *);
		
		void contextMenuEvent(QContextMenuEvent *);
};

TableListWidget.cpp
...
void TableListWidget::contextMenuEvent( QContextMenuEvent *event)
{
	QMenu popupMenu(this);
		connect( popupMenu.addAction("&Edit Table"), SIGNAL(triggered()), this, SLOT(editActionTriggered()) );
		connect( popupMenu.addAction("&Drop Table"), SIGNAL(triggered()), this, SLOT(dropActionTriggered()) );
	// 	qWarning(tablesList->itemAt(pos)->text().toAscii());
		
// 		popupMenu.exec(pos + QPoint(tablesList->horizontalScrollBar()->value(), tablesList->verticalScrollBar()->value()));
		popupMenu.exec(this->mapToGlobal(event->pos()));
}
...

To copy to clipboard, switch view to plain text mode 

Thanx everybody for the answers anyway.

Cheers!

[jpn]

My code ended up like this:

popupMenu.exec(this->mapToGlobal(event->pos()));

popupMenu.exec(this->mapToGlobal(event->pos()));

To copy to clipboard, switch view to plain text mode 

By the way, QContextMenuEvent already contains a global position of the request so it could be even:

popupMenu.exec(event->globalPos());

popupMenu.exec(event->globalPos());

To copy to clipboard, switch view to plain text mode 

[kandalf]

popupMenu.exec(event->globalPos());

popupMenu.exec(event->globalPos());

To copy to clipboard, switch view to plain text mode 

Thanx! I saw this method in docs and I thought I tried it and did not work. But, actually, I didn't, because it DOES work.

Thanx again.
Cheers.