fix: Conventional Commit Helper not working in Interactive Rebase (#1446)

Signed-off-by: leo <longshuang@msn.cn>

src/App.axaml.cs

@@ -531,8 +531,8 @@ namespace SourceGit
                 return true;
             }
 
-            var editor = new Views.StandaloneCommitMessageEditor();
+            var editor = new Views.CommitMessageEditor();
-            editor.SetFile(file);
+            editor.AsStandalone(file);
             desktop.MainWindow = editor;
             return true;
         }

src/Views/CommitMessageEditor.axaml

@@ -4,7 +4,7 @@
                     xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
                     xmlns:v="using:SourceGit.Views"
                     mc:Ignorable="d" d:DesignWidth="800" d:DesignHeight="450"
-                    x:Class="SourceGit.Views.StandaloneCommitMessageEditor"
+                    x:Class="SourceGit.Views.CommitMessageEditor"
                     x:Name="ThisControl"
                     Icon="/App.ico"
                     Title="{DynamicResource Text.CodeEditor}"

src/Views/CommitMessageEditor.axaml.cs

@@ -0,0 +1,69 @@
+using System;
+using System.IO;
+
+using Avalonia.Interactivity;
+
+namespace SourceGit.Views
+{
+    public partial class CommitMessageEditor : ChromelessWindow
+    {
+        public CommitMessageEditor()
+        {
+            InitializeComponent();
+        }
+
+        public void AsStandalone(string file)
+        {
+            _onSave = msg => File.WriteAllText(file, msg);
+            _shouldExitApp = true;
+
+            var content = File.ReadAllText(file).ReplaceLineEndings("\n").Trim();
+            var firstLineEnd = content.IndexOf('\n', StringComparison.Ordinal);
+            if (firstLineEnd == -1)
+            {
+                Editor.SubjectEditor.Text = content;
+            }
+            else
+            {
+                Editor.SubjectEditor.Text = content.Substring(0, firstLineEnd);
+                Editor.DescriptionEditor.Text = content.Substring(firstLineEnd + 1).Trim();
+            }
+        }
+
+        public void AsBuiltin(string msg, Action<string> onSave)
+        {
+            _onSave = onSave;
+            _shouldExitApp = false;
+
+            var firstLineEnd = msg.IndexOf('\n', StringComparison.Ordinal);
+            if (firstLineEnd == -1)
+            {
+                Editor.SubjectEditor.Text = msg;
+            }
+            else
+            {
+                Editor.SubjectEditor.Text = msg.Substring(0, firstLineEnd);
+                Editor.DescriptionEditor.Text = msg.Substring(firstLineEnd + 1).Trim();
+            }
+        }
+
+        protected override void OnClosed(EventArgs e)
+        {
+            base.OnClosed(e);
+
+            if (_shouldExitApp)
+                App.Quit(_exitCode);
+        }
+
+        private void SaveAndClose(object _1, RoutedEventArgs _2)
+        {
+            _onSave?.Invoke(Editor.Text);
+            _exitCode = 0;
+            Close();
+        }
+
+        private Action<string> _onSave = null;
+        private bool _shouldExitApp = true;
+        private int _exitCode = -1;
+    }
+}

src/Views/CommitMessageTextBox.axaml.cs

@@ -201,7 +201,14 @@ namespace SourceGit.Views
 
         private void OnOpenConventionalCommitHelper(object _, RoutedEventArgs e)
         {
-            App.ShowWindow(new ViewModels.ConventionalCommitMessageBuilder(text => Text = text), true);
+            var toplevel = TopLevel.GetTopLevel(this);
+            if (toplevel is Window owner)
+            {
+                var vm = new ViewModels.ConventionalCommitMessageBuilder(text => Text = text);
+                var builder = new ConventionalCommitMessageBuilder() { DataContext = vm };
+                builder.ShowDialog(owner);
+            }
+
             e.Handled = true;
         }
 

src/Views/InteractiveRebase.axaml

@@ -189,14 +189,11 @@
 
                 <!-- Subject -->
                 <Grid Grid.Column="2" ColumnDefinitions="Auto,*" ClipToBounds="True">
-                  <Button Grid.Column="0" Classes="icon_button" Margin="0,0,8,0" IsVisible="{Binding Action, Converter={x:Static c:InteractiveRebaseActionConverters.CanEditMessage}}">
+                  <Button Grid.Column="0"
-                    <Button.Flyout>
+                          Classes="icon_button"
-                      <Flyout Placement="BottomEdgeAlignedLeft">
+                          Margin="0,0,8,0"
-                        <Panel Width="600" Height="120">
+                          IsVisible="{Binding Action, Converter={x:Static c:InteractiveRebaseActionConverters.CanEditMessage}}"
-                          <v:CommitMessageTextBox Text="{Binding FullMessage, Mode=TwoWay}"/>
+                          Click="OnOpenCommitMessageEditor">
-                        </Panel>
-                      </Flyout>
-                    </Button.Flyout>
                     <Path Width="14" Height="14" Margin="0,4,0,0" Data="{StaticResource Icons.Edit}"/>
                   </Button>
                   <TextBlock Grid.Column="1" Classes="primary" Margin="0,0,4,0" Text="{Binding Subject}"/>

src/Views/InteractiveRebase.axaml.cs

@@ -167,6 +167,18 @@ namespace SourceGit.Views
             e.Handled = true;
         }
 
+        private void OnOpenCommitMessageEditor(object sender, RoutedEventArgs e)
+        {
+            if (sender is Button { DataContext: ViewModels.InteractiveRebaseItem item })
+            {
+                var dialog = new CommitMessageEditor();
+                dialog.AsBuiltin(item.FullMessage, msg => item.FullMessage = msg);
+                dialog.ShowDialog(this);
+            }
+
+            e.Handled = true;
+        }
+
         private async void OnStartJobs(object _1, RoutedEventArgs _2)
         {
             var vm = DataContext as ViewModels.InteractiveRebase;

src/Views/StandaloneCommitMessageEditor.axaml.cs

@@ -1,52 +0,0 @@
-using System;
-using System.IO;
-
-using Avalonia.Interactivity;
-
-namespace SourceGit.Views
-{
-    public partial class StandaloneCommitMessageEditor : ChromelessWindow
-    {
-        public StandaloneCommitMessageEditor()
-        {
-            InitializeComponent();
-        }
-
-        public void SetFile(string file)
-        {
-            _file = file;
-
-            var content = File.ReadAllText(file).ReplaceLineEndings("\n").Trim();
-            var firstLineEnd = content.IndexOf('\n');
-            if (firstLineEnd == -1)
-            {
-                Editor.SubjectEditor.Text = content;
-            }
-            else
-            {
-                Editor.SubjectEditor.Text = content.Substring(0, firstLineEnd);
-                Editor.DescriptionEditor.Text = content.Substring(firstLineEnd + 1).Trim();
-            }
-        }
-
-        protected override void OnClosed(EventArgs e)
-        {
-            base.OnClosed(e);
-            App.Quit(_exitCode);
-        }
-
-        private void SaveAndClose(object _1, RoutedEventArgs _2)
-        {
-            if (!string.IsNullOrEmpty(_file))
-            {
-                File.WriteAllText(_file, Editor.Text);
-                _exitCode = 0;
-            }
-
-            Close();
-        }
-
-        private string _file = string.Empty;
-        private int _exitCode = -1;
-    }
-}