QTreeWidget Drag and Drop

[jpn]

QAbstractItemModel::dropMimeData() is called by QAbstractItemView::dropEvent() so reimplementing dropEvent() without calling the base class implementation prevents dropMimeData() from being called. Furthermore, reimplementing other drag and drop related event handlers, again, without calling the base class implementation like in the code above, prevents QAbstractItemView DND from functioning properly. Take a look at src/gui/itemviews/qabstractitemview.cpp and see what QAbstractItemView::dragEnterEvent() and QAbstractItemView::dragMoveEvent() do. Writing:

void myQTTreeWidget::dragEnterEvent(QDragEnterEvent *e)
{
    e->accept();
}
 
void myQTTreeWidget::dragMoveEvent(QDragMoveEvent *e)
{
    e->accept();       
}

void myQTTreeWidget::dragEnterEvent(QDragEnterEvent *e)
{
    e->accept();
}
 
void myQTTreeWidget::dragMoveEvent(QDragMoveEvent *e)
{
    e->accept();       
}

To copy to clipboard, switch view to plain text mode 

overrides all that functionality with.. nothing.

[ram136682]

Greetings ,

In the two functions below override the original events by accepting the event but what next.

It will accept the event and no action is performed. ( Try emitting a signal within these functions and perform the required actions accordingly )

void myQTTreeWidget::dragEnterEvent(QDragEnterEvent *e)
void myQTTreeWidget::dragMoveEvent(QDragMoveEvent *e)

Regards
-Ram

[wysota]

Is it a continuation of the same problem by a different forum user, a completely separate issue or an answer to the original problem? I got completely lost here...

[tommydent]

Thank you very much for your help.

Even if it's a very silly question, I have to ask it (sorry 'bout that):

Whats the "correct" way to implement the dragEnterEvent and dragMoveEvent functions - is this the way?

void myQTTreeWidget::dragEnterEvent(QDragEnterEvent *e)
{
    // always call base class implementation 
    QTreeWidget::dragEnterEvent(e);
 
    if ( isDragEnterPossible(e) )
    {
        e->accept();
    }
    else
    {
        e->ignore();
    }
}
 
void myQTTreeWidget::dragMoveEvent(QDragMoveEvent *e)
{
    // always call base class implementation 
    QTreeWidget::dragMoveEvent(e);
 
    if ( isDragMovePossible(e) )
    {
        e->accept();
    }
    else
    {
        e->ignore();
    }
}

void myQTTreeWidget::dragEnterEvent(QDragEnterEvent *e)
{
    // always call base class implementation 
    QTreeWidget::dragEnterEvent(e);

    if ( isDragEnterPossible(e) )
    {
        e->accept();
    }
    else
    {
        e->ignore();
    }
}
       
void myQTTreeWidget::dragMoveEvent(QDragMoveEvent *e)
{
    // always call base class implementation 
    QTreeWidget::dragMoveEvent(e);

    if ( isDragMovePossible(e) )
    {
        e->accept();
    }
    else
    {
        e->ignore();
    }
}

To copy to clipboard, switch view to plain text mode 

isDragMovePossible and isDragEnterPossible are member functions, returning a bool value and telling us if dropping is possible.

[wysota]

The question is why would you want to reimplement them? What's wrong with the default implementation?

[tommydent]

Thank you for your answer.

So "what is this all about":

I have an application with lots of different widgets (treeviews, listviews, iconviews .... ). All widgest support dnd (all of them are still old qt3 style, but will/should () be "converted" soon). In my (derived) QTreeView I want do accept the drags from some of the other widgets, but not from all of the other widget. I thought the dragEnterEvent can (has to) be used for defining which events are accepted and which are not:

void myQTListView::dragEnterEvent(QDragEnterEvent *e)
{
	if ( e )
	{
		if ( e->provides("mystuff/mything") )
		{
			e->accept();
		}
		else
		{
			e->ignore();
		}
	}
}

void myQTListView::dragEnterEvent(QDragEnterEvent *e)
{
	if ( e )
	{
		if ( e->provides("mystuff/mything") )
		{
			e->accept();
		}
		else
		{
			e->ignore();
		}
	}
}

To copy to clipboard, switch view to plain text mode 

I also have different items (all derived from QTreeWidgetItem of course) in my treeview. Some of them accept drops and some of them don't. So I use the dragMoveEvent to get the "current dropping target" by calling

void myQTListView::dragMoveEvent(QDragMoveEvent *e)
{
	QTreeWidgetItem *item = itemAt(e->pos());
 
	bool accept = false;
 
	if ( item )
	{
		if ( dynamic_cast<MySpecialListViewItem*>(item) )
		{
			accept = true;
		}
	}
 
	if ( accept )
	{
		e->accept();		
	}
        else
        {
                 e->ignore();
        }
}

void myQTListView::dragMoveEvent(QDragMoveEvent *e)
{
	QTreeWidgetItem *item = itemAt(e->pos());

	bool accept = false;

	if ( item )
	{
		if ( dynamic_cast<MySpecialListViewItem*>(item) )
		{
			accept = true;
		}
	}

	if ( accept )
	{
		e->accept();		
	}
        else
        {
                 e->ignore();
        }
}

To copy to clipboard, switch view to plain text mode 

and check (by dynamic cast) if this (derived) QTreeWidgetItem is one of the items that accept drops.

I use the event->accept() or event->ignore() function to indicate if dropping is possible or not.

ps. Sorry that this has become a beginner tutorial on using drag and drop with QTreeViews, maybe I was running completely in the wrong direction?

[wysota]

But most of what you want is already implemented by default handlers, there is no need to reinvent the wheel. You can tell the widget which types of data it should accept. Take a look at mimeTypes(), mimeData() and dropMimeData().