1 files changed, 5 insertions, 11 deletions

diff --git a/apps/menu.c b/apps/menu.c
index f3355278f9..e359f75799 100644
--- a/apps/menu.c
+++ b/apps/menu.c
@@ -55,8 +55,6 @@ static void menu_draw(int m)
          i++) {
         lcd_puts(1, i-menus[m].top, menus[m].items[i].desc);
     }
-
-    lcd_puts(0, menus[m].cursor - menus[m].top, "-");
     lcd_update();
 }
 
@@ -72,7 +70,7 @@ static void put_cursor(int m, int target)
         menus[m].top--;
         menu_draw(m);
     }
-    else if ( target > MENU_LINES-1 ) {
+    else if ( target-menus[m].top > MENU_LINES-1 ) {
         menus[m].top++;
         menu_draw(m);
     }
@@ -110,6 +108,7 @@ void menu_exit(int m)
 void menu_run(int m)
 {
     menu_draw(m);
+    lcd_puts(0, menus[m].cursor - menus[m].top, "-");
     
     while(1) {
         switch( button_get(true) ) {
@@ -118,10 +117,7 @@ void menu_run(int m)
 #else
             case BUTTON_LEFT:
 #endif
-                if (menus[m].cursor == 0) {
-                    /* wrap around to menu bottom */
-                    put_cursor(m, menus[m].itemcount-1);
-                } else {
+                if (menus[m].cursor) {
                     /* move up */
                     put_cursor(m, menus[m].cursor-1);
                 }
@@ -132,10 +128,7 @@ void menu_run(int m)
 #else
             case BUTTON_RIGHT:
 #endif
-                if (menus[m].cursor == menus[m].itemcount-1) {
-                    /* wrap around to menu top */
-                    put_cursor(m, 0);
-                } else {
+                if (menus[m].cursor < menus[m].itemcount-1) {
                     /* move down */
                     put_cursor(m, menus[m].cursor+1);
                 }
@@ -152,6 +145,7 @@ void menu_run(int m)
             
                 /* Return to previous display state */
                 menu_draw(m);
+                lcd_puts(0, menus[m].cursor - menus[m].top, "-");
                 break;
 
 #ifdef HAVE_RECORDER_KEYPAD