From 390bf78eccf29e41a4c7915e256920ff2bf58b07 Mon Sep 17 00:00:00 2001
From: Aleix Pol <aleixpol@kde.org>
Date: Fri, 5 Aug 2011 23:47:35 +0200
Subject: [PATCH] Change mouse cursor when changing the current tool

Update the cursors when changing the current mode. Addresses the problems in the bug report.

- Use a opened/closed hand for browsing.
- Use cross for selection.

BUG: 271816
REVIEW: 102221
---
 ui/pageview.cpp | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/ui/pageview.cpp b/ui/pageview.cpp
index f103b2fb5..688604f51 100644
--- a/ui/pageview.cpp
+++ b/ui/pageview.cpp
@@ -1520,7 +1520,7 @@ void PageView::mouseMoveEvent( QMouseEvent * e )
                 // drag page
                 else if ( !d->mouseGrabPos.isNull() )
                 {
-                    setCursor( Qt::SizeAllCursor );
+                    setCursor( Qt::ClosedHandCursor );
 
                     QPoint mousePos = e->globalPos();
                     QPoint delta = d->mouseGrabPos - mousePos;
@@ -2838,6 +2838,8 @@ void PageView::updateCursor( const QPoint &p )
         // if over a ObjectRect (of type Link) change cursor to hand
         if ( d->mouseMode == MouseTextSelect )
             setCursor( Qt::IBeamCursor );
+        else if ( d->mouseMode == MouseSelect )
+            setCursor( Qt::CrossCursor );
         else if ( d->mouseAnn )
             setCursor( Qt::ClosedHandCursor );
         else
@@ -2858,6 +2860,10 @@ void PageView::updateCursor( const QPoint &p )
                 {
                     setCursor( Qt::OpenHandCursor );
                 }
+                else if ( d->mouseMode == MouseNormal )
+                {
+                    setCursor( Qt::OpenHandCursor );
+                }
                 else
                 {
                     setCursor( Qt::ArrowCursor );