SLVS-2526 Show rule help panel on double click

Author: gabriela-trutan-sonarsource

src/IssueViz.Security/ReportView/ReportViewControl.xaml

@@ -57,6 +57,7 @@
                 <Setter Property="IsExpanded" Value="True" />
                 <Setter Property="Foreground" Value="{DynamicResource {x:Static vsShell:VsBrushes.BrandedUITextKey}}" />
                 <EventSetter Event="PreviewMouseUp" Handler="TreeViewItem_OnPreviewMouseUp"/>
+                <EventSetter Event="MouseDoubleClick" Handler="TreeViewItem_MouseDoubleClick"/>
                 <Setter Property="Template">
                     <Setter.Value>
                         <ControlTemplate TargetType="TreeViewItem">

src/IssueViz.Security/ReportView/ReportViewControl.xaml.cs

@@ -147,4 +147,18 @@ private async void ChangeScaStatusMenuItem_OnClick(object sender, RoutedEventArg
     private void TreeView_OnSelectedItemChanged(object sender, RoutedPropertyChangedEventArgs<object> e) => ReportViewModel.SelectedItem = e.NewValue as IIssueViewModel;
 
     private void Hyperlink_OnRequestNavigate(object sender, RequestNavigateEventArgs e) => browserService.Navigate(e.Uri.AbsoluteUri);
+
+    private void TreeViewItem_MouseDoubleClick(object sender, MouseButtonEventArgs e)
+    {
+        if ((sender as FrameworkElement)?.DataContext is not IIssueViewModel issueViewModel)
+        {
+            return;
+        }
+
+        var commandParam = new NavigateToRuleDescriptionCommandParam { FullRuleKey = issueViewModel.RuleInfo.RuleKey, IssueId = issueViewModel.RuleInfo.IssueId };
+        if (ReportViewModel.NavigateToRuleDescriptionCommand.CanExecute(commandParam))
+        {
+            ReportViewModel.NavigateToRuleDescriptionCommand.Execute(commandParam);
+        }
+    }
 }

src/IssueViz/IssueVisualizationControl/ViewModels/Commands/NavigateToRuleDescriptionCommand.cs

@@ -18,9 +18,7 @@
  * Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
  */
 
-using System;
 using System.ComponentModel.Composition;
-using System.Diagnostics;
 using System.Globalization;
 using System.Windows.Data;
 using System.Windows.Input;
@@ -48,13 +46,13 @@ public NavigateToRuleDescriptionCommand(IEducation educationService)
                     }
                 },
                 parameter => parameter is NavigateToRuleDescriptionCommandParam s &&
-                    !string.IsNullOrEmpty(s.FullRuleKey) &&
-                SonarCompositeRuleId.TryParse(s.FullRuleKey, out var _))
+                             !string.IsNullOrEmpty(s.FullRuleKey) &&
+                             SonarCompositeRuleId.TryParse(s.FullRuleKey, out var _))
         {
         }
     }
 
-    internal class NavigateToRuleDescriptionCommandParam
+    public class NavigateToRuleDescriptionCommandParam
     {
         /// <summary>
         /// The id of the issue that comes from SlCore
@@ -65,7 +63,11 @@ internal class NavigateToRuleDescriptionCommandParam
 
     public class NavigateToRuleDescriptionCommandConverter : IMultiValueConverter
     {
-        public object Convert(object[] values, Type targetType, object parameter, CultureInfo culture)
+        public object Convert(
+            object[] values,
+            Type targetType,
+            object parameter,
+            CultureInfo culture)
         {
             if (values.Length == 2 && values[0] is string && (values[1] is Guid || values[1] == null))
             {
@@ -74,7 +76,11 @@ public object Convert(object[] values, Type targetType, object parameter, Cultur
             return null;
         }
 
-        public object[] ConvertBack(object value, Type[] targetTypes, object parameter, CultureInfo culture)
+        public object[] ConvertBack(
+            object value,
+            Type[] targetTypes,
+            object parameter,
+            CultureInfo culture)
         {
             Debug.Fail("We should not hit here");
             return null;