Is this function for making the players screen change on damage efficient?

Cykkieatric · October 20, 2022, 9:56pm

Heres an image of the code:

Katrist · October 20, 2022, 9:57pm

No, you should be using .HealthChanged for this. Paste your code here and I can fix it.

Cykkieatric · October 20, 2022, 9:58pm

function VignetteDisplay()
	RunService.RenderStepped:Connect(function()
		
		for i, v in ipairs(Display:GetDescendants()) do
			if v:IsA("ImageLabel") then
				v.ImageTransparency = Humanoid.Health / 100
			end
		end
		Lighting.ColorCorrection.Saturation = Humanoid.Health / 100 - 1
	end)
end

Katrist · October 20, 2022, 10:00pm

Change it to this:

function VignetteDisplay()
	Humanoid.HealthChanged:Connect(function(health)
		for i, descendant in ipairs(Display:GetDescendants()) do
			if descendant:IsA("ImageLabel") then
				descendant.ImageTransparency = health / 100
			end
		end
		
		Lighting.ColorCorrection.Saturation = health / 100 - 1
	end)
end