enhance: revision file viewer

love-linger · love-linger · commit ac55bed81228 · 2025-06-06T10:07:58.000+08:00
- show current file path
- add a toggle button to use global syntax highlighting setting (sometimes TextMateSharp will crash this app)

Signed-off-by: leo &lt;longshuang@msn.cn&gt;
diff --git a/src/ViewModels/CommitDetail.cs b/src/ViewModels/CommitDetail.cs
@@ -105,10 +105,16 @@ public string SearchChangeFilter
             }
         }
 
+        public string ViewRevisionFilePath
+        {
+            get => _viewRevisionFilePath;
+            private set => SetProperty(ref _viewRevisionFilePath, value);
+        }
+
         public object ViewRevisionFileContent
         {
             get => _viewRevisionFileContent;
-            set => SetProperty(ref _viewRevisionFileContent, value);
+            private set => SetProperty(ref _viewRevisionFileContent, value);
         }
 
         public string RevisionFileSearchFilter
@@ -189,10 +195,13 @@ public void ViewRevisionFile(Models.Object file)
         {
             if (file == null)
             {
+                ViewRevisionFilePath = string.Empty;
                 ViewRevisionFileContent = null;
                 return;
             }
 
+            ViewRevisionFilePath = file.Path;
+
             switch (file.Type)
             {
                 case Models.ObjectType.Blob:
@@ -893,6 +902,7 @@ private Task ResetToParentRevision(Models.Change change)
         private List<Models.Change> _selectedChanges = null;
         private string _searchChangeFilter = string.Empty;
         private DiffContext _diffContext = null;
+        private string _viewRevisionFilePath = string.Empty;
         private object _viewRevisionFileContent = null;
         private CancellationTokenSource _cancellationSource = null;
         private bool _requestingRevisionFiles = false;
diff --git a/src/Views/RevisionFileContentViewer.axaml b/src/Views/RevisionFileContentViewer.axaml
@@ -23,6 +23,7 @@
       <v:RevisionTextFileView FontFamily="{DynamicResource Fonts.Monospace}"
                               FontSize="{Binding Source={x:Static vm:Preferences.Instance}, Path=EditorFontSize}"
                               TabWidth="{Binding Source={x:Static vm:Preferences.Instance}, Path=EditorTabWidth}"
+                              UseSyntaxHighlighting="{Binding Source={x:Static vm:Preferences.Instance}, Path=UseSyntaxHighlighting}"
                               Background="{DynamicResource Brush.Contents}"/>
     </DataTemplate>
 
diff --git a/src/Views/RevisionFileContentViewer.axaml.cs b/src/Views/RevisionFileContentViewer.axaml.cs
@@ -24,6 +24,15 @@ public int TabWidth
             set => SetValue(TabWidthProperty, value);
         }
 
+        public static readonly StyledProperty<bool> UseSyntaxHighlightingProperty =
+            AvaloniaProperty.Register<RevisionTextFileView, bool>(nameof(UseSyntaxHighlighting));
+
+        public bool UseSyntaxHighlighting
+        {
+            get => GetValue(UseSyntaxHighlightingProperty);
+            set => SetValue(UseSyntaxHighlightingProperty, value);
+        }
+
         protected override Type StyleKeyOverride => typeof(TextEditor);
 
         public RevisionTextFileView() : base(new TextArea(), new TextDocument())
@@ -72,8 +81,8 @@ protected override void OnDataContextChanged(EventArgs e)
 
             if (DataContext is Models.RevisionTextFile source)
             {
-                UpdateTextMate();
                 Text = source.Content;
+                Models.TextMateHelper.SetGrammarByFileName(_textMate, source.FileName);
             }
             else
             {
@@ -86,9 +95,9 @@ protected override void OnPropertyChanged(AvaloniaPropertyChangedEventArgs chang
             base.OnPropertyChanged(change);
 
             if (change.Property == TabWidthProperty)
-            {
                 Options.IndentationSize = TabWidth;
-            }
+            else if (change.Property == UseSyntaxHighlightingProperty)
+                UpdateTextMate();
         }
 
         private void OnTextViewContextRequested(object sender, ContextRequestedEventArgs e)
@@ -124,11 +133,22 @@ private void OnTextViewContextRequested(object sender, ContextRequestedEventArgs
 
         private void UpdateTextMate()
         {
-            if (_textMate == null)
-                _textMate = Models.TextMateHelper.CreateForEditor(this);
+            if (UseSyntaxHighlighting)
+            {
+                if (_textMate == null)
+                    _textMate = Models.TextMateHelper.CreateForEditor(this);
 
-            if (DataContext is Models.RevisionTextFile file)
-                Models.TextMateHelper.SetGrammarByFileName(_textMate, file.FileName);
+                if (DataContext is Models.RevisionTextFile file)
+                    Models.TextMateHelper.SetGrammarByFileName(_textMate, file.FileName);
+            }
+            else if (_textMate != null)
+            {
+                _textMate.Dispose();
+                _textMate = null;
+                GC.Collect();
+
+                TextArea.TextView.Redraw();
+            }
         }
 
         private TextMate.Installation _textMate = null;
diff --git a/src/Views/RevisionFileTreeView.axaml.cs b/src/Views/RevisionFileTreeView.axaml.cs
@@ -313,16 +313,13 @@ private void OnTreeNodeDoubleTapped(object sender, TappedEventArgs e)
 
         private void OnRowsSelectionChanged(object sender, SelectionChangedEventArgs _)
         {
-            if (_disableSelectionChangingEvent)
+            if (_disableSelectionChangingEvent || DataContext is not ViewModels.CommitDetail vm)
                 return;
 
-            if (sender is ListBox { SelectedItem: ViewModels.RevisionFileTreeNode node } && DataContext is ViewModels.CommitDetail vm)
-            {
-                if (!node.IsFolder)
-                    vm.ViewRevisionFile(node.Backend);
-                else
-                    vm.ViewRevisionFile(null);
-            }
+            if (sender is ListBox { SelectedItem: ViewModels.RevisionFileTreeNode { IsFolder: false } node })
+                vm.ViewRevisionFile(node.Backend);
+            else
+                vm.ViewRevisionFile(null);
         }
 
         private List<ViewModels.RevisionFileTreeNode> GetChildrenOfTreeNode(ViewModels.RevisionFileTreeNode node)
diff --git a/src/Views/RevisionFiles.axaml b/src/Views/RevisionFiles.axaml
@@ -113,7 +113,33 @@
     <!-- Right: File Content Viewer -->
     <Grid Grid.Column="2">
       <Border BorderThickness="1" BorderBrush="{DynamicResource Brush.Border2}">
-        <v:RevisionFileContentViewer Content="{Binding ViewRevisionFileContent}"/>
+        <Grid RowDefinitions="Auto,*">
+          <Border Grid.Row="0"
+                  Height="26"
+                  BorderBrush="{DynamicResource Brush.Border2}" BorderThickness="0,0,0,1"
+                  IsVisible="{Binding ViewRevisionFilePath, Converter={x:Static StringConverters.IsNotNullOrEmpty}}">
+            <Grid ColumnDefinitions="Auto,*,Auto">
+              <Path Grid.Column="0" Width="12" Height="12" Data="{StaticResource Icons.File}" Margin="8,0,0,0"/>
+              <TextBlock Grid.Column="1"
+                         Classes="primary"
+                         Margin="4,0,0,0"
+                         Text="{Binding ViewRevisionFilePath}"
+                         FontSize="11"
+                         TextTrimming="CharacterEllipsis"/>
+
+              <ToggleButton Grid.Column="2"
+                            Classes="line_path"
+                            Width="28"
+                            Background="Transparent"
+                            IsChecked="{Binding Source={x:Static vm:Preferences.Instance}, Path=UseSyntaxHighlighting, Mode=TwoWay}"
+                            ToolTip.Tip="{DynamicResource Text.Diff.SyntaxHighlight}">
+                <Path Width="13" Height="13" Data="{StaticResource Icons.SyntaxHighlight}" Margin="0,3,0,0"/>
+              </ToggleButton>
+            </Grid>
+          </Border>
+          
+          <v:RevisionFileContentViewer Grid.Row="1" Content="{Binding ViewRevisionFileContent}"/>
+        </Grid>
       </Border>
     </Grid>
   </Grid>

Original file line number	Diff line number	Diff line change
`@@ -105,10 +105,16 @@ public string SearchChangeFilter`
`105`	`105`	`}`
`106`	`106`	`}`
`107`	`107`
	`108`	`+ public string ViewRevisionFilePath`
	`109`	`+ {`
	`110`	`+ get => _viewRevisionFilePath;`
	`111`	`+ private set => SetProperty(ref _viewRevisionFilePath, value);`
	`112`	`+ }`
	`113`	`+`
`108`	`114`	`public object ViewRevisionFileContent`
`109`	`115`	`{`
`110`	`116`	`get => _viewRevisionFileContent;`
`111`		`- set => SetProperty(ref _viewRevisionFileContent, value);`
	`117`	`+ private set => SetProperty(ref _viewRevisionFileContent, value);`
`112`	`118`	`}`
`113`	`119`
`114`	`120`	`public string RevisionFileSearchFilter`
`@@ -189,10 +195,13 @@ public void ViewRevisionFile(Models.Object file)`
`189`	`195`	`{`
`190`	`196`	`if (file == null)`
`191`	`197`	`{`
	`198`	`+ ViewRevisionFilePath = string.Empty;`
`192`	`199`	`ViewRevisionFileContent = null;`
`193`	`200`	`return;`
`194`	`201`	`}`
`195`	`202`
	`203`	`+ ViewRevisionFilePath = file.Path;`
	`204`	`+`
`196`	`205`	`switch (file.Type)`
`197`	`206`	`{`
`198`	`207`	`case Models.ObjectType.Blob:`
`@@ -893,6 +902,7 @@ private Task ResetToParentRevision(Models.Change change)`
`893`	`902`	`private List<Models.Change> _selectedChanges = null;`
`894`	`903`	`private string _searchChangeFilter = string.Empty;`
`895`	`904`	`private DiffContext _diffContext = null;`
	`905`	`+ private string _viewRevisionFilePath = string.Empty;`
`896`	`906`	`private object _viewRevisionFileContent = null;`
`897`	`907`	`private CancellationTokenSource _cancellationSource = null;`
`898`	`908`	`private bool _requestingRevisionFiles = false;`