Fixed #126 (again)

Got burnt by the dodgy way I have implemented page up/down checking for the option KeyboardBrowseLCDScreen.
pi1541 · Nov 18, 2019 · b77fa6f · b77fa6f
1 parent 853e518
commit b77fa6f
Showing 1 changed file with 12 additions and 4 deletions.
diff --git a/src/FileBrowser.cpp b/src/FileBrowser.cpp
@@ -273,8 +273,12 @@ bool FileBrowser::BrowsableListView::CheckBrowseNavigation(bool pageOnly)
 		}
 		else
 		{
-			list->currentIndex = 0;
-			dirty = true;
+			if (!pageOnly)
+			{
+				list->currentIndex = 0;
+				list->SetCurrent();
+				dirty = true;
+			}
 		}
 	}
 	if (inputMappings->BrowseUp())
@@ -292,8 +296,12 @@ bool FileBrowser::BrowsableListView::CheckBrowseNavigation(bool pageOnly)
 		}
 		else
 		{
-			list->currentIndex = list->entries.size() - 1;
-			dirty = true;
+			if (!pageOnly)
+			{
+				list->currentIndex = list->entries.size() - 1;
+				list->SetCurrent();
+				dirty = true;
+			}
 		}
 	}
 	if ((lcdPgUpDown && inputMappings->BrowsePageDownLCD()) || (!lcdPgUpDown && inputMappings->BrowsePageDown()))