From 5da99116ceb60b0c226fee3f56e2484ffefb35b3 Mon Sep 17 00:00:00 2001
From: Gabriel Tofvesson <gabriel@tofvesson.se>
Date: Sat, 12 Oct 2024 04:59:04 +0200
Subject: [PATCH] Fix bounds check on request buttons

---
 gfx/text.lua       | 2 +-
 itemcontroller.lua | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/gfx/text.lua b/gfx/text.lua
index eb992d9..91335bb 100644
--- a/gfx/text.lua
+++ b/gfx/text.lua
@@ -12,7 +12,7 @@ function Text:setText(text)
   local current = self:getText()
   if current ~= text then
     self:setDirty()
-    
+
     local needReload = #current ~= #text
     self.text = text
     if needReload then
diff --git a/itemcontroller.lua b/itemcontroller.lua
index 7e332d8..926c48d 100644
--- a/itemcontroller.lua
+++ b/itemcontroller.lua
@@ -560,7 +560,7 @@ local PAGES = {
 
       dataRequestText:setText(requestText)
       dataDividerPad:setPadding{
-        left = math.ceil((state.width - 1 - (PADDING_RQC_H * 2) - #requestText - #availableText) / 2),
+        left = math.floor((state.width - 1 - (PADDING_RQC_H * 2) - #requestText - #availableText) / 2),
         right = math.floor((state.width - 1 - (PADDING_RQC_H * 2) - #requestText - #availableText) / 2)
       }
       dataAvailableText:setText(availableText)
@@ -572,8 +572,8 @@ local PAGES = {
     local function bindRequestButton(increment)
       local id = increment > 0 and ("+"..tostring(increment)) or tostring(increment)
       paddedRequestCount:findById(id):setOnClick(function(e, x, y, s)
-        local newValue = pageState.request + increment
-        if newValue >= 0 and newValue <= group:getItemCount() then
+        local newValue = math.max(0, math.min(pageState.request + increment, group:getItemCount()))
+        if pageState.request ~= newValue then
           pageState.request = newValue
           updateDisplayState()
         end