From 294f54d9afcceb384725610559c452d70cf71dec Mon Sep 17 00:00:00 2001
From: ysakamoto <yus.sakamoto@gmail.com>
Date: Sun, 12 Apr 2015 18:03:16 -0500
Subject: [PATCH] fixed cursor position going over the number of windows

---
 rtv/page.py | 26 +++++++++++++++++++-------
 1 file changed, 19 insertions(+), 7 deletions(-)

diff --git a/rtv/page.py b/rtv/page.py
index 11a7274..8ba4276 100644
--- a/rtv/page.py
+++ b/rtv/page.py
@@ -85,7 +85,7 @@ class Navigator(object):
         return valid, redraw
 
     def move_page(self, direction, n_windows):
-        """Move the page and cursor down (positive direction) or up (negative
+        """Move the page down (positive direction) or up (negative
         direction)"""
 
         # top of submission page: act as normal move
@@ -101,18 +101,26 @@ class Navigator(object):
                 # not submission mode: starting index is 0
                 if not self._is_valid(self.absolute_index):
                     self.page_index = 0
-                
                 valid = True
             else:
-                self.page_index += n_windows*direction
-                valid = self._is_valid(self.absolute_index)
-                if not valid:
-                    self.page_index -= n_windows*direction
+                valid = False
+                adj = 0
+                # check if reached the bottom
+                while not valid:
+                    n_move = n_windows-adj
+                    if n_move == 0:
+                        break
+
+                    self.page_index += n_move*direction
+                    valid = self._is_valid(self.absolute_index)
+                    if not valid:
+                        self.page_index -= n_move*direction
+                        adj += 1
 
             redraw = True
 
         return valid, redraw
-            
+
     def flip(self, n_windows):
         "Flip the orientation of the page"
 
@@ -432,6 +440,10 @@ class BasePage(object):
         if self.nav.absolute_index < 0:
             return
 
+        # Don't allow the cursor to go over the subwindow number
+        if self.nav.cursor_index >= len(self._subwindows):
+            self.nav.cursor_index = len(self._subwindows)-1
+
         window, attr = self._subwindows[self.nav.cursor_index]
         if attr is not None:
             attribute |= attr