Delegate editor sizing problem

[smacchia]

I have have implemented a custom delegate and model for editing all the QTableView cells in a very controlled way. When my delegates create the editor, it doesn't fill up the available space in the cell. Is there something that I need to do to make this happen?

Thanks in advance,
Susan

[marcel]

Reimplement the sizeHint method, and return the cell size.

[smacchia]

How does the delegate find out the size of the cell? It has no direct access to the view that I know of.

[marcel]

Sorry about that... sizeHint is a wrong path.

Try overriding QItemDelegate::updateEditorGeometry(...);
Like this:

[COLOR=#8B0000][FONT='Courier New,courier'][I][/I][/FONT][/COLOR]void MyDelegate::updateEditorGeometry( QWidget* editor, const QStyleOptionViewItem &option, const QModelIndex &index )
{
    editor->setGeometry( option.rect );
}

[COLOR=#8B0000][FONT='Courier New,courier'][I][/I][/FONT][/COLOR]void MyDelegate::updateEditorGeometry( QWidget* editor, const QStyleOptionViewItem &option, const QModelIndex &index )
{
    editor->setGeometry( option.rect );
}

To copy to clipboard, switch view to plain text mode 

Using option.rect should resize your editor to fit perfectly into the cell.

Regards

[smacchia]

Just what I was looking for! I'll give this a try.

--Susan

[smacchia]

Unfortunately, this method never gets called!!! I am not sure why. I have an example MVC implementation from a training class I attended and the delgates never implemented this method and the editor (spinbox or combo) was always sized correctly... Ideas?

[marcel]

No idea why it isn't called, but take a look at the Spin Box delegate example in the Qt Demos ( Item Views section ).

It does something similar to what you want.

[smacchia]

I think its not called because in the Qt source for QItemDelegete, it isn't virtual!! (it is supposed to be). Stepping through the Qt code for creating the delegate editor, it invokes QItemDelegate::updateEditorGeometry, not my derivation! I'll look at the example that you site, but I think this is the culprit. My example from the class is pretty straight forward too and doesn't event reimplement that method...

[marcel]

void QItemDelegate::updateEditorGeometry ( QWidget * editor, const QStyleOptionViewItem & option, const QModelIndex & index ) const [virtual] Updates the editor for the item specified by index according to the style option given.
Reimplemented from QAbstractItemDelegate.

This is the member description from QItemDelegate. As you can see it is virtual. Also this member is public and const. Perhaps you didn't declare it as public in your class?

[marcel]

Actually you're right. I just took a look at QItemDelegate.h and the member is not virtual.
But it is in QAbstractItemDelegate. Couldn't you derive your delegate from QAbstractItemDelegate instead?

[jacek]

I think its not called because in the Qt source for QItemDelegete, it isn't virtual!!

It is virtual. It doesn't have to be marked as virtual in QItemDelegate definition, since it was declared as virtual in QAbstractItemDelegate.

Also this member is public and const. Perhaps you didn't declare it as public in your class?

Also make sure you didn't forget about "const". It's quite common mistake.

[smacchia]

Oh duh - it was the "const". Thanks! It works great now.

[maximAL]

i got a similar problem with delegates and a QTreeView, but i'm using editing widgets that contain other widgets, eg. a QWidget containing a QLineEdit.
first, i tried adding a layout to the container widget and adding sub-widgets to it, but the result was that the item didn't seem to be editable anymore at all (or the widget has hone somewhere / was zero-sized or whatever).
i tried overriding the setGeometry function for my subclasses but when the delegate calls setGeometry, allways the standard QWidget function is called.
i declared setGeometry as:

virtual void setGeometry(const QRect &r)

virtual void setGeometry(const QRect &r)

To copy to clipboard, switch view to plain text mode