fix: tooltip did not hide after pointer move out (#1178)

Signed-off-by: leo <[email protected]>

Author: love-linger

src/Views/Blame.axaml

@@ -59,7 +59,14 @@
                          FontFamily="{DynamicResource Fonts.Monospace}"
                          FontSize="{Binding Source={x:Static vm:Preferences.Instance}, Path=EditorFontSize}"
                          TabWidth="{Binding Source={x:Static vm:Preferences.Instance}, Path=EditorTabWidth}"
-                         BlameData="{Binding Data}"/>
+                         BlameData="{Binding Data}">
+        <ToolTip.IsOpen>
+          <MultiBinding Converter="{x:Static BoolConverters.And}">
+            <Binding Path="$self.IsPointerOver"/>
+            <Binding Path="$self.(ToolTip.Tip)" Converter="{x:Static ObjectConverters.IsNotNull}"/>
+          </MultiBinding>
+        </ToolTip.IsOpen>
+      </v:BlameTextEditor>
 
       <!-- Not supported mask (for binary files) -->
       <StackPanel Orientation="Vertical"

src/Views/Blame.axaml.cs

@@ -180,15 +180,14 @@ protected override void OnPointerMoved(PointerEventArgs e)
                             {
                                 var msg = blame.GetCommitMessage(info.CommitSHA);
                                 ToolTip.SetTip(this, msg);
-                                ToolTip.SetIsOpen(this, true);
                             }
 
                             return;
                         }
                     }
 
                     Cursor = Cursor.Default;
-                    ToolTip.SetIsOpen(this, false);
+                    ToolTip.SetTip(this, null);
                 }
             }
 

src/Views/CommitBaseInfo.axaml

@@ -114,6 +114,13 @@
                            PointerEntered="OnSHAPointerEntered"
                            PointerPressed="OnSHAPressed"
                            ToolTip.ShowDelay="0">
+                  <ToolTip.IsOpen>
+                    <MultiBinding Converter="{x:Static BoolConverters.And}">
+                      <Binding Path="$self.IsPointerOver"/>
+                      <Binding Path="$self.(ToolTip.Tip)" Converter="{x:Static ObjectConverters.IsNotNull}"/>
+                    </MultiBinding>
+                  </ToolTip.IsOpen>
+                  
                   <TextBlock.DataTemplates>
                     <DataTemplate DataType="m:Commit">
                       <StackPanel MinWidth="400" Orientation="Vertical">
@@ -152,6 +159,13 @@
                            PointerEntered="OnSHAPointerEntered"
                            PointerPressed="OnSHAPressed"
                            ToolTip.ShowDelay="0">
+                  <ToolTip.IsOpen>
+                    <MultiBinding Converter="{x:Static BoolConverters.And}">
+                      <Binding Path="$self.IsPointerOver"/>
+                      <Binding Path="$self.(ToolTip.Tip)" Converter="{x:Static ObjectConverters.IsNotNull}"/>
+                    </MultiBinding>
+                  </ToolTip.IsOpen>
+                  
                   <TextBlock.DataTemplates>
                     <DataTemplate DataType="m:Commit">
                       <StackPanel MinWidth="400" Orientation="Vertical">

src/Views/CommitBaseInfo.axaml.cs

@@ -133,12 +133,7 @@ private void OnSHAPointerEntered(object sender, PointerEventArgs e)
                     Dispatcher.UIThread.Invoke(() =>
                     {
                         if (ctl.IsEffectivelyVisible && ctl.DataContext is string newSHA && newSHA == sha)
-                        {
                             ToolTip.SetTip(ctl, c);
-
-                            if (ctl.IsPointerOver)
-                                ToolTip.SetIsOpen(ctl, true);
-                        }
                     });
                 });
             }

src/Views/CommitMessagePresenter.cs

@@ -5,6 +5,8 @@
 using Avalonia;
 using Avalonia.Controls;
 using Avalonia.Controls.Documents;
+using Avalonia.Data;
+using Avalonia.Data.Converters;
 using Avalonia.Input;
 using Avalonia.Threading;
 using Avalonia.VisualTree;
@@ -24,6 +26,21 @@ public Models.CommitFullMessage FullMessage
 
         protected override Type StyleKeyOverride => typeof(SelectableTextBlock);
 
+        public CommitMessagePresenter()
+        {
+            var bindings = new MultiBinding()
+            {
+                Converter = BoolConverters.And,
+                Bindings = new[]
+                {
+                    new Binding() { Path = "IsPointerOver", Source = this },
+                    new Binding() { Path = "(ToolTip.Tip)", Source = this, TypeResolver = (_, name) => name == "ToolTip" ? typeof(ToolTip) : null, Converter = ObjectConverters.IsNotNull },
+                }
+            };
+
+            Bind(ToolTip.IsOpenProperty, bindings);
+        }
+
         protected override void OnPropertyChanged(AvaloniaPropertyChangedEventArgs change)
         {
             base.OnPropertyChanged(change);
@@ -107,14 +124,9 @@ protected override void OnPointerMoved(PointerEventArgs e)
 
                     _lastHover = link;
                     if (!link.IsCommitSHA)
-                    {
                         ToolTip.SetTip(this, link.Link);
-                        ToolTip.SetIsOpen(this, true);
-                    }
                     else
-                    {
                         ProcessHoverCommitLink(link);
-                    }
 
                     return;
                 }
@@ -264,12 +276,7 @@ private void ProcessHoverCommitLink(Models.Hyperlink link)
             // If we have already queried this SHA, just use it.
             if (_inlineCommits.TryGetValue(sha, out var exist))
             {
-                if (exist != null)
-                {
-                    ToolTip.SetTip(this, exist);
-                    ToolTip.SetIsOpen(this, true);
-                }
-
+                ToolTip.SetTip(this, exist);
                 return;
             }
 
@@ -294,10 +301,7 @@ private void ProcessHoverCommitLink(Models.Hyperlink link)
 
                             // Make sure user still hovers the target SHA.
                             if (_lastHover == link && c != null)
-                            {
                                 ToolTip.SetTip(this, c);
-                                ToolTip.SetIsOpen(this, true);
-                            }
                         }
                     });
                 });

src/Views/CommitSubjectPresenter.cs

@@ -144,7 +144,6 @@ protected override void OnPointerMoved(PointerEventArgs e)
                     _lastHover = match;
                     SetCurrentValue(CursorProperty, Cursor.Parse("Hand"));
                     ToolTip.SetTip(this, match.Link);
-                    ToolTip.SetIsOpen(this, true);
                     e.Handled = true;
                     return;
                 }