Text Button causing game to freeze

Help and Feedback Scripting Support
#1

I’m not sure if this is the correct category but I think it has some relation about this bugs/glitch.

So basically I have this TextButton UIS that when clicked it will shows a frame, or we can call it Inventory uis. The errors/glitch I’m encountering is that it cause the game (Client) to freeze a bit or sometimes can be long when i clicked it but the codes are workings properly.

Is there any reason why this is happening and how would i fix it?

1 Like
#2

Can’t say anything until we see the code. Show code for the click

#3

I could be because you’re exhausting the script maybe add task.wait(). Also I would like to see the code to find the issue like what RafehPvP said.

#4

well its very long but i’ll send some of the codes soon

#5

We just need the part where you handle the interaction with the Text button.

#6 
-- Activating the GUI
local open = 0
		InventoryButton.Activated:Connect(function()
			if open == 0 then
				open = 1
				selectedTower = tower.Name
				UpdateSkins()
				if tower.InvisibleForever == false then
					gui.Inventory.Towers.TowersScroll:FindFirstChild(newButtonTower.ViewportFrame.TowerName.Text).Visible = true
					gui.Inventory.Towers.TowersScroll:FindFirstChild(newButtonTower.ViewportFrame.TowerName.Text).Visible = true
				end
			end
		end)
#7

Show us UpdateSkins now, that’s probably the culprit (My Guess is marketplaceservice) let’s see it

1 Like
#8

Let me guess, Its because i looped through many things?

	for i, skin in pairs(skinshop) do
		local oldButton = gui.Inventory.Skins.TowersScroll:FindFirstChild(skin.Name)

		if oldButton then
			oldButton:Destroy()
		end

		local clonedTower = game.ReplicatedStorage.Skins[skin.Name]:Clone()
		local newButtonTower =  gui.Inventory.Skins.TowersScroll.Template:Clone()
		newButtonTower.Name = skin.Name
		newButtonTower.ViewportFrame.TowerName.Text = skin.Display
		newButtonTower.Parent = gui.Inventory.Skins.TowersScroll
		newButtonTower.LayoutOrder = skin.Price	
		newButtonTower.ViewportFrame.TowerName.TextColor3 = skin.RarityColor

		clonedTower.Parent = newButtonTower.ViewportFrame.WorldModel
		clonedTower.PrimaryPart = clonedTower.HumanoidRootPart
		local TowerAnim = clonedTower.Humanoid.Animator:LoadAnimation(clonedTower.Animations.Idle)
		TowerAnim:Play()
#9

Hmm I’m still not entirely sure BUT do these things please, Make sure you do all the changes you want to the properties Before you parent these things. For Example, you’re parenting the newButtonTower before you’re setting the LayoutOrder and Setting the TextColor, same for the PrimaryPart for clonedTower. Do that and let me know fi that helped.

1 Like