creating list with variable number of entries

mzimmers

@mrjj Oh, that's a thing of beauty. Thanks for the reminder on that.

And you're correct, adding the parent to the creation of the QLineEdit eliminated the new window.

This issue is mostly fixed now, but there's still a bit of tuning I need to do:

When editing a line, pressing the enter key exits the window. This is logical, but probably not what a lot of users expect. I need to cause it to exit the editor (I think). I think I could fix this by overriding the EditorEvent() method, but I don't know what I'd furnish for the model argument.
It appears that the editor also remains open when the user clicks on a different line. Basically, the same issue as above (I think).

Thanks again...

mrjj

@mzimmers
Hi
I don't recall pressing enter would do anything but finish editing so not sure all code is as it should be.
Can you check your code against
https://stackoverflow.com/questions/26614678/validating-user-input-in-a-qtableview
As this is the same you want to do, using a Validator.
It don't matters its other type of view. The delegate is the same.

mzimmers

My code had some minor differences, but I changed them to match the example, and the behavior is the same.

I did realize something, though -- I still had my itemClicked() signal calling a slot that used openPersistentEditor(). I'm guessing that this wasn't what I wanted. But when I disabled this, I get no editor at all. Here's where I use the delegate:

    ICdelegate *icDelegate = new ICdelegate;
    ui->listWidgetServers->setItemDelegate(icDelegate);

Does this look OK to you?
EDIT:
PS: I didn't mention that I took SGaist's suggestion above to mean that the only method I overrode was createEditor() -- is this correct?
UPDATE:
here's my slot for when the item is double-clicked:

connect(ui->listWidgetServers, &QListWidget::itemDoubleClicked, this, &Informacast::editItem);
void Informacast::editItem(QListWidgetItem *item)
{
    Qt::ItemFlags flags;
    flags = item->flags();
    flags |= Qt::ItemIsEditable;
    item->setFlags(Qt::ItemIsEditable);
    ui->listWidgetServers->editItem(item);
}

Still not editable, though the coloring changes:

I feel that I'm close to getting this to work, but am overlooking something.

SGaist

AFAIR, items are editable by defaults.

Did you set the edit triggers ?

mzimmers

@SGaist I hate to disagree with you (mostly because you're always right), but here's my slot:

void Informacast::editItem(QListWidgetItem *item)
{
    Qt::ItemFlags flags;
    flags = item->flags();
    if ((flags & Qt::ItemIsEditable) == 0)
    {
        flags |= Qt::ItemIsEditable;
    }
    item->setFlags(Qt::ItemIsEditable);
    ui->listWidgetServers->setEditTriggers(QAbstractItemView::DoubleClicked);
    ui->listWidgetServers->editItem(item);
}

When I step through this in the debugger, I hit the line where I assign the editable flag.

To answer your question: no, I hadn't set the edit triggers. But now I am, and the behavior is still the same. In looking at the EditTrigger enums, I don't see another one that I need to set; am I overlooking something?

EDIT: from a debugging session, it appears that the DoubleClicked was already set.

JonB

@mzimmers said in creating list with variable number of entries:

item->setFlags(Qt::ItemIsEditable);

This resets all your flags, to editable only. Yet you earlier picked up the existing flags. I have a feeling you intended

item->setFlags(flags);

to enable editability? I have no idea whether this is relevant to your issue, it's just a code observation.

mzimmers

@JonB ding ding ding we have a winner! Good catch there. I was going to ask whether the setFlags worked that way, but I got distracted by all my other mistakes.

Now, the item is editable, but...my validator doesn't seem to be taking effect. I think I know why, but I'm not sure what to do about it. Here's my class:

class ICdelegate : public QStyledItemDelegate
{
    Q_OBJECT
private:
public:
    explicit ICdelegate(QObject *parent = nullptr) :
        QStyledItemDelegate(parent)
    {
    }
    QLineEdit *createEditor(QWidget *parent, const QStyleOptionViewItem &option,
                          const QModelIndex &index) const override
    {
        // some junk code to disable compiler warnings about unused variables.
        const QModelIndex xxx(index);
        if (xxx == index)
        {
            ;
        }
        const QStyleOptionViewItem yyy(option);

        QLineEdit *qle;
        qle = new QLineEdit(parent);
        qle->setValidator(new IpValidator);

        return (qle);
    }
};

Am I doing something wrong with qle? I tried making it a member variable, but got a compiler error.

Thanks...

SGaist

Try with:


qle->setValidator(new IpValidator(qle));

mzimmers

@SGaist another winner! The validator is now invoked. So, do I understand this to mean that the call to setValidator isn't sufficient to associate the QLineEdit with its validator; the validator uses its parent as the basis for what to operate on?

As a side benefit, pressing "enter" no longer dismisses the dialog. I think this problem is solved (but I'm not going to so mark it quite yet).

Thanks to everyone who helped with this.

JonB

@mzimmers
I'm waiting for The Master's answer on this too :) I looked up the docs and I do not see where it indicates this is required, and OK your validator may leak but I don't see it will die...