Details
-
Bug
-
Resolution: Done
-
P2: Important
-
5.15.10, 6.2.5, 6.4.0 Beta3
-
None
-
5cb3ba93da (qt/qtdeclarative/dev) 33eb0fc79d (qt/qtdeclarative/6.3) 9e2705e027 (qt/qtdeclarative/6.4) 33eb0fc79d (qt/tqtc-qtdeclarative/6.3) 9e2705e027 (qt/tqtc-qtdeclarative/6.4) 3e698f7c9f (qt/tqtc-qtdeclarative/6.2)
Description
In QQuickPinchHandler::handlePointerEventImpl(), we have this in the else block for when the event is not a native gesture:
m_activeScale = qBound(m_minimumScale/m_startScale, m_activeScale, m_maximumScale/m_startScale);
but no similar use of those min/max values when it's a native gesture. So you can scale things infinitely with a touchpad that sends those events, regardless of limits.
Attachments
Issue Links
- is required for
-
QTBUG-104769 pdfviewer example: Pinch zoom on laptop trackpad stops working if zoom way in
- Closed
- relates to
-
QTBUG-108903 [PinchHandler] minimum/maximumScale are not functional
- Closed
For Gerrit Dashboard: QTBUG-106110 | ||||||
---|---|---|---|---|---|---|
# | Subject | Branch | Project | Status | CR | V |
429981,5 | PinchHandler: Enforce min/max scale limits with native gestures | dev | qt/qtdeclarative | Status: MERGED | +2 | 0 |
430194,2 | PinchHandler: Enforce min/max scale limits with native gestures | 6.3 | qt/qtdeclarative | Status: MERGED | +2 | 0 |
430195,2 | PinchHandler: Enforce min/max scale limits with native gestures | 6.4 | qt/qtdeclarative | Status: MERGED | +2 | 0 |
430217,2 | PinchHandler: Enforce min/max scale limits with native gestures | tqtc/lts-6.2 | qt/tqtc-qtdeclarative | Status: MERGED | +2 | 0 |