Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-25091

Qt 5 to-do's in QGraphicsProxyWidget

XMLWordPrintable

      The QGraphicsProxyWidget code contains several to-do comments for Qt 5:

      src/widgets/graphicsview/qgraphicsproxywidget.cpp
      /*!
          \internal
      
          Reimplemented from QGraphicsItemPrivate. ### Qt 5: Move impl to
          reimplementation QGraphicsProxyWidget::inputMethodQuery().
      */
      QVariant QGraphicsProxyWidgetPrivate::inputMethodQueryHelper(Qt::InputMethodQuery query) const
      ...
          case QEvent::InputMethod: {
              // Forward input method events if the focus widget enables
              // input methods.
              // ### Qt 4.5: this code must also go into a reimplementation
              // of inputMethodEvent().
              QWidget *focusWidget = d->widget->focusWidget();
              if (focusWidget && focusWidget->testAttribute(Qt::WA_InputMethodEnabled))
                  QApplication::sendEvent(focusWidget, event);
              break;
          }
      
      src/widgets/graphicsview/qgraphicsproxywidget.h
          // ### Qt 4.5:
          // QVariant inputMethodQuery(Qt::InputMethodQuery query) const;
          // void inputMethodEvent(QInputMethodEvent *event);
      
      src/widgets/graphicsview/qgraphicsproxywidget_p.h
          // ### Qt 5: Remove. Workaround for reimplementation added after Qt 4.4.
          QVariant inputMethodQueryHelper(Qt::InputMethodQuery query) const;
      

      Those comments that can be handled without breaking source-compatibility should be actioned for Qt 5.0.0. Any others should be removed or changed to Qt 6 to-do's.

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

            paeglis Gatis Paeglis
            jasmcdon Jason McDonald (Closed Nokia Identity. Please assign to "macadder" instead) (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Created:
              Updated:
              Resolved:

                There are no open Gerrit changes