Skip to content
This repository was archived by the owner on Jan 24, 2023. It is now read-only.

Commit c8fee28

Browse files
committed
D.R.Y. Don't Repeat Yourself
1 parent bd22bdc commit c8fee28

File tree

2 files changed

+5
-16
lines changed

2 files changed

+5
-16
lines changed

UI/QuickMenu/ReMenuCategory.cs

Lines changed: 3 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -185,17 +185,10 @@ public ReMenuButton AddSpacer(Sprite sprite = null) {
185185
return spacer;
186186
}
187187

188-
public ReMenuToggle AddToggle(string text, string tooltip, Action<bool> onToggle, bool defaultValue = false)
189-
{
190-
var toggle = new ReMenuToggle(text, tooltip, onToggle, _buttonContainer.RectTransform, defaultValue);
191-
return toggle;
192-
}
193-
188+
public ReMenuToggle AddToggle(string text, string tooltip, Action<bool> onToggle, bool defaultValue = false)
189+
=> AddToggle(text, tooltip, onToggle, defaultValue, null, null);
194190
public ReMenuToggle AddToggle(string text, string tooltip, ConfigValue<bool> configValue)
195-
{
196-
var toggle = new ReMenuToggle(text, tooltip, configValue.SetValue, _buttonContainer.RectTransform, configValue);
197-
return toggle;
198-
}
191+
=> AddToggle(text, tooltip, configValue, null, null);
199192
public ReMenuToggle AddToggle(string text, string tooltip, Action<bool> onToggle, bool defaultValue = false, Sprite iconOn = null, Sprite iconOff = null)
200193
{
201194
var toggle = new ReMenuToggle(text, tooltip, onToggle, _buttonContainer.RectTransform, defaultValue, iconOn, iconOff);

UI/QuickMenu/ReMenuPage.cs

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -161,13 +161,9 @@ public ReMenuButton AddSpacer(Sprite sprite = null)
161161
}
162162

163163
public ReMenuToggle AddToggle(string text, string tooltip, Action<bool> onToggle, bool defaultValue = false)
164-
{
165-
return new ReMenuToggle(text, tooltip, onToggle, _container, defaultValue);
166-
}
164+
=> AddToggle(text, tooltip, onToggle, defaultValue, null, null);
167165
public ReMenuToggle AddToggle(string text, string tooltip, ConfigValue<bool> configValue)
168-
{
169-
return new ReMenuToggle(text, tooltip, configValue.SetValue, _container, configValue);
170-
}
166+
=> AddToggle(text, tooltip, configValue, null, null);
171167
public ReMenuToggle AddToggle(string text, string tooltip, Action<bool> onToggle, bool defaultValue = false, Sprite iconOn = null, Sprite iconOff = null)
172168
{
173169
return new ReMenuToggle(text, tooltip, onToggle, _container, defaultValue, iconOn, iconOff);

0 commit comments

Comments
 (0)