Merge lp:~kissiel/checkbox/fix-1492314-scrolling-test-desc into lp:checkbox

Proposed by Maciej Kisielewski
Status: Merged
Approved by: Zygmunt Krynicki
Approved revision: 4012
Merged at revision: 4012
Proposed branch: lp:~kissiel/checkbox/fix-1492314-scrolling-test-desc
Merge into: lp:checkbox
Diff against target: 16 lines (+2/-1)
1 file modified
checkbox-touch/components/TestPageBody.qml (+2/-1)
To merge this branch: bzr merge lp:~kissiel/checkbox/fix-1492314-scrolling-test-desc
Reviewer Review Type Date Requested Status
Zygmunt Krynicki (community) Approve
Review via email: mp+271335@code.launchpad.net

Description of the change

This MR fixes scrolling test description bug

197e0ee checkbox-touch: make test description really flickable

To post a comment you must log in.
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

+1, nice

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'checkbox-touch/components/TestPageBody.qml'
2--- checkbox-touch/components/TestPageBody.qml 2015-09-15 10:44:09 +0000
3+++ checkbox-touch/components/TestPageBody.qml 2015-09-16 15:45:02 +0000
4@@ -64,10 +64,11 @@
5 Flickable {
6 Layout.fillWidth: true
7 Layout.fillHeight: fullHeightBody
8- contentHeight: childrenRect.height
9+ contentHeight: bodyLabel.contentHeight
10 flickableDirection: Flickable.VerticalFlick
11 clip: true
12 Label {
13+ id: bodyLabel
14 fontSize: "medium"
15 anchors.fill: parent
16 wrapMode: Text.WrapAtWordBoundaryOrAnywhere

Subscribers

People subscribed via source and target branches