Added context menu to PR list.

Author: grokys

src/GitHub.App/SampleData/PullRequestListViewModelDesigner.cs

@@ -63,6 +63,7 @@ public PullRequestListViewModelDesigner()
         public Uri WebUrl => null;
         public ReactiveCommand<object> CreatePullRequest { get; }
         public ReactiveCommand<Unit> OpenItem { get; }
+        public ReactiveCommand<object> OpenItemInBrowser { get; }
 
         public Task InitializeAsync(ILocalRepositoryModel repository, IConnection connection) => Task.CompletedTask;
     }

src/GitHub.App/ViewModels/GitHubPane/PullRequestListViewModel.cs

@@ -51,6 +51,7 @@ public PullRequestListViewModel(
                 .Select(x => x?.CloneUrl?.ToRepositoryUrl().Append("pulls"))
                 .ToProperty(this, x => x.WebUrl);
             CreatePullRequest = ReactiveCommand.Create().OnExecuteCompleted(_ => NavigateTo("pull/new"));
+            OpenItemInBrowser = ReactiveCommand.Create();
         }
 
         /// <inheritdoc/>
@@ -62,6 +63,9 @@ public PullRequestListViewModel(
         /// <inheritdoc/>
         public ReactiveCommand<object> CreatePullRequest { get; }
 
+        /// <inheritdoc/>
+        public ReactiveCommand<object> OpenItemInBrowser { get; }
+
         /// <inheritdoc/>
         protected override IVirtualizingListSource<IIssueListItemViewModelBase> CreateItemSource()
         {

src/GitHub.Exports.Reactive/ViewModels/GitHubPane/IPullRequestListViewModel.cs

@@ -12,5 +12,10 @@ public interface IPullRequestListViewModel : IIssueListViewModelBase, IOpenInBro
         /// Gets a command which navigates to the "Create Pull Request" view.
         /// </summary>
         ReactiveCommand<object> CreatePullRequest { get; }
+
+        /// <summary>
+        /// Gets a command that opens pull request item on GitHub.
+        /// </summary>
+        ReactiveCommand<object> OpenItemInBrowser { get; }
     }
 }

src/GitHub.VisualStudio.UI/Resources.Designer.cs

@@ -434,4 +434,7 @@
   <data name="UpdateComment" xml:space="preserve">
     <value>Update comment</value>
   </data>
+  <data name="Open" xml:space="preserve">
+    <value>Open</value>
+  </data>
 </root>

src/GitHub.VisualStudio.UI/Resources.resx

@@ -1,11 +1,13 @@
 <UserControl x:Class="GitHub.VisualStudio.Views.GitHubPane.PullRequestListView"
+             Name="root"
              xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
              xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
              xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
              xmlns:ghfvs="https://github.com/github/VisualStudio"
              xmlns:local="clr-namespace:GitHub.VisualStudio.Views.GitHubPane"
              xmlns:v="clr-namespace:GitHub.VisualStudio.Views"
              xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
+             xmlns:prop="clr-namespace:GitHub.VisualStudio.UI;assembly=GitHub.VisualStudio.UI"
              mc:Ignorable="d" d:DesignHeight="300" d:DesignWidth="300"
              AutomationProperties.AutomationId="{x:Static ghfvs:AutomationIDs.PullRequestListViewCustom}">
     <d:DesignProperties.DataContext>
@@ -107,7 +109,14 @@
             <ListBox ItemsSource="{Binding ItemsView}"
                      BorderThickness="0"
                      ScrollViewer.HorizontalScrollBarVisibility="Disabled"
-                     Visibility="{Binding Message, Converter={ghfvs:EqualsToVisibilityConverter None}}">
+                     Visibility="{Binding Message, Converter={ghfvs:EqualsToVisibilityConverter None}}"
+                     ContextMenuOpening="ListBox_ContextMenuOpening">
+                <ListBox.ContextMenu>
+                    <ContextMenu>
+                        <MenuItem Header="{x:Static prop:Resources.Open}" Command="{Binding OpenItem}"/>
+                        <MenuItem Header="{x:Static prop:Resources.openInBrowser}" Command="{Binding OpenItemInBrowser}"/>
+                    </ContextMenu>
+                </ListBox.ContextMenu>
                 <ListBox.ItemContainerStyle>
                     <Style TargetType="ListBoxItem" BasedOn="{StaticResource {x:Type ListBoxItem}}">
                         <Setter Property="HorizontalContentAlignment" Value="Stretch"/>

src/GitHub.VisualStudio/Views/GitHubPane/PullRequestListView.xaml

@@ -1,9 +1,17 @@
 using System;
 using System.ComponentModel.Composition;
+using System.Linq;
+using System.Reactive.Disposables;
 using System.Reactive.Linq;
+using System.Windows;
 using System.Windows.Controls;
+using System.Windows.Documents;
 using System.Windows.Input;
+using System.Windows.Media;
 using GitHub.Exports;
+using GitHub.Extensions;
+using GitHub.Services;
+using GitHub.UI.Helpers;
 using GitHub.ViewModels.GitHubPane;
 using ReactiveUI;
 
@@ -23,13 +31,34 @@ public PullRequestListView()
             {
                 var vm = DataContext as IPullRequestListViewModel;
                 subscription?.Dispose();
-                subscription = vm.AuthorFilter.WhenAnyValue(x => x.Selected)
-                    .Skip(1)
-                    .Subscribe(_ => authorFilterDropDown.IsOpen = false);
+                subscription = null;
+
+                if (vm != null)
+                {
+                    subscription = new CompositeDisposable(
+                        vm.AuthorFilter.WhenAnyValue(x => x.Selected)
+                            .Skip(1)
+                            .Subscribe(_ => authorFilterDropDown.IsOpen = false),
+                        vm.OpenItemInBrowser.Subscribe(x => OpenInBrowser((IPullRequestListItemViewModel)x)));
+                }
             };
         }
 
-        private void ListBoxItem_MouseDoubleClick(object sender, MouseButtonEventArgs e)
+        [Import]
+        IVisualStudioBrowser VisualStudioBrowser { get; set; }
+
+        void OpenInBrowser(IPullRequestListItemViewModel item)
+        {
+            var vm = DataContext as IPullRequestListViewModel;
+
+            if (vm != null)
+            {
+                var uri = vm.RemoteRepository.CloneUrl.ToRepositoryUrl().Append("pull/" + item.Number);
+                VisualStudioBrowser.OpenUrl(uri);
+            }
+        }
+
+        void ListBoxItem_MouseDoubleClick(object sender, MouseButtonEventArgs e)
         {
             var control = sender as ListBoxItem;
             var pr = control?.DataContext as IPullRequestListItemViewModel;
@@ -41,9 +70,50 @@ private void ListBoxItem_MouseDoubleClick(object sender, MouseButtonEventArgs e)
             }
         }
 
-        private void authorFilterDropDown_PopupOpened(object sender, EventArgs e)
+        void authorFilterDropDown_PopupOpened(object sender, EventArgs e)
         {
             authorFilter.FocusSearchBox();
         }
+
+        void ListBox_ContextMenuOpening(object sender, ContextMenuEventArgs e)
+        {
+            ApplyContextMenuBinding<ListBoxItem>(sender, e);
+        }
+
+        void ApplyContextMenuBinding<TItem>(object sender, ContextMenuEventArgs e) where TItem : Control
+        {
+            var container = (Control)sender;
+            var item = GetVisual(e.OriginalSource)?.GetSelfAndVisualAncestors().OfType<TItem>().FirstOrDefault();
+
+            e.Handled = true;
+
+            if (item?.DataContext is IPullRequestListItemViewModel listItem)
+            {
+                container.ContextMenu.DataContext = this.DataContext;
+
+                foreach (var menuItem in container.ContextMenu.Items.OfType<MenuItem>())
+                {
+                    menuItem.CommandParameter = listItem;
+                }
+
+                e.Handled = false;
+            }
+        }
+
+        private Visual GetVisual(object element)
+        {
+            if (element is Visual v)
+            {
+                return v;
+            }
+            else if (element is TextElement e)
+            {
+                return e.Parent as Visual;
+            }
+            else
+            {
+                return null;
+            }
+        }
     }
 }