Skip to content
This repository was archived by the owner on Dec 5, 2024. It is now read-only.

Commit de19150

Browse files
Using isBusy to denote loading
1 parent 0b12f7f commit de19150

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

src/UnityExtension/Assets/Editor/GitHub.Unity/UI/PUblishView.cs

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@ class PublishView : Subview
2222
[SerializeField] private bool togglePrivate;
2323

2424
[NonSerialized] private IApiClient client;
25-
[NonSerialized] private bool isLoading;
2625
[NonSerialized] private bool isBusy;
2726
[NonSerialized] private string error;
2827

@@ -65,7 +64,7 @@ private void PopulateView()
6564
{
6665
Logger.Trace("GetCurrentUser");
6766

68-
isLoading = true;
67+
isBusy = true;
6968

7069
Client.GetCurrentUser(user => {
7170
if (user == null)
@@ -96,7 +95,7 @@ private void PopulateView()
9695
owners = owners.Union(organizationLogins).ToArray();
9796
});
9897
}).Finally(task => {
99-
isLoading = false;
98+
isBusy = false;
10099
});
101100
}
102101
else
@@ -133,7 +132,7 @@ public override void OnGUI()
133132

134133
GUILayout.Space(Styles.PublishViewSpacingHeight);
135134

136-
EditorGUI.BeginDisabledGroup(isLoading || isBusy);
135+
EditorGUI.BeginDisabledGroup(isBusy);
137136
{
138137
GUILayout.BeginHorizontal();
139138
{

0 commit comments

Comments
 (0)