refactor: use MultiBinding instead of code to control visibility of Commit & Push button

Author: love-linger

src/ViewModels/WorkingCopy.cs

@@ -29,11 +29,7 @@ public bool IncludeUntracked
         public bool CanCommitWithPush
         {
             get => _canCommitWithPush;
-            set
-            {
-                if (SetProperty(ref _canCommitWithPush, value))
-                    OnPropertyChanged(nameof(IsCommitWithPushVisible));
-            }
+            set => SetProperty(ref _canCommitWithPush, value);
         }
 
         public bool HasUnsolvedConflicts
@@ -89,16 +85,10 @@ public bool UseAmend
 
                     Staged = GetStagedChanges();
                     SelectedStaged = [];
-                    OnPropertyChanged(nameof(IsCommitWithPushVisible));
                 }
             }
         }
 
-        public bool IsCommitWithPushVisible
-        {
-            get => !UseAmend && CanCommitWithPush;
-        }
-
         public List<Models.Change> Unstaged
         {
             get => _unstaged;

src/Views/WorkingCopy.axaml

@@ -355,8 +355,15 @@
                 HotKey="Alt+Enter"
                 ToolTip.Tip="{OnPlatform Alt+Enter, macOS=⌥+Enter}"
                 ToolTip.Placement="Top"
-                ToolTip.VerticalOffset="0"
-                IsVisible="{Binding IsCommitWithPushVisible}"/>
+                ToolTip.VerticalOffset="0">
+          <Button.IsVisible>
+            <MultiBinding Converter="{x:Static BoolConverters.And}">
+              <Binding Path="UseAmend" Converter="{x:Static BoolConverters.Not}"/>
+              <Binding Path="CanCommitWithPush"/>
+              <Binding Path="InProgressContext" Converter="{x:Static ObjectConverters.IsNull}"/>
+            </MultiBinding>
+          </Button.IsVisible>  
+        </Button>
       </Grid>
     </Grid>
   </Grid>