Adding an editor to the already existing list of QItemEditorFactory

[talk2amulya]

thats a very good catch! but i dont know how you are able to edit both data types when commenting setDefaultFactory()...anyways, couple of ways to resolve your issue:

1. using delegate: dont set this factory as default, just call setItemEditorFactory(QItemEditorFactory *) on QStyledItemDelegate, Qt's right hand for drawing views. thus when QVariant::Type is Color, it will create your custom editor, for all other QVariant types, it will use default factory.

2. derive from QItemEditorFactory, implement createEditor(..) in a way that if type is Color, it will use ColorEditor, otherwise it will call default factory's createEditor(..). next, set this new class as the default editor factory.

i think second one will work better for you, i.e. without interfering with any delegate.

[schall_l]

Thank you talk2amulya, the second one did effectively worked for me.

Here is the code if someone is interested in:

class IdentifiersEditor : public QItemEditorFactory
{
public:
  inline IdentifiersEditor() {}
  QWidget *createEditor(QVariant::Type type, QWidget *parent) const;
};

class IdentifiersEditor : public QItemEditorFactory
{
public:
  inline IdentifiersEditor() {}
  QWidget *createEditor(QVariant::Type type, QWidget *parent) const;
};

To copy to clipboard, switch view to plain text mode 

QWidget *
IdentifiersEditor::createEditor(QVariant::Type type, QWidget *parent) const {
  switch (type) {
    case QVariant::Color: {
      ColorListEditor *cb = new ColorListEditor(parent);
      cb->setFrame(false);
      return cb; 
    }
    default:
      return QItemEditorFactory::createEditor(type, parent);
      break;
  }
  return 0;
}

QWidget *
IdentifiersEditor::createEditor(QVariant::Type type, QWidget *parent) const {
  switch (type) {
    case QVariant::Color: {
      ColorListEditor *cb = new ColorListEditor(parent);
      cb->setFrame(false);
      return cb; 
    }
    default:
      return QItemEditorFactory::createEditor(type, parent);
      break;
  }
  return 0;
}

To copy to clipboard, switch view to plain text mode 

[aamer4yu]

I have a doubt. You will still need to set your factory somewhere right ?
Also in the example in post #1, you create a QItemEditorFactory and thats why you dont get the default editors with it.

Why cant you obtain the factory from item delegate and register you editor ??
QItemDelegate::itemEditorFactory and QStyledItemDelegate::itemEditorFactory

Wont it be easier ?