Open details view when PR status is clicked on

Author: jcansdale

src/GitHub.InlineReviews/Services/PullRequestStatusManager.cs

@@ -1,13 +1,16 @@
 using System;
 using System.Windows;
 using System.Windows.Media;
+using System.Windows.Input;
 using System.Windows.Controls;
 using System.Windows.Controls.Primitives;
+using System.ComponentModel;
 using System.ComponentModel.Composition;
 using GitHub.InlineReviews.Views;
 using GitHub.InlineReviews.ViewModels;
 using GitHub.Services;
-using System.ComponentModel;
+using GitHub.VisualStudio;
+using Microsoft.VisualStudio.Shell;
 
 namespace GitHub.InlineReviews.Services
 {
@@ -22,29 +25,15 @@ public class PullRequestStatusManager : IPullRequestStatusManager
         readonly PullRequestStatusViewModel pullRequestStatusViewModel;
 
         [ImportingConstructor]
-        public PullRequestStatusManager(IPullRequestSessionManager pullRequestSessionManager) : this(new PullRequestStatusViewModel())
+        public PullRequestStatusManager(SVsServiceProvider serviceProvider, IPullRequestSessionManager pullRequestSessionManager)
+            : this(CreatePullRequestStatusViewModel(serviceProvider))
         {
             this.pullRequestSessionManager = pullRequestSessionManager;
 
             RefreshCurrentSession();
             pullRequestSessionManager.PropertyChanged += PullRequestSessionManager_PropertyChanged;
         }
 
-        void PullRequestSessionManager_PropertyChanged(object sender, PropertyChangedEventArgs e)
-        {
-            if (e.PropertyName == nameof(PullRequestSessionManager.CurrentSession))
-            {
-                RefreshCurrentSession();
-            }
-        }
-
-        void RefreshCurrentSession()
-        {
-            var pullRequest = pullRequestSessionManager.CurrentSession?.PullRequest;
-            pullRequestStatusViewModel.Number = pullRequest?.Number;
-            pullRequestStatusViewModel.Title = pullRequest?.Title;
-        }
-
         public PullRequestStatusManager(PullRequestStatusViewModel pullRequestStatusViewModel)
         {
             this.pullRequestStatusViewModel = pullRequestStatusViewModel;
@@ -78,6 +67,27 @@ public void HideStatus()
             }
         }
 
+        static PullRequestStatusViewModel CreatePullRequestStatusViewModel(IServiceProvider serviceProvider)
+        {
+            var command = new RaiseVsCommand(serviceProvider, Guids.guidGitHubCmdSetString, PkgCmdIDList.showCurrentPullRequestCommand);
+            return new PullRequestStatusViewModel(command);
+        }
+
+        void PullRequestSessionManager_PropertyChanged(object sender, PropertyChangedEventArgs e)
+        {
+            if (e.PropertyName == nameof(PullRequestSessionManager.CurrentSession))
+            {
+                RefreshCurrentSession();
+            }
+        }
+
+        void RefreshCurrentSession()
+        {
+            var pullRequest = pullRequestSessionManager.CurrentSession?.PullRequest;
+            pullRequestStatusViewModel.Number = pullRequest?.Number;
+            pullRequestStatusViewModel.Title = pullRequest?.Title;
+        }
+
         static UserControl FindPullRequestStatusView(StatusBar statusBar)
         {
             return FindChild<UserControl>(statusBar, GitHubStatusName);
@@ -132,12 +142,46 @@ static T FindChild<T>(DependencyObject parent, string childName) where T : Depen
             return foundChild;
         }
 
+        class RaiseVsCommand : ICommand
+        {
+            readonly IServiceProvider serviceProvider;
+            readonly string guid;
+            readonly int id;
+
+            internal RaiseVsCommand(IServiceProvider serviceProvider, string guid, int id)
+            {
+                this.serviceProvider = serviceProvider;
+                this.guid = guid;
+                this.id = id;
+            }
+
+            public bool CanExecute(object parameter) => true;
+
+            public void Execute(object parameter)
+            {
+                try
+                {
+                    var dte = serviceProvider.GetService(typeof(EnvDTE.DTE)) as EnvDTE.DTE;
+                    object customIn = null;
+                    object customOut = null;
+                    dte.Commands.Raise(guid, id, ref customIn, ref customOut);
+                }
+                catch (Exception e)
+                {
+                    VsOutputLogger.WriteLine("Couldn't raise {0}: {1}", nameof(PkgCmdIDList.openPullRequestsCommand), e);
+                    System.Diagnostics.Trace.WriteLine(e);
+                }
+            }
+
+            public event EventHandler CanExecuteChanged;
+        }
+
         class PullRequestStatusManagerInstaller
         {
             [STAThread]
             void Install()
             {
-                var viewModel = new PullRequestStatusViewModel { Number = 666, Title = "A beast of a PR" };
+                var viewModel = new PullRequestStatusViewModel(null) { Number = 666, Title = "A beast of a PR" };
                 var provider = new PullRequestStatusManager(viewModel);
                 provider.HideStatus();
                 provider.ShowStatus();

src/GitHub.InlineReviews/ViewModels/PullRequestStatusViewModel.cs

@@ -1,4 +1,5 @@
 using System;
+using System.Windows.Input;
 using System.ComponentModel;
 
 namespace GitHub.InlineReviews.ViewModels
@@ -8,6 +9,11 @@ public class PullRequestStatusViewModel : INotifyPropertyChanged
         int? number;
         string title;
 
+        public PullRequestStatusViewModel(ICommand showCurrentPullRequestCommand)
+        {
+            ShowCurrentPullRequestCommand = showCurrentPullRequestCommand;
+        }
+
         public int? Number
         {
             get { return number; }
@@ -34,6 +40,8 @@ public string Title
             }
         }
 
+        public ICommand ShowCurrentPullRequestCommand { get; }
+
         public event PropertyChangedEventHandler PropertyChanged;
     }
 }

src/GitHub.InlineReviews/Views/PullRequestStatusView.xaml

@@ -8,7 +8,9 @@
              d:DesignHeight="20" d:DesignWidth="60">
 
     <Grid Visibility="{Binding Number, TargetNullValue=Collapsed}">
-        <TextBlock>PR# <Run Text="{Binding Number}" /></TextBlock>
+        <Button Command="{Binding ShowCurrentPullRequestCommand}">
+            <TextBlock>PR #<Run Text="{Binding Number}" /></TextBlock>
+        </Button>
         <Grid.ToolTip>
             <TextBlock Text="{Binding Title}" />
         </Grid.ToolTip>