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

consider moving dragThreshold property from QQPointerHandler to QQMultiPointHandler; qreal?

XMLWordPrintable

    • Icon: Task Task
    • Resolution: Unresolved
    • Icon: P3: Somewhat important P3: Somewhat important
    • None
    • 6.5.0 FF
    • None

      Only DragHandler and PinchHandler need it so far. But the property is in the base class, since https://codereview.qt-project.org/c/qt/qtdeclarative/+/267206 DragHandler used to be a single-point handler, but not by that time.

      There could be a single-point handler that needs a threshold though; so this is questionable.

      Also the type is int... I forgot if there's a good reason API-wise, but it's an oddity when most distance-measuring properties are qreal. Implementation-wise, it's a qint16 for better packing; that's nearly a moot point, and the "packing" is currently mixing signed and unsigned types, which won't work anyway.

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

            srutledg Shawn Rutledge
            srutledg Shawn Rutledge
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:

                There are no open Gerrit changes