Refactor: Moved the progress bar toggle to a separate function

WowRarity · Jul 19, 2023 · d063743 · d063743
1 parent dd6e6ec
commit d063743
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/Core/GUI/AnchoredTrackingBar.lua b/Core/GUI/AnchoredTrackingBar.lua
@@ -91,3 +91,9 @@ function GUI:UpdateBar()
 		self.barGroup:Lock()
 	end
 end
+
+function GUI:ToggleProgressBar()
+	Rarity.db.profile.bar.visible = not Rarity.db.profile.bar.visible
+	Rarity.GUI:UpdateBar()
+	Rarity.GUI:UpdateText()
+end
diff --git a/Core/GUI/DataBrokerDisplay.lua b/Core/GUI/DataBrokerDisplay.lua
@@ -72,10 +72,7 @@ function dataobj:OnClick(button)
 		(self.db.profile.tooltipActivation == CONSTANTS.TOOLTIP.ACTIVATION_METHOD_CLICK and isRightButton)
 		or (self.db.profile.tooltipActivation == CONSTANTS.TOOLTIP.ACTIVATION_METHOD_HOVER and isLeftButton)
 	then
-		-- Toggle progress bar visibility
-		R.db.profile.bar.visible = not R.db.profile.bar.visible
-		Rarity.GUI:UpdateBar()
-		Rarity.GUI:UpdateText()
+		Rarity.GUI:ToggleProgressBar()
 	elseif self.db.profile.tooltipActivation == CONSTANTS.TOOLTIP.ACTIVATION_METHOD_CLICK and isLeftButton then
 		if Rarity.Tooltips:IsTooltipAcquired("RarityTooltip") then
 			Rarity:HideTooltip()