From 8489a06edff6a0aec59d7edf0dd7dfa1079d3935 Mon Sep 17 00:00:00 2001 From: LukieD4 Date: Fri, 6 Mar 2026 15:56:38 +0000 Subject: [PATCH] Fix page variable increment/decrement with a controller from 5 (TabSpec::rows) to 1 --- .../Common/UI/IUIScene_CreativeMenu.cpp | 56 +++++++++---------- 1 file changed, 28 insertions(+), 28 deletions(-) diff --git a/Minecraft.Client/Common/UI/IUIScene_CreativeMenu.cpp b/Minecraft.Client/Common/UI/IUIScene_CreativeMenu.cpp index 0d3f7dace..4f6d8fe16 100644 --- a/Minecraft.Client/Common/UI/IUIScene_CreativeMenu.cpp +++ b/Minecraft.Client/Common/UI/IUIScene_CreativeMenu.cpp @@ -1099,42 +1099,42 @@ void IUIScene_CreativeMenu::handleAdditionalKeyPress(int iAction) break; case ACTION_MENU_OTHER_STICK_DOWN: { - int pageStep = TabSpec::rows; + int pageStep = 1; // override: use 1 so thumbstick increment/decrement behaves alike the mouse wheel implementation. #ifdef _WINDOWS64 - if (g_KBMInput.WasMouseWheelConsumed()) - { - pageStep = 1; - } + if (g_KBMInput.WasMouseWheelConsumed()) + { + pageStep = 1; + } #endif - m_tabPage[m_curTab] += pageStep; - if(m_tabPage[m_curTab] >= specs[m_curTab]->getPageCount()) - { - m_tabPage[m_curTab] = specs[m_curTab]->getPageCount() - 1; - } - else - { - switchTab(m_curTab); - } + m_tabPage[m_curTab] += pageStep; + if(m_tabPage[m_curTab] >= specs[m_curTab]->getPageCount()) + { + m_tabPage[m_curTab] = specs[m_curTab]->getPageCount() - 1; + } + else + { + switchTab(m_curTab); + } } break; case ACTION_MENU_OTHER_STICK_UP: { - int pageStep = TabSpec::rows; + int pageStep = 1; // override: use 1 so thumbstick increment/decrement behaves alike the mouse wheel implementation. #ifdef _WINDOWS64 - if (g_KBMInput.WasMouseWheelConsumed()) - { - pageStep = 1; - } + if (g_KBMInput.WasMouseWheelConsumed()) + { + pageStep = 1; + } #endif - m_tabPage[m_curTab] -= pageStep; - if(m_tabPage[m_curTab] < 0) - { - m_tabPage[m_curTab] = 0; - } - else - { - switchTab(m_curTab); - } + m_tabPage[m_curTab] -= pageStep; + if(m_tabPage[m_curTab] < 0) + { + m_tabPage[m_curTab] = 0; + } + else + { + switchTab(m_curTab); + } } break; }