From 118902ca1b5b4dbdaf87626faa415a2b2e59475b Mon Sep 17 00:00:00 2001 From: LIBERGOD <37589902+libergod@users.noreply.github.com> Date: Thu, 15 Jan 2026 05:21:16 -0800 Subject: [PATCH] fix: deactivate draggable scrollbar skills in old protocol (#1598) --- modules/game_skills/skills.lua | 16 ++++++++++++++++ modules/game_skills/skills.otui | 2 +- 2 files changed, 17 insertions(+), 1 deletion(-) diff --git a/modules/game_skills/skills.lua b/modules/game_skills/skills.lua index 5071ea25b..9684adeff 100644 --- a/modules/game_skills/skills.lua +++ b/modules/game_skills/skills.lua @@ -301,6 +301,22 @@ local function hideOldClientStats() setSkillGroupVisibility('GameAdditionalSkills', features.additionalSkills) setSkillGroupVisibility('GameForgeSkillStats1332', features.forgeSkills and version >= 1332) setSkillGroupVisibility('GameForgeSkillStats', features.forgeSkills) + -- For very old clients (before 1098) the skills list fits without a slider. + -- Keep the scrollbar track visible but hide the draggable slider so it's not interactive. + if version < 1098 then + local scroll = skillsWindow:recursiveGetChildById('miniwindowScrollBar') + if scroll then + local slider = scroll:getChildById('sliderButton') + if slider then + slider:setVisible(false) + end + end + -- Also hide the offence info separator for old clients + local sep = skillsWindow:recursiveGetChildById('separadorOnOffenceInfoChange') + if sep then + sep:setVisible(false) + end + end end local function hideMenuOptionsForOldClients(menu) diff --git a/modules/game_skills/skills.otui b/modules/game_skills/skills.otui index 471d65b61..4445f3bd9 100644 --- a/modules/game_skills/skills.otui +++ b/modules/game_skills/skills.otui @@ -286,10 +286,10 @@ MiniWindow image-source: /images/icons/icon_fishing image-size: 9 9 SkillPercentPanel + margin-bottom: 5 HorizontalSeparator id: separadorOnOffenceInfoChange - margin-top: 5 margin-bottom: 5 SmallSkillButton