remove explicit datatemplate, we are setting this globally in applica…

…tion
jinek · Dec 14, 2024 · 4e902fb · 4e902fb
1 parent a399f28
commit 4e902fb
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 14 deletions.
diff --git a/src/Consolonia.Core/Controls/ConsoloniaAccessText.cs b/src/Consolonia.Core/Controls/ConsoloniaAccessText.cs
@@ -12,6 +12,11 @@ public sealed class ConsoloniaAccessText : AccessText
     {
         private Run _accessRun;
 
+        static ConsoloniaAccessText()
+        {
+            AffectsRender<ConsoloniaAccessText>(ShowAccessKeyProperty);
+        }
+
         public ConsoloniaAccessText()
         {
             this.PropertyChanged += OnPropertyChanged;

diff --git a/src/Consolonia.Core/Infrastructure/ConsoloniaApplication.cs b/src/Consolonia.Core/Infrastructure/ConsoloniaApplication.cs
@@ -25,9 +25,9 @@ public override void OnFrameworkInitializationCompleted()
         {
             // override AccessText to use ConsoloniaAccessText as default contentpresenter for unknown data types (aka string)
             this.DataTemplates.Add(new FuncDataTemplate<object>(
-                (data, s) =>
+                (data, _) =>
                 {
-                    if (data != null && data is string str)
+                    if (data != null && data is string)
                     {
                         var result = new ConsoloniaAccessText();
                         result.Bind(TextBlock.TextProperty,

diff --git a/src/Consolonia.Themes/Templates/Controls/Menu.axaml b/src/Consolonia.Themes/Templates/Controls/Menu.axaml
@@ -22,13 +22,6 @@
                             <ContentPresenter Name="PART_HeaderPresenter"
                                               Content="{TemplateBinding Header}"
                                               Margin="{TemplateBinding Padding}">
-                                <ContentPresenter.DataTemplates>
-                                    <DataTemplate DataType="system:String">
-                                        <!-- ReSharper disable once Xaml.BindingWithContextNotResolved Just workaround - Its not binding, it's TemplateBinding-->
-                                        <core:ConsoloniaAccessText Text="{Binding}"
-                                                    Foreground="{TemplateBinding (TemplatedControl.Foreground)}" />
-                                    </DataTemplate>
-                                </ContentPresenter.DataTemplates>
                             </ContentPresenter>
                         </helpers:CaretControl>
                         <Popup Name="PART_Popup"

diff --git a/src/Consolonia.Themes/Templates/Controls/MenuItem.axaml b/src/Consolonia.Themes/Templates/Controls/MenuItem.axaml
@@ -50,11 +50,6 @@
                                               Margin="{TemplateBinding Padding}"
                                               VerticalAlignment="Center"
                                               Grid.Column="2">
-                                <ContentPresenter.DataTemplates>
-                                    <DataTemplate DataType="sys:String">
-                                        <core:ConsoloniaAccessText Text="{Binding}" />
-                                    </DataTemplate>
-                                </ContentPresenter.DataTemplates>
                             </ContentPresenter>
                             <TextBlock x:Name="PART_InputGestureText"
                                        Grid.Column="3"