Text from a QTableView's cell shows through inline QComboBox - OSX
-
@SGaist Sure.
class ComboBoxDelegate : public QStyledItemDelegate { Q_OBJECT public: explicit ComboBoxDelegate(QObject* parent); ~ComboBoxDelegate() override = default; QWidget* createEditor(QWidget *parent, const QStyleOptionViewItem &option, const QModelIndex &index) const override; void setEditorData(QWidget *editor, const QModelIndex &index) const override; void setModelData(QWidget *editor, QAbstractItemModel *model, const QModelIndex &index) const override; }; ComboBoxDelegate::ComboBoxDelegate(QObject* parent) : QStyledItemDelegate(parent) { } QWidget* ComboBoxDelegate::createEditor(QWidget* parent, const QStyleOptionViewItem& option, const QModelIndex& index) const { QComboBox* comboBox = new QComboBox(parent); comboBox->setModel(new ValueListModel(index)); return comboBox; } void ComboBoxDelegate::setEditorData(QWidget* editor, const QModelIndex& index) const { qobject_cast<QComboBox*>(editor)->setCurrentText(index.data(Qt::EditRole).toString()); } void ComboBoxDelegate::setModelData(QWidget* editor, QAbstractItemModel* model, const QModelIndex& index) const { model->setData(index, qobject_cast<QComboBox*>(editor)->currentText()); }Thank you for looking into this.
@rpadrela I feel there is some thing wrong here(not sure)
{ qobject_cast<QComboBox*>(editor)->setCurrentText(index.data(Qt::EditRole).toString()); }Why are you setting the combo box text here?
Is it editable comboBox?
Set the comboBox setCurrentIndex here. If comboBox list of elements is set through model. -
@rpadrela I feel there is some thing wrong here(not sure)
{ qobject_cast<QComboBox*>(editor)->setCurrentText(index.data(Qt::EditRole).toString()); }Why are you setting the combo box text here?
Is it editable comboBox?
Set the comboBox setCurrentIndex here. If comboBox list of elements is set through model.@nagesh The current code is correct.
from https://doc.qt.io/qt-5/qcombobox.html#currentText-prop
The setter setCurrentText() simply calls setEditText() if the combo box is editable. Otherwise, if there is a matching text in the list, currentIndex is set to the corresponding index.
-
@rpadrela I feel there is some thing wrong here(not sure)
{ qobject_cast<QComboBox*>(editor)->setCurrentText(index.data(Qt::EditRole).toString()); }Why are you setting the combo box text here?
Is it editable comboBox?
Set the comboBox setCurrentIndex here. If comboBox list of elements is set through model. -
@nagesh QComboBox is not editable.
Could this be an issue currently affecting Mac OS X only?
@rpadrela said in Text from a QTableView's cell shows through inline QComboBox - OSX:
Could this be an issue currently affecting Mac OS X only?
On windows, both the windows style and Fusion work correctly with this delegate.
Could you try adding
QApplication::setStyle(QStyleFactory::create("Fusion"));after you create theQApplicationin yourmain()? and see if you still have the problem on Fusion? -
Did you test with 6.1 ?
From the looks of it's indeed macOS specific. IIRC there have been several glitches in the rendering of widgets with item views. -
@rpadrela said in Text from a QTableView's cell shows through inline QComboBox - OSX:
Could this be an issue currently affecting Mac OS X only?
On windows, both the windows style and Fusion work correctly with this delegate.
Could you try adding
QApplication::setStyle(QStyleFactory::create("Fusion"));after you create theQApplicationin yourmain()? and see if you still have the problem on Fusion?
