Merge lp:~tpeeters/ubuntu-docviewer-app/bug1577277-workaround into lp:ubuntu-docviewer-app

Proposed by Tim Peeters
Status: Merged
Approved by: Alan Pope 🍺🐧🐱 πŸ¦„
Approved revision: 350
Merged at revision: 350
Proposed branch: lp:~tpeeters/ubuntu-docviewer-app/bug1577277-workaround
Merge into: lp:ubuntu-docviewer-app
Diff against target: 14 lines (+4/-0)
1 file modified
src/app/qml/pdfView/PdfView.qml (+4/-0)
To merge this branch: bzr merge lp:~tpeeters/ubuntu-docviewer-app/bug1577277-workaround
Reviewer Review Type Date Requested Status
Jenkins Bot continuous-integration Approve
Alan Pope 🍺🐧🐱 πŸ¦„ (community) Approve
Review via email: mp+304561@code.launchpad.net

Commit message

Workaround for bug 1577277.

To post a comment you must log in.
Revision history for this message
Alan Pope 🍺🐧🐱 πŸ¦„ (popey) wrote :

WFM.

review: Approve
Revision history for this message
Jenkins Bot (ubuntu-core-apps-jenkins-bot) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/app/qml/pdfView/PdfView.qml'
--- src/app/qml/pdfView/PdfView.qml 2016-03-26 18:44:53 +0000
+++ src/app/qml/pdfView/PdfView.qml 2016-08-31 20:15:12 +0000
@@ -96,6 +96,10 @@
96 Component.onDestruction: window.releaseResources()96 Component.onDestruction: window.releaseResources()
97 }97 }
9898
99 Component.onDestruction: {
100 // workaround for bug #1577277
101 pdfPage.header.flickable = null;
102 }
99 // FIXME: On zooming, keep the same content position.103 // FIXME: On zooming, keep the same content position.
100 PinchArea {104 PinchArea {
101 id: pinchy105 id: pinchy

Subscribers

People subscribed via source and target branches