← Back to team overview

ubuntu-touch-coreapps-reviewers team mailing list archive

[Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/pdf-presentation-hide-mouse-cursor into lp:ubuntu-docviewer-app

 

Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-docviewer-app/pdf-presentation-hide-mouse-cursor into lp:ubuntu-docviewer-app.

Commit message:
[PdfPresentation] Hide mouse cursor when there's no on-going mouse event

Requested reviews:
  Jenkins Bot (ubuntu-core-apps-jenkins-bot): continuous-integration
  Ubuntu Document Viewer Developers (ubuntu-docviewer-dev)

For more details, see:
https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/pdf-presentation-hide-mouse-cursor/+merge/283719

[PdfPresentation] Hide mouse cursor when there's no on-going mouse event
-- 
Your team Ubuntu Document Viewer Developers is requested to review the proposed merge of lp:~verzegnassi-stefano/ubuntu-docviewer-app/pdf-presentation-hide-mouse-cursor into lp:ubuntu-docviewer-app.
=== modified file 'src/app/qml/pdfView/PdfPresentation.qml'
--- src/app/qml/pdfView/PdfPresentation.qml	2016-01-14 13:48:06 +0000
+++ src/app/qml/pdfView/PdfPresentation.qml	2016-01-23 13:06:54 +0000
@@ -85,6 +85,19 @@
         MouseArea {
             anchors.fill: parent
             onDoubleClicked: pdfPage.header.visible = !pdfPage.header.visible
+
+            // Hide mouse curson when there's no on-going mouse event
+            hoverEnabled: true
+            // Flickable can steal mouse handling from MouseArea,
+            // so check for any on-going dragging too
+            cursorShape: (showCursorTimer.running || pdfView.dragging) ? Qt.ArrowCursor : Qt.BlankCursor
+            onPositionChanged: {
+                showCursorTimer.restart()
+            }
+            Timer {
+                id: showCursorTimer
+                interval: 500
+            }
         }
     }