Skip to content
  • Categories
  • Recent
  • Tags
  • Popular
  • Users
  • Groups
  • Search
  • Get Qt Extensions
  • Unsolved
Collapse
Brand Logo
  1. Home
  2. General talk
  3. Qt 6
  4. Text from a QTableView's cell shows through inline QComboBox - OSX
Forum Updated to NodeBB v4.3 + New Features

Text from a QTableView's cell shows through inline QComboBox - OSX

Scheduled Pinned Locked Moved Unsolved Qt 6
17 Posts 4 Posters 1.6k Views 2 Watching
  • Oldest to Newest
  • Newest to Oldest
  • Most Votes
Reply
  • Reply as topic
Log in to reply
This topic has been deleted. Only users with topic management privileges can see it.
  • VRoninV VRonin

    Try adding:

    public:
    void updateEditorGeometry(QWidget *editor, const QStyleOptionViewItem &option, const QModelIndex &) const override{
    editor->setGeometry(option.rect);
    }
    
    R Offline
    R Offline
    rpadrela
    wrote on last edited by
    #7

    @VRonin Thanks for the suggestion. Unfortunately, it makes no difference.

    1 Reply Last reply
    0
    • VRoninV Offline
      VRoninV Offline
      VRonin
      wrote on last edited by
      #8

      this is very odd... do you have a custom stylesheet applyed?

      "La mort n'est rien, mais vivre vaincu et sans gloire, c'est mourir tous les jours"
      ~Napoleon Bonaparte

      On a crusade to banish setIndexWidget() from the holy land of Qt

      R 1 Reply Last reply
      0
      • VRoninV VRonin

        this is very odd... do you have a custom stylesheet applyed?

        R Offline
        R Offline
        rpadrela
        wrote on last edited by
        #9

        @VRonin Nope

        1 Reply Last reply
        0
        • R rpadrela

          @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.

          nageshN Offline
          nageshN Offline
          nagesh
          wrote on last edited by
          #10

          @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.

          VRoninV R 2 Replies Last reply
          0
          • nageshN nagesh

            @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.

            VRoninV Offline
            VRoninV Offline
            VRonin
            wrote on last edited by VRonin
            #11

            @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.

            "La mort n'est rien, mais vivre vaincu et sans gloire, c'est mourir tous les jours"
            ~Napoleon Bonaparte

            On a crusade to banish setIndexWidget() from the holy land of Qt

            1 Reply Last reply
            0
            • nageshN nagesh

              @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.

              R Offline
              R Offline
              rpadrela
              wrote on last edited by
              #12

              @nagesh QComboBox is not editable.

              Could this be an issue currently affecting Mac OS X only?

              VRoninV 1 Reply Last reply
              0
              • R rpadrela

                @nagesh QComboBox is not editable.

                Could this be an issue currently affecting Mac OS X only?

                VRoninV Offline
                VRoninV Offline
                VRonin
                wrote on last edited by VRonin
                #13

                @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 the QApplication in your main()? and see if you still have the problem on Fusion?

                "La mort n'est rien, mais vivre vaincu et sans gloire, c'est mourir tous les jours"
                ~Napoleon Bonaparte

                On a crusade to banish setIndexWidget() from the holy land of Qt

                R 1 Reply Last reply
                1
                • SGaistS Offline
                  SGaistS Offline
                  SGaist
                  Lifetime Qt Champion
                  wrote on last edited by
                  #14

                  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.

                  Interested in AI ? www.idiap.ch
                  Please read the Qt Code of Conduct - https://forum.qt.io/topic/113070/qt-code-of-conduct

                  1 Reply Last reply
                  0
                  • VRoninV VRonin

                    @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 the QApplication in your main()? and see if you still have the problem on Fusion?

                    R Offline
                    R Offline
                    rpadrela
                    wrote on last edited by
                    #15

                    @VRonin

                    This

                    QApplication::setStyle(QStyleFactory::create("Fusion"));
                    

                    fixes it.

                    I haven't looked into styles yet but it seems like I should. :D

                    Thank you for your help.

                    1 Reply Last reply
                    0
                    • VRoninV Offline
                      VRoninV Offline
                      VRonin
                      wrote on last edited by
                      #16

                      This means it's a bug in MacOS style, might be worth reporting it

                      "La mort n'est rien, mais vivre vaincu et sans gloire, c'est mourir tous les jours"
                      ~Napoleon Bonaparte

                      On a crusade to banish setIndexWidget() from the holy land of Qt

                      R 1 Reply Last reply
                      0
                      • VRoninV VRonin

                        This means it's a bug in MacOS style, might be worth reporting it

                        R Offline
                        R Offline
                        rpadrela
                        wrote on last edited by
                        #17

                        @VRonin just reported it here https://bugreports.qt.io/browse/QTBUG-93731

                        Thank you everyone for your help.

                        1 Reply Last reply
                        0

                        • Login

                        • Login or register to search.
                        • First post
                          Last post
                        0
                        • Categories
                        • Recent
                        • Tags
                        • Popular
                        • Users
                        • Groups
                        • Search
                        • Get Qt Extensions
                        • Unsolved