QAbstractItemDelegate and editing

goetz

Connect an appropriate signal of the widget containing your item view to a slot endEdit() in you item view subclass:

@
void MyItemViewSubclass::endEdit()
{
QModelIndex index = currentIndex();
currentChanged( index, index );
}
@

This simulates clicking on an item and thus ends the editing process of the combobox.

You should call this slot too, if you are going to commit the data via some save button or the like. It works well and we use it all the time.

Anticross

This delegate is using by more then one item View sub class. I can't even imagine how many times it will be used in project in feature and by how many people, because it not so small. So I need some local solution in my delegate or in AItemDelegate class. Is there any other ideas ?

goetz

The delegate does not catch widget state changes. So I see no better solution. You might create a central subclass of the needed item view/item widgets and implement endEdit there, to ease the use of the method.

Anticross

Thanks for answer. So this will take a little bit more work than I expect.

Anticross

And one more question: which signal I need to connect ? For example in QTreeView ? I can't find appropriate signal. Or maybe it must be my own signal emitted from one of delegate functions ?

goetz

I mean some signals or events from the containing widget (hide, close, etc.). maybe focusOut on the item view too. Also, if you have, for example, a save button, it is likely that it is connected to some slot, you should call endEdit there too.

Anticross

There is one more simple solution that I find. In my delegate I need to add connect @ public: void setEditorData(QWidget * editor, const QModelIndex & index) const {
QComboBox * cb = static_cast<QComboBox *>(editor);
QString s = index.data(Qt::DisplayRole).toString();
int i = cb->findText(s);
cb->setCurrentIndex(i);

                connect(cb, SIGNAL(currentIndexChanged(int)), this, SLOT(onCurrentIndexChanged(int)));
            }@

And add a slot onCurrentIndexChanged : @public slots: void onCurrentIndexChanged(int index) {
QComboBox * cb = static_cast<QComboBox *>(sender());
emit commitData(cb);
emit closeEditor(cb);
}@
So it solved locally and works pretty well.

goetz

Be aware that this will work with your combo boxes, but not with default generated line edits or the like!

brianz

I had the exact same issue and the currentChanged trick worked very nicely.

Thanks for the tip!

maximus

Just want to thanks for this trick, I modified it a little bit so it works with every QWidget editor

1- In createEditor, connect a custom signal in your QWidget Editor to a custom slot in the Delegate
@QWidget *IntervalDelegate::createEditor(QWidget *parent, const QStyleOptionViewItem &option, const QModelIndex &index) const

{
else if (index.column() == 3 ) {
CadenceEditor *editor = new CadenceEditor(parent);
connect (editor, SIGNAL(endEdit()), this, SLOT(closeWidgetEditor()) );
return editor;
}@

2- close the editor in the custom delegate slot :
@void IntervalDelegate::closeWidgetEditor() {

qDebug() << "CLOSE EDITOR NOW!";
QWidget *editor = qobject_cast<QWidget*>(sender());

emit commitData(editor);
emit closeEditor(editor);

}@

maximus

I can now close the editor with a custom "OK" button, show in this image:
https://www.dropbox.com/s/95agb2c38f9p354/customEditors.png

Problem is when I click the OK button, the click get propaged under it and it open another editor, how can I stop the click? I already have setAttribute(Qt::WA_NoMousePropagation); on my custom QWidget editor..

Thanks

maximus

I think it is the default behavior in a QTableView, when a cell editing is completed, the editor automatically move to the adjacent(right) cell.

Kind of like excel.
I need to find how to turn off this feature now :)