QListWidget + Delete Key

[bpetty]

I was wondering what the best way to delete a QListWidget item when you press the delete key.

Should I write my own class that inherits QListWidget and overwrite the keyPressEvent() method to capture the delete key? There must be a better design.

Thanks!
Brandon P.

[jpn]

Either that or use an event filter.

[jacek]

Or QShortcut.

[bpetty]

Jacek, I like the idea of using QShortcut.

I noticed the example from the documentation:

shortcut = QShortcut(QKeySequence(tr("Ctrl+O", "File|Open")), parent);

shortcut = QShortcut(QKeySequence(tr("Ctrl+O", "File|Open")), parent);

To copy to clipboard, switch view to plain text mode 

So instead of Ctrl+O I would use Qt::Key_Delete...
and instead of parent, I would use the name of my QListWidget object...
I would also most likely have to change the context to that of a Qt::WidgetShortcut

Is this correct? I guess I don't understand then what it returns and where the correct place to put this QShortcut would be.

[jpn]

Let's assume you have a list widget in a main window. Then, a natural place to create the shortcut would be in the constructor of the main window (or a function called from there). In addition, you'll need a slot where to react to the shortcut event.

class MainWindow...
{
...
private slots:
    void deleteItem();
 
private:
    QListWidget* listWidget;
};

class MainWindow...
{
...
private slots:
    void deleteItem();

private:
    QListWidget* listWidget;
};

To copy to clipboard, switch view to plain text mode 

MainWindow::MainWindow(...) : ...
{
    // create the shortcut after the list widget has been created
 
    // option A (pressing DEL anywhere in the main window activates the slot)
    new QShortcut(QKeySequence(Qt::Key_Delete), this, SLOT(deleteItem()));
 
    // option B (pressing DEL activates the slots only when list widget has focus)
    QShortcut* shortcut = new QShortcut(QKeySequence(Qt::Key_Delete), listWidget);
    connect(shortcut, SIGNAL(activated()), this, SLOT(deleteItem()));
}
 
void MainWindow::deleteItem()
{
    delete listWidget->currentItem();
}

MainWindow::MainWindow(...) : ...
{
    // create the shortcut after the list widget has been created

    // option A (pressing DEL anywhere in the main window activates the slot)
    new QShortcut(QKeySequence(Qt::Key_Delete), this, SLOT(deleteItem()));

    // option B (pressing DEL activates the slots only when list widget has focus)
    QShortcut* shortcut = new QShortcut(QKeySequence(Qt::Key_Delete), listWidget);
    connect(shortcut, SIGNAL(activated()), this, SLOT(deleteItem()));
}

void MainWindow::deleteItem()
{
    delete listWidget->currentItem();
}

To copy to clipboard, switch view to plain text mode 

[bpetty]

Thanks a lot JPN. I guess I took it too literally. They said "shortcut = QShorcut(..)" and I was atleast expecting to see a new. I was also thinking that the new "shortcut" pointer gets destoyed... but I guess Qt knows what I want.

Thanks a lot.