fix: Added more dynamic bank trimming

Zoinkwiz · Nov 27, 2024 · a8b5b31 · a8b5b31
1 parent 43542e9
commit a8b5b31
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/src/main/java/com/questhelper/bank/banktab/QuestBankTab.java b/src/main/java/com/questhelper/bank/banktab/QuestBankTab.java
@@ -110,6 +110,8 @@ public class QuestBankTab
 
 	private final HashMap<Widget, BankTabItem> widgetItems = new HashMap<>();
 
+	private int originalContainerChildren = -1;
+
 	public void startUp()
 	{
 		if (questHelper.getSelectedQuest() != null)
@@ -327,11 +329,13 @@ private void resetWidgets()
 
 	private void removeAddedWidgets()
 	{
+		if (originalContainerChildren == -1) return;
+
 		if (addedWidgets.isEmpty()) return;
 		Widget parent = addedWidgets.get(0).getParent();
 		if (parent == null) return;
 
-		parent.setChildren(Arrays.copyOf(parent.getChildren(), 1248));
+		parent.setChildren(Arrays.copyOf(parent.getChildren(), originalContainerChildren));
 		parent.revalidate();
 
 		addedWidgets.clear();
@@ -363,6 +367,8 @@ public void onScriptPostFired(ScriptPostFired event)
 			return;
 		}
 
+		removeAddedWidgets();
+
 		if (!questBankTabInterface.isQuestTabActive())
 		{
 			return;
@@ -373,8 +379,8 @@ public void onScriptPostFired(ScriptPostFired event)
 		{
 			return;
 		}
-
-		removeAddedWidgets();
+		Widget[] children = itemContainer.getChildren();
+		if (children != null && originalContainerChildren == -1) originalContainerChildren = children.length;
 
 		Widget[] containerChildren = itemContainer.getDynamicChildren();