Merge pull request #68 from CapnRat/ui/utility-authentication-window

Additional cleanup to the authentication window.

Author: StanleyGoldman

src/UnityExtension/Assets/Editor/GitHub.Unity/Misc/Styles.cs

@@ -266,7 +266,7 @@ public static GUIStyle ErrorLabel
         {
           errorLabel = new GUIStyle(EditorStyles.label);
           errorLabel.name = "ErrorLabel";
-
+          errorLabel.wordWrap = true;
           errorLabel.normal.textColor = Color.red;
         }
 

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

@@ -7,15 +7,14 @@ namespace GitHub.Unity
     [Serializable]
     class AuthenticationView : Subview
     {
-        const string serverLabel = "Server";
         const string usernameLabel = "Username";
         const string passwordLabel = "Password";
-        const string twofaLabel = "Authentication code";
+        const string twofaLabel = "2FA Code";
         const string loginButton = "Sign in";
         const string backButton = "Back";
         const string authTitle = "Sign in to GitHub";
-        const string twofaTitle = "Two-factor authentication";
-        const string twofaDescription = "Open the two-factor authentication app on your device to view your authentication code and verify your identity.";
+        const string twofaTitle = "Two-Factor Authentication";
+        const string twofaDescription = "Open the two-factor authentication app on your device to view your 2FA code and verify your identity.";
         const string twofaButton = "Verify";
 
         [SerializeField] private Vector2 scroll;
@@ -76,6 +75,8 @@ public override void OnGUI()
         {
             HandleEnterPressed();
 
+            EditorGUIUtility.labelWidth = 90f;
+
             scroll = GUILayout.BeginScrollView(scroll);
             {
                 Rect authHeader = EditorGUILayout.BeginHorizontal(Styles.AuthHeaderBoxStyle);
@@ -114,7 +115,6 @@ public override void OnGUI()
                 }
 
                 GUILayout.EndVertical();
-                GUILayout.Space(Styles.BaseSpacing);
             }
             GUILayout.EndScrollView();
         }
@@ -135,15 +135,15 @@ private void OnGUILogin()
             GUILayout.BeginHorizontal();
             {
                 if (busy) GUI.enabled = false;
-                username = EditorGUILayout.TextField("Username" ,username, Styles.TextFieldStyle);
+                username = EditorGUILayout.TextField(usernameLabel ,username, Styles.TextFieldStyle);
                 GUI.enabled = true;
             }
             GUILayout.EndHorizontal();
             GUILayout.Space(Styles.BaseSpacing);
             GUILayout.BeginHorizontal();
             {
                 if (busy) GUI.enabled = false;
-                password = EditorGUILayout.PasswordField("Password", password, Styles.TextFieldStyle);
+                password = EditorGUILayout.PasswordField(passwordLabel, password, Styles.TextFieldStyle);
                 GUI.enabled = true;
             }
             GUILayout.EndHorizontal();
@@ -196,8 +196,6 @@ private void OnGUI2FA()
                 Redraw();
             }
 
-            GUILayout.Space(Styles.BaseSpacing);
-
             if (GUILayout.Button(twofaButton) || (GUI.enabled && enterPressed))
             {
                 GUI.FocusControl(null);
@@ -209,7 +207,6 @@ private void OnGUI2FA()
             GUI.enabled = true;
             GUILayout.Space(Styles.BaseSpacing);
             GUILayout.EndVertical();
-            GUILayout.Space(Styles.BaseSpacing);
         }
 
         private void DoRequire2fa(string msg)

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

@@ -1,4 +1,4 @@
-using System;
+using System;
 using UnityEditor;
 using UnityEngine;
 
@@ -7,7 +7,7 @@ namespace GitHub.Unity
     [Serializable]
     class AuthenticationWindow : BaseWindow
     {
-        private const string Title = "Sign in";
+        private const string Title = "Authentication";
 
         [SerializeField] private AuthenticationView authView;
 
@@ -19,10 +19,10 @@ public static void Launch()
 
         public static IView Open(Action<bool> onClose = null)
         {
-            AuthenticationWindow authWindow = GetWindow<AuthenticationWindow>();
+            AuthenticationWindow authWindow = GetWindow<AuthenticationWindow>(true);
             if (onClose != null)
                 authWindow.OnClose += onClose;
-            authWindow.minSize = new Vector2(290, 290);
+            authWindow.minSize = authWindow.maxSize = new Vector2(290, 290);
             authWindow.Show();
             return authWindow;
        }