Change the way the connect section is shown

So we can start fixing #371 and also show the list of github repos in
the connect area.

Author: shana

src/GitHub.Exports/Services/VSExtensions.cs

@@ -1,4 +1,5 @@
 using GitHub.Services;
+using Microsoft.TeamFoundation.Controls;
 using System;
 using System.Diagnostics;
 
@@ -34,5 +35,10 @@ public static T GetService<T>(this IServiceProvider serviceProvider)
         {
             return (T)serviceProvider.GetService(typeof(T));
         }
+
+        public static ITeamExplorerSection GetSection(this IServiceProvider serviceProvider, Guid section)
+        {
+            return serviceProvider?.GetService<ITeamExplorerPage>()?.GetSection(section);
+        }
     }
 }

src/GitHub.VisualStudio/Base/TeamExplorerInvitationBase.cs

@@ -7,6 +7,8 @@ namespace GitHub.VisualStudio.Base
 {
     public class TeamExplorerInvitationBase : TeamExplorerBase, ITeamExplorerServiceInvitation, INotifyPropertySource
     {
+        public static readonly Guid TeamExplorerInvitationSectionGuid = new Guid("8914ac06-d960-4537-8345-cb13c00378d8");
+
         public virtual void Initialize(IServiceProvider serviceProvider)
         {
             ServiceProvider = serviceProvider;

src/GitHub.VisualStudio/TeamExplorer/Connect/GitHubConnectSection.cs

@@ -1,11 +1,14 @@
-using GitHub.Api;
+using System;
+using GitHub.Api;
 using GitHub.Models;
 using GitHub.Services;
 using GitHub.UI;
+using GitHub.Extensions;
 using GitHub.VisualStudio.Base;
 using GitHub.VisualStudio.UI.Views;
 using Microsoft.TeamFoundation.Controls;
 using System.Collections.Specialized;
+using GitHub.VisualStudio.Helpers;
 
 namespace GitHub.VisualStudio.TeamExplorer.Connect
 {
@@ -21,13 +24,38 @@ protected GitHubConnectContent View
 
         protected IConnection SectionConnection { get; set; }
 
+        bool loggedIn;
+        bool LoggedIn
+        {
+            get { return loggedIn; }
+            set {
+                loggedIn = ShowLogout = value;
+                ShowLogin = !value;
+            }
+        }
+
+        bool showLogin;
+        public bool ShowLogin
+        {
+            get { return showLogin; }
+            set { showLogin = value;  this.RaisePropertyChange(); }
+        }
+
+        bool showLogout;
+        public bool ShowLogout
+        {
+            get { return showLogout; }
+            set { showLogout = value;  this.RaisePropertyChange(); }
+        }
+
         public GitHubConnectSection(ISimpleApiClientFactory apiFactory, ITeamExplorerServiceHolder holder, IConnectionManager manager, int index)
             : base(apiFactory, holder, manager)
         {
             Title = "GitHub";
             IsEnabled = true;
             IsVisible = false;
             IsExpanded = true;
+            LoggedIn = false;
 
             sectionIndex = index;
 
@@ -55,13 +83,26 @@ protected void Refresh(IConnection connection)
         {
             if (connection == null)
             {
+                LoggedIn = false;
                 IsVisible = false;
+                if (sectionIndex == 0 && ServiceProvider != null)
+                {
+                    var section = ServiceProvider?.GetSection(TeamExplorerInvitationBase.TeamExplorerInvitationSectionGuid);
+                    IsVisible = !(section?.IsVisible ?? true); // only show this when the invitation section is hidden. When in doubt, don't show it.
+                    if (section != null)
+                        section.PropertyChanged += (s, p) =>
+                        {
+                            if (p.PropertyName == "IsVisible")
+                                IsVisible = LoggedIn || !((ITeamExplorerSection)s).IsVisible;
+                        };
+                }
             }
             else if (SectionConnection != connection)
             {
                 SectionConnection = connection;
                 Title = connection.HostAddress.Title;
                 IsVisible = true;
+                LoggedIn = true;
             }
         }
 
@@ -74,27 +115,21 @@ public override void Refresh()
         public override void Initialize(object sender, SectionInitializeEventArgs e)
         {
             base.Initialize(sender, e);
-
-            // when we start up the connection list is loaded before we can get notifications so refresh manually
-            if (SectionConnection == null && connectionManager.Connections.Count > sectionIndex)
-                Refresh(connectionManager.Connections[sectionIndex]);
+            UpdateConnection();
         }
 
         void UpdateConnection()
         {
             if (connectionManager.Connections.Count > sectionIndex)
                 Refresh(connectionManager.Connections[sectionIndex]);
             else
-                Refresh(null);
+                Refresh(SectionConnection);
         }
 
         void OnPropertyChange(object sender, System.ComponentModel.PropertyChangedEventArgs e)
         {
             if (e.PropertyName == "IsVisible" && IsVisible && View == null)
-            {
-                View = new GitHubConnectContent();
-                View.ViewModel = this;
-            }
+                View = new GitHubConnectContent() { DataContext = this };
         }
 
         public void DoCreate()
@@ -116,6 +151,11 @@ public void SignOut()
             SectionConnection.Logout();
         }
 
+        public void Login()
+        {
+            StartFlow(UIControllerFlow.Authentication);
+        }
+
         void StartFlow(UIControllerFlow controllerFlow)
         {
             var uiProvider = ServiceProvider.GetExportedValue<IUIProvider>();

src/GitHub.VisualStudio/UI/Views/GitHubConnectContent.xaml

@@ -18,6 +18,7 @@
                 <cache:SharedDictionaryManager Source="pack://application:,,,/GitHub.VisualStudio;component/SharedDictionary.xaml" />
                 <cache:SharedDictionaryManager Source="pack://application:,,,/GitHub.UI;component/SharedDictionary.xaml" />
             </ResourceDictionary.MergedDictionaries>
+            <ui:BooleanToVisibilityConverter x:Key="BooleanToVisibilityConverter" />
             <Style TargetType="Label">
                 <Setter Property="Foreground" Value="{DynamicResource VsBrush.GrayText}" />
                 <Setter Property="Padding" Value="0" />
@@ -30,7 +31,10 @@
             <Label Margin="3 0 3 0">|</Label>
             <Button Style="{StaticResource ActionLinkButton}" KeyboardNavigation.TabIndex="1" x:Name="createLink" Click="createLink_Click">Create</Button>
             <Label Margin="3 0 3 0">|</Label>
-            <Button Style="{StaticResource ActionLinkButton}" KeyboardNavigation.TabIndex="2" x:Name="signOut" Click="signOut_Click">Sign out</Button>
+            <Button Style="{StaticResource ActionLinkButton}" KeyboardNavigation.TabIndex="2" x:Name="signOut" Click="signOut_Click"
+                    Visibility="{Binding ShowLogout, Converter={StaticResource BooleanToVisibilityConverter}}">Sign out</Button>
+            <Button Style="{StaticResource ActionLinkButton}" KeyboardNavigation.TabIndex="2" x:Name="login" Click="login_Click"
+                    Visibility="{Binding ShowLogin, Converter={StaticResource BooleanToVisibilityConverter}}">Login</Button>
         </WrapPanel>
       </Grid>
 </UserControl>

src/GitHub.VisualStudio/UI/Views/GitHubConnectContent.xaml.cs

@@ -14,6 +14,8 @@ public partial class GitHubConnectContent : UserControl
         public GitHubConnectContent()
         {
             InitializeComponent();
+
+            DataContextChanged += (s, e) => ViewModel = e.NewValue as GitHubConnectSection;
         }
 
         void cloneLink_Click(object sender, RoutedEventArgs e)
@@ -37,6 +39,13 @@ void signOut_Click(object sender, RoutedEventArgs e)
             signOut.IsEnabled = true;
         }
 
+        private void login_Click(object sender, RoutedEventArgs e)
+        {
+            login.IsEnabled = false;
+            ViewModel.Login();
+            login.IsEnabled = true;
+        }
+
         public GitHubConnectSection ViewModel
         {
             get { return (GitHubConnectSection)GetValue(ViewModelProperty); }