Skip to content

Commit f7cda6c

Browse files
Fix ambiguous call to DrawHeaderToggle in Core (#2363)
1 parent a64dd14 commit f7cda6c

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

com.unity.render-pipelines.core/Editor/CoreEditorUtils.cs

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -458,10 +458,9 @@ public static bool DrawHeaderToggle(string title, SerializedProperty group, Seri
458458
/// <param name="contextAction">The context action</param>
459459
/// <param name="hasMoreOptions">Delegate saying if we have MoreOptions</param>
460460
/// <param name="toggleMoreOptions">Callback called when the MoreOptions is toggled</param>
461-
/// <param name="documentationURL">Documentation URL</param>
462461
/// <returns>return the state of the foldout header</returns>
463-
public static bool DrawHeaderToggle(string title, SerializedProperty group, SerializedProperty activeField, Action<Vector2> contextAction = null, Func<bool> hasMoreOptions = null, Action toggleMoreOptions = null, string documentationURL = null)
464-
=> DrawHeaderToggle(EditorGUIUtility.TrTextContent(title), group, activeField, contextAction, hasMoreOptions, toggleMoreOptions, documentationURL);
462+
public static bool DrawHeaderToggle(GUIContent title, SerializedProperty group, SerializedProperty activeField, Action<Vector2> contextAction = null, Func<bool> hasMoreOptions = null, Action toggleMoreOptions = null)
463+
=> DrawHeaderToggle(title, group, activeField, contextAction, hasMoreOptions, toggleMoreOptions, null);
465464

466465
/// <summary>Draw a header toggle like in Volumes</summary>
467466
/// <param name="title"> The title of the header </param>
@@ -470,9 +469,11 @@ public static bool DrawHeaderToggle(string title, SerializedProperty group, Seri
470469
/// <param name="contextAction">The context action</param>
471470
/// <param name="hasMoreOptions">Delegate saying if we have MoreOptions</param>
472471
/// <param name="toggleMoreOptions">Callback called when the MoreOptions is toggled</param>
472+
/// <param name="documentationURL">Documentation URL</param>
473473
/// <returns>return the state of the foldout header</returns>
474-
public static bool DrawHeaderToggle(GUIContent title, SerializedProperty group, SerializedProperty activeField, Action<Vector2> contextAction = null, Func<bool> hasMoreOptions = null, Action toggleMoreOptions = null)
475-
=> DrawHeaderToggle(title, group, activeField, contextAction, hasMoreOptions, toggleMoreOptions, null);
474+
public static bool DrawHeaderToggle(string title, SerializedProperty group, SerializedProperty activeField, Action<Vector2> contextAction, Func<bool> hasMoreOptions, Action toggleMoreOptions, string documentationURL)
475+
=> DrawHeaderToggle(title, group, activeField, contextAction, hasMoreOptions, toggleMoreOptions, documentationURL);
476+
476477
/// <summary>Draw a header toggle like in Volumes</summary>
477478
/// <param name="title"> The title of the header </param>
478479
/// <param name="group"> The group of the header </param>
@@ -482,7 +483,7 @@ public static bool DrawHeaderToggle(GUIContent title, SerializedProperty group,
482483
/// <param name="toggleMoreOptions">Callback called when the MoreOptions is toggled</param>
483484
/// <param name="documentationURL">Documentation URL</param>
484485
/// <returns>return the state of the foldout header</returns>
485-
public static bool DrawHeaderToggle(GUIContent title, SerializedProperty group, SerializedProperty activeField, Action<Vector2> contextAction = null, Func<bool> hasMoreOptions = null, Action toggleMoreOptions = null, string documentationURL = null)
486+
public static bool DrawHeaderToggle(GUIContent title, SerializedProperty group, SerializedProperty activeField, Action<Vector2> contextAction, Func<bool> hasMoreOptions, Action toggleMoreOptions, string documentationURL)
486487
{
487488
var backgroundRect = GUILayoutUtility.GetRect(1f, 17f);
488489

0 commit comments

Comments
 (0)