Flickable's widthRatio and heightRatio are wrong when the content is smaller than the view and has topMargin/leftMargin
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Canonical System Image |
Fix Released
|
Medium
|
Zoltan Balogh | ||
qtdeclarative-opensource-src (Ubuntu) |
Fix Released
|
Medium
|
Unassigned | ||
qtdeclarative-opensource-src (Ubuntu RTM) |
Fix Released
|
Medium
|
Unassigned |
Bug Description
UPSTREAM BUG: https:/
Description (copied from the upstream bug I created)
According to Flickable's documentation, widthRatio and heightRatio define
"the percentage of the full view currently visible, scaled to 0.0 - 1.0".
At the moment, defining leftMargin so that
leftMargin+
causes widthRatio to be wrongly evaluated to something != 1, whereas it should be 1, because Flickable's defaults at -leftMargin
(see https:/
and when contentX is -leftMargin, the whole content fits inside the view.
As a consequence of that, in the current implementation you can also scroll the item from the testcase left and right, which shouldn't be possible (because item+margin are still smaller than the view).
=======
Additional Ubuntu-specific info:
This is currently blocking the correct implementation of the new scrollbars inside TextFields (ping kalikiana or me (faenil) on IRC for more info)
=======
import QtQuick 2.0
Flickable {
id: flickable
width: 200
height: 200
contentWidth: item.width
contentHeight: item.height
topMargin: 20
leftMargin: 40
Component.
Connections {
target: flickable.
}
Rectangle {
id: item
width: 100
height: 100
color: "black"
}
}
=======
How to reproduce:
1) qmlscene testcase.qml
2) watch the console output
Actual result: widthRatio is != 1
Expected result: widthRatio == 1
=======
Fix: I worked on a fix which is being reviewed by upstream ---> https:/
description: | updated |
description: | updated |
description: | updated |
Changed in qtdeclarative-opensource-src (Ubuntu RTM): | |
status: | New → In Progress |
Changed in qtdeclarative-opensource-src (Ubuntu): | |
importance: | Undecided → Medium |
Changed in qtdeclarative-opensource-src (Ubuntu RTM): | |
importance: | Undecided → Medium |
Changed in canonical-devices-system-image: | |
importance: | Undecided → Medium |
assignee: | nobody → Zoltan Balogh (bzoltan) |
milestone: | none → 13 |
status: | New → In Progress |
Changed in canonical-devices-system-image: | |
status: | In Progress → Fix Committed |
Changed in canonical-devices-system-image: | |
status: | Fix Committed → Fix Released |
Status changed to 'Confirmed' because the bug affects multiple users.