From afe93ae1ad97369be78e227b323ed89c2d6f8713 Mon Sep 17 00:00:00 2001
From: nightfox <>
Date: Sun, 10 Sep 2017 03:38:49 +0000
Subject: [PATCH] Bug fix: Updated GetTopItemIdxToTopOfLastPage() and
 SetTopItemIdxToTopOfLastPage() to return/set the top item index to 0 if there
 is only one page of menu items

---
 exec/load/DDLightbarMenu.js | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/exec/load/DDLightbarMenu.js b/exec/load/DDLightbarMenu.js
index 7e3cce6a4e..a3419fd8d2 100644
--- a/exec/load/DDLightbarMenu.js
+++ b/exec/load/DDLightbarMenu.js
@@ -1131,7 +1131,10 @@ function DDLightbarMenu_GetTopItemIdxToTopOfLastPage()
 	var numItemsPerPage = this.size.height;
 	if (this.borderEnabled)
 		numItemsPerPage -= 2;
-	return this.items.length - numItemsPerPage;
+	var topItemIndex = this.items.length - numItemsPerPage;
+	if (topItemIndex < 0)
+		topItemIndex = 0;
+	return topItemIndex;
 }
 
 // Sets the top item index to the top item of the last page of items
@@ -1141,6 +1144,8 @@ function DDLightbarMenu_SetTopItemIdxToTopOfLastPage()
 	if (this.borderEnabled)
 		numItemsPerPage -= 2;
 	this.topItemIdx = this.items.length - numItemsPerPage;
+	if (this.topItemIdx < 0)
+		this.topItemIdx = 0;
 }
 
 // Inputs a keypress from the user and handles some ESC-based
-- 
GitLab