Swap rows QTableWidget

JonB

I'm trying to make it possible to rearrange the rows

bool QAbstractItemModel::moveRow(const QModelIndex &sourceParent, int sourceRow, const QModelIndex &destinationParent, int destinationChild) allows a row to be moved in a model. Assuming the model used by QTableWidget implements this that should allow re-ordering. I don't know whether the fact there are cell widgets would be relevant.

Paddle

@JoeCFD said in Swap rows QTableWidget:

@Paddle For cellWidgets, you can try to swap them.
auto cell_widget = your_table->cellWidget(int row, int column); 

your_table->removeCellWidget(int row, int column);

your_table->setCellWidget(int new_row, int new_column, cell_widget);
Note that cell widget is added on top of item. You can not swap items for cell widgets and they have to be swapped separately, I guess.

Thanks. This doesn't work. Apparantly from what I read, when you call removeCellWidget the widget is destroyed. It seems that ownership of the widget cannot be changed.

JoeCFD

@Paddle Sorry, I missed that one. Maybe you can create a new one and it does not cost anything, right?

Paddle

@JonB said in Swap rows QTableWidget:

@Paddle said in Swap rows QTableWidget:

I'm trying to make it possible to rearrange the rows

bool QAbstractItemModel::moveRow(const QModelIndex &sourceParent, int sourceRow, const QModelIndex &destinationParent, int destinationChild) allows a row to be moved in a model. Assuming the model used by QTableWidget implements this that should allow re-ordering. I don't know whether the fact there are cell widgets would be relevant.

I tried the following but nothing happens. Am I doing something wrong?

QModelIndex sourceParent = ui->workbenchTable->model()->index(rowIndex, 0);
QModelIndex destinationParent = ui->workbenchTable->model()->index(rowIndex + (up ? -1 : 1), 0);
ui->workbenchTable->model()->moveRow(sourceParent, rowIndex, destinationParent, rowIndex + (up ? -1 : 1));

Paddle

@JoeCFD said in Swap rows QTableWidget:

@Paddle Sorry, I missed that one. Maybe you can create a new one and it does not cost anything, right?

No problem of course, your suggestions are very welcome.

I guess I can recreate the widget yes. I'll try that

JoeCFD

@Paddle Just curious: these buttons look the same for all rows. Can you simply ignore their swaps. Like:

for (int i = 0; i < columnCount - 2; i++) {
    ........
}

JonB

@Paddle said in Swap rows QTableWidget:

I tried the following but nothing happens. Am I doing something wrong?

It returns a bool which you can check to see if it worked. Unfortunately I think your code looks correct, so does it return false?. QAbstractItemModel::moveRows() does that and does nothing by default, the derived class has to implement it. If QTableWidget's model does not do that then this is a blind alley.

Paddle

In the end I got it working by creating new widgets and swapping their layout:

auto widget = ui->workbenchTable->cellWidget(rowIndex, 0);
auto widget2 = ui->workbenchTable->cellWidget(rowIndex + (up ? -1 : 1), 0);

auto newWidget = new QWidget(this);
newWidget->setObjectName(widget->objectName());
newWidget->setLayout(widget->layout());

auto newWidget2 = new QWidget(this);
newWidget2->setObjectName(widget2->objectName());
newWidget2->setLayout(widget2->layout());

ui->workbenchTable->removeCellWidget(rowIndex, 0);
ui->workbenchTable->removeCellWidget(rowIndex + (up ? -1 : 1), 0);

ui->workbenchTable->setCellWidget(rowIndex + (up ? -1 : 1), 0, newWidget);
ui->workbenchTable->setCellWidget(rowIndex                , 0, newWidget2);

Paddle

Sub question: How do you enable drap and drop to move rows ?

I used

ui->workbenchTable->setDragDropMode(QAbstractItemView::InternalMove);
ui->workbenchTable->setSelectionMode(QAbstractItemView::SingleSelection);
ui->workbenchTable->setSelectionBehavior(QAbstractItemView::SelectRows);
ui->workbenchTable->viewport()->setAcceptDrops(true);
ui->workbenchTable->setDropIndicatorShown(true);
ui->workbenchTable->setDragEnabled(true);
ui->workbenchTable->setAcceptDrops(true);
ui->workbenchTable->setDefaultDropAction(Qt::MoveAction);
ui->workbenchTable->setDragDropOverwriteMode(false);

with this the rows can be dragged, meaning clicking them and dragging moves a rectangle around (the widget itself is not visible in the dragged rectangle).

But when I drop nothing happens.

How do I connect the signal or catch the drop event? Thanks

SGaist

Hi,

You should use InternalMove if you plan to only move things around within the same QTableWidget.

That said, you really should consider implementing delegates. As explained in the documentation, setCellWidget should only be used to show some static content.