ux: new style for pushing a new branch to remote

Author: love-linger

src/ViewModels/Push.cs

@@ -26,7 +26,6 @@ public Models.Branch SelectedLocalBranch
         public List<Models.Branch> LocalBranches
         {
             get;
-            private set;
         }
 
         public List<Models.Remote> Remotes
@@ -157,7 +156,7 @@ public override Task<bool> Sure()
         {
             _repo.SetWatcherEnabled(false);
 
-            var remoteBranchName = _selectedRemoteBranch.Name.Replace(" (new)", "");
+            var remoteBranchName = _selectedRemoteBranch.Name;
             ProgressDescription = $"Push {_selectedLocalBranch.Name} -> {_selectedRemote.Name}/{remoteBranchName} ...";
 
             return Task.Run(() =>
@@ -187,7 +186,7 @@ private void AutoSelectBranchByRemote()
                     branches.Add(branch);
             }
 
-            // If selected local branch has upstream branch. Try to find it in current remote branches.
+            // If selected local branch has upstream. Try to find it in current remote branches.
             if (!string.IsNullOrEmpty(_selectedLocalBranch.Upstream))
             {
                 foreach (var branch in branches)
@@ -201,7 +200,7 @@ private void AutoSelectBranchByRemote()
                 }
             }
 
-            // Find best remote branch by name.
+            // Try to find a remote branch with the same name of selected local branch.
             foreach (var branch in branches)
             {
                 if (_selectedLocalBranch.Name == branch.Name)
@@ -215,7 +214,7 @@ private void AutoSelectBranchByRemote()
             // Add a fake new branch.
             var fake = new Models.Branch()
             {
-                Name = $"{_selectedLocalBranch.Name} (new)",
+                Name = _selectedLocalBranch.Name,
                 Remote = _selectedRemote.Name,
             };
             branches.Add(fake);
@@ -226,7 +225,7 @@ private void AutoSelectBranchByRemote()
         private readonly Repository _repo = null;
         private Models.Branch _selectedLocalBranch = null;
         private Models.Remote _selectedRemote = null;
-        private List<Models.Branch> _remoteBranches = new List<Models.Branch>();
+        private List<Models.Branch> _remoteBranches = [];
         private Models.Branch _selectedRemoteBranch = null;
         private bool _isSetTrackOptionVisible = false;
     }

src/Views/Push.axaml

@@ -65,7 +65,15 @@
           <DataTemplate x:DataType="{x:Type m:Branch}">
             <StackPanel Orientation="Horizontal" Height="20" VerticalAlignment="Center">
               <Path Margin="0,0,8,0" Width="14" Height="14" Fill="{DynamicResource Brush.FG1}" Data="{StaticResource Icons.Branch}"/>
-              <TextBlock Text="{Binding Name}"/>
+              <TextBlock Text="{Binding Name}" VerticalAlignment="Center"/>
+              <Border Height="14"
+                      CornerRadius="7"
+                      Margin="4,0,0,0" Padding="6,0" 
+                      VerticalAlignment="Center" 
+                      Background="Green"
+                      IsVisible="{Binding Head, Converter={x:Static StringConverters.IsNullOrEmpty}}">
+                <TextBlock Text="NEW" FontSize="9" FontFamily="{DynamicResource Fonts.Monospace}" Foreground="White" VerticalAlignment="Center"/>
+              </Border>
             </StackPanel>
           </DataTemplate>
         </ComboBox.ItemTemplate>