From cd3f9336abe5ac9c99c817bd4abcf37c8c940bde Mon Sep 17 00:00:00 2001
From: Filippos Karapetis
Date: Sat, 31 Oct 2009 12:17:38 +0000
Subject: Cursor positioning now takes screen scaling into account

svn-id: r45556
---
 engines/sci/gui/gui_cursor.cpp | 23 ++++++++++++++++++-----
 1 file changed, 18 insertions(+), 5 deletions(-)

diff --git a/engines/sci/gui/gui_cursor.cpp b/engines/sci/gui/gui_cursor.cpp
index db1ae1ca6d..e24e6e36b5 100644
--- a/engines/sci/gui/gui_cursor.cpp
+++ b/engines/sci/gui/gui_cursor.cpp
@@ -39,8 +39,8 @@ namespace Sci {
 SciGuiCursor::SciGuiCursor(ResourceManager *resMan, SciGuiPalette *palette, SciGuiScreen *screen)
 	: _resMan(resMan), _palette(palette), _screen(screen) {
 
-	setPosition(Common::Point(160, 150));		// TODO: how is that different in 640x400 games?
-	setMoveZone(Common::Rect(0, 0, 320, 200));	// TODO: hires games
+	setPosition(Common::Point(160, 150));
+	setMoveZone(Common::Rect(0, 0, _screen->_displayWidth, _screen->_displayHeight));
 }
 
 SciGuiCursor::~SciGuiCursor() {
@@ -157,11 +157,24 @@ void SciGuiCursor::setView(GuiResourceId viewNum, int loopNum, int celNum, Commo
 }
 
 void SciGuiCursor::setPosition(Common::Point pos) {
-	g_system->warpMouse(pos.x, pos.y);
+	int scaleFactor = _screen->getScaleFactor();
+
+	if (scaleFactor == 1)
+		g_system->warpMouse(pos.x, pos.y);
+	else
+		g_system->warpMouse(pos.x * scaleFactor, pos.y * scaleFactor);
 }
 
 Common::Point SciGuiCursor::getPosition() {
-	return g_system->getEventManager()->getMousePos();
+	int scaleFactor = _screen->getScaleFactor();
+	Common::Point pos = g_system->getEventManager()->getMousePos();
+
+	if (scaleFactor != 1) {
+		pos.x /= scaleFactor;
+		pos.y /= scaleFactor;
+	}
+
+	return pos;
 }
 
 void SciGuiCursor::refreshPosition() {
@@ -186,7 +199,7 @@ void SciGuiCursor::refreshPosition() {
 
 	// FIXME: Do this only when mouse is grabbed?
 	if (clipped)
-		g_system->warpMouse(mousePoint.x, mousePoint.y);
+		setPosition(mousePoint);
 }
 
 } // End of namespace Sci
-- 
cgit v1.2.3