Fixed the bug in the scrolling limitation that made it possible to scroll outside of the map in the south and east.

2 files changed, 15 insertions, 6 deletions

diff --git a/ChangeLog b/ChangeLog
index 880a3b9d..1ef0afbe 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,4 +1,9 @@
-2007-02-04  Bjørn Lindeijer  <bjorn@lindeijer.nl>
+2007-02-10  Philipp Sehmisch  <tmw@crushnet.org>
+
+	* src/gui/viewport.cpp: Fixed the bug in the scrolling limitation that made
+	it possible to scroll outside of the map in the south and east.
+
+2007-02-04  Bjørn Lindeijer  <bjorn@lindeijer.nl>
 
 	* src/gui/menuwindow.cpp: Fixed a small glitch when dragging the menu
 	window.
diff --git a/src/gui/viewport.cpp b/src/gui/viewport.cpp
index 90f3536c..098b913a 100644
--- a/src/gui/viewport.cpp
+++ b/src/gui/viewport.cpp
@@ -134,18 +134,22 @@ Viewport::draw(gcn::Graphics *gcnGraphics)
         mViewY = player_y;
     };
 
-    if (mMap) {
+    // Don't move camera so that the end of the map is on screen
+    int viewXmax = ((mMap->getWidth() - 1) * 32) - graphics->getWidth();
+    int viewYmax = ((mMap->getHeight() - 1) * 32) - graphics->getHeight();
+    if (mMap)
+    {
         if (mViewX < 0) {
             mViewX = 0;
         }
         if (mViewY < 0) {
             mViewY = 0;
         }
-        if (mViewX > (mMap->getWidth() - midTileX) * 32) {
-            mViewX = (mMap->getWidth() - midTileX) * 32;
+        if (mViewX > viewXmax) {
+            mViewX = viewXmax;
         }
-        if (mViewY > (mMap->getHeight() - midTileY) * 32) {
-            mViewY = (mMap->getHeight() - midTileY) * 32;
+        if (mViewY > viewYmax) {
+            mViewY = viewYmax;
         }
     }