Changing an image using QStyledItemDelegate in TableView

[VRonin]

Looks like you forgot some #includes, added them to my original post

[gabor53]

@VRonin
I have in the .h file:

virtual void setEditorData(QWidget *editor, const QModelIndex &index);

In the .cpp:

void ImageDelegate::setEditorData(QWidget *editor, const QModelIndex &index)
{
    ImagePickButton* imgPick = qobject_cast<ImagePickButton*>(editor);
    Q_ASSERT(imgPick);
    imgPick->setSelectedFile(index.data(Qt::UserRole));
}

It gives me the following error message:
C:\Programming\Projects\Folkfriends_1_0\imagedelegate.cpp:18: error: no matching function for call to 'ImagePickButton::setSelectedFile(QVariant)'
imgPick->setSelectedFile(index.data(Qt::UserRole));

                                                  ^

C:\Programming\Projects\Folkfriends_1_0\imagedelegate.h:4: In file included from ..\Folkfriends_1_0\imagedelegate.h:4:0,
C:\Programming\Projects\Folkfriends_1_0\imagedelegate.cpp:1: from ..\Folkfriends_1_0\imagedelegate.cpp:1:
C:\Programming\Projects\Folkfriends_1_0\imagepickbutton.h:18: candidate: void ImagePickButton::setSelectedFile(const QString&)
void setSelectedFile(const QString& val);
^
C:\Programming\Projects\Folkfriends_1_0\imagepickbutton.h:18: note: no known conversion for argument 1 from 'QVariant' to 'const QString&'

What should I change? Everything else builds correctly.

[p3c0]

@gabor53 setSelectedFile expects a QString parameter so convert the QVariant returned by data to QString.

[VRonin]

easy fix imgPick->setSelectedFile(index.data(Qt::UserRole).toString());

[gabor53]

@VRonin
Thank you. This part works now. I added

ImagePickButton *myImagePickButton = new ImagePickButton(this);
myImagePickButton->show();

to mydeelgate.cpp createEditor.

When the user clicks on the image now it creates the button saying "Select Image". But this button is not on the view; it is separated from it and appears in the top left corner of the screen. How can I add it to the view so the button is on the actual image the user is trying to change? Thank you.

[VRonin]

do not call show() just return that widget from that method. Again I'd advise you just to use the code I posted above even for the delegate

[gabor53]

@VRonin
I definitely want to use the code you posted. Should I call the method returning the widget from myDelegate.cpp or from fixdb.cpp where the model is set up? Currently I have this:

 ui->tableView_Fix->setItemDelegate (new myDelegate);

This declares that the delegate will be myDelegate from now on for everything. Can this work somehow ?

    ui->tableView_Fix->setItemDelegateForColumn(3, new ImageDelegate);
    ui->tableView_Fix->setItemDelegate (new myDelegate);

Saying that if column 3 is chosen go with ImageDelegate; all other cases myDelegate.
It this is the way, what is missing? Currently it opens a text window in column 4). Thank you.

[VRonin]

do it the other way around and do not leak memory:

    ui->tableView_Fix->setItemDelegate (new myDelegate(this));
ui->tableView_Fix->setItemDelegateForColumn(3, new ImageDelegate(this));

[gabor53]

This post is deleted!

[gabor53]

@VRonin
So far I came across 2 problems.

1. If I add

ui->tableView_Fix->setItemDelegateForColumn(3, new ImageDelegate(this));

to fixdb.cpp , nothing happens, because myDelegate.cpp evaluates which cell the user clicked on. I can add

return new ImagePickButton(parent);

to myDelegate createEditor and it returns the button, but I couldn't figure out how to connect imageDelegate to the existing code. Please advise which is the best solution.

2. The button is about 10px smaller on the top and on the left than the original cell. How can I make it bigger to cover the size of the original cell (originally its 100 x 100). It doesn't respond to any of the size settings.
   Thank you for your help.