From 3052938602ed026436d82504f2a82ea8c64bafb8 Mon Sep 17 00:00:00 2001
From: Bastian Schmidt <batzen@gmx.org>
Date: Thu, 21 May 2020 09:56:22 +0200
Subject: [PATCH] Fixes #795 by changing order in xaml

---
 Changelog.md                                |  1 +
 Fluent.Ribbon/Themes/Controls/MenuItem.xaml | 72 ++++++++++-----------
 2 files changed, 37 insertions(+), 36 deletions(-)

diff --git a/Changelog.md b/Changelog.md
index c12043d71..bb1b84400 100644
--- a/Changelog.md
+++ b/Changelog.md
@@ -21,6 +21,7 @@
   - [#788](../../issues/788) - Maximize icon is incorrectly drawn on high-dpi displays
   - [#789](../../issues/789) - Opening gallery messes up InRibbonGallery resizing.
   - [#791](../../issues/791) - Problem with dynamic items in InRibbonGallery
+  - [#795](../../issues/795) - Check mark of menuitem not visible
 
 ## 7.1.0
 
diff --git a/Fluent.Ribbon/Themes/Controls/MenuItem.xaml b/Fluent.Ribbon/Themes/Controls/MenuItem.xaml
index 169f147e2..e65e71edb 100644
--- a/Fluent.Ribbon/Themes/Controls/MenuItem.xaml
+++ b/Fluent.Ribbon/Themes/Controls/MenuItem.xaml
@@ -172,15 +172,6 @@
                               Grid.Column="1"
                               d:LayoutOverrides="Width"
                               RecognizesAccessKey="{Binding Path=(Fluent:MenuItem.RecognizesAccessKey), RelativeSource={RelativeSource TemplatedParent}}" />
-            <Image x:Name="checkedImage"
-                   HorizontalAlignment="Center"
-                   VerticalAlignment="Center"
-                   Width="16"
-                   Height="16"
-                   Stretch="Uniform"
-                   Source="{DynamicResource Fluent.Ribbon.Images.Checked}"
-                   Visibility="Collapsed"
-                   SnapsToDevicePixels="True" />
             <Border x:Name="iconImageBorder"
                     BorderThickness="1">
                 <ContentPresenter x:Name="iconImage"
@@ -192,6 +183,15 @@
                                   Margin="3,2,2,2"
                                   SnapsToDevicePixels="True" />
             </Border>
+            <Image x:Name="checkedImage"
+                   HorizontalAlignment="Center"
+                   VerticalAlignment="Center"
+                   Width="16"
+                   Height="16"
+                   Stretch="Uniform"
+                   Source="{DynamicResource Fluent.Ribbon.Images.Checked}"
+                   Visibility="Collapsed"
+                   SnapsToDevicePixels="True" />
             <TextBlock x:Name="gestureTextBlock"
                        Text="{TemplateBinding InputGestureText}"
                        Width="Auto"
@@ -305,15 +305,6 @@
                               Grid.Column="1"
                               d:LayoutOverrides="Width"
                               RecognizesAccessKey="{Binding Path=(Fluent:MenuItem.RecognizesAccessKey), RelativeSource={RelativeSource TemplatedParent}}" />
-            <Image x:Name="checkedImage"
-                   HorizontalAlignment="Center"
-                   VerticalAlignment="Center"
-                   Width="16"
-                   Height="16"
-                   Stretch="Uniform"
-                   Source="{DynamicResource Fluent.Ribbon.Images.Checked}"
-                   Visibility="Collapsed"
-                   SnapsToDevicePixels="True" />
             <Border x:Name="iconImageBorder"
                     BorderThickness="1">
                 <ContentPresenter x:Name="iconImage"
@@ -325,6 +316,15 @@
                                   Margin="3,2,2,2"
                                   SnapsToDevicePixels="True" />
             </Border>
+            <Image x:Name="checkedImage"
+                   HorizontalAlignment="Center"
+                   VerticalAlignment="Center"
+                   Width="16"
+                   Height="16"
+                   Stretch="Uniform"
+                   Source="{DynamicResource Fluent.Ribbon.Images.Checked}"
+                   Visibility="Collapsed"
+                   SnapsToDevicePixels="True" />
             <Path x:Name="path"
                   Stretch="Fill"
                   Stroke="{x:Null}"
@@ -566,15 +566,6 @@
                               Grid.Column="1"
                               d:LayoutOverrides="Width"
                               RecognizesAccessKey="{Binding Path=(Fluent:MenuItem.RecognizesAccessKey), RelativeSource={RelativeSource TemplatedParent}}" />
-            <Image x:Name="checkedImage"
-                   HorizontalAlignment="Center"
-                   VerticalAlignment="Center"
-                   Width="16"
-                   Height="16"
-                   Stretch="Uniform"
-                   Source="{DynamicResource Fluent.Ribbon.Images.Checked}"
-                   Visibility="Collapsed"
-                   SnapsToDevicePixels="True" />
             <Border x:Name="iconImageBorder"
                     BorderThickness="1">
                 <ContentPresenter x:Name="iconImage"
@@ -586,6 +577,15 @@
                                   Margin="3,2,2,2"
                                   SnapsToDevicePixels="True" />
             </Border>
+            <Image x:Name="checkedImage"
+                   HorizontalAlignment="Center"
+                   VerticalAlignment="Center"
+                   Width="16"
+                   Height="16"
+                   Stretch="Uniform"
+                   Source="{DynamicResource Fluent.Ribbon.Images.Checked}"
+                   Visibility="Collapsed"
+                   SnapsToDevicePixels="True" />
             <Path x:Name="path"
                   Stretch="Fill"
                   Stroke="{x:Null}"
@@ -827,15 +827,6 @@
                               Grid.Column="1"
                               d:LayoutOverrides="Width"
                               RecognizesAccessKey="{Binding Path=(Fluent:MenuItem.RecognizesAccessKey), RelativeSource={RelativeSource TemplatedParent}}" />
-            <Image x:Name="checkedImage"
-                   HorizontalAlignment="Center"
-                   VerticalAlignment="Center"
-                   Width="16"
-                   Height="16"
-                   Stretch="Uniform"
-                   Source="{DynamicResource Fluent.Ribbon.Images.Checked}"
-                   Visibility="Collapsed"
-                   SnapsToDevicePixels="True" />
             <Border x:Name="iconImageBorder"
                     BorderThickness="1">
                 <ContentPresenter x:Name="iconImage"
@@ -847,6 +838,15 @@
                                   Margin="3,2,2,2"
                                   SnapsToDevicePixels="True" />
             </Border>
+            <Image x:Name="checkedImage"
+                   HorizontalAlignment="Center"
+                   VerticalAlignment="Center"
+                   Width="16"
+                   Height="16"
+                   Stretch="Uniform"
+                   Source="{DynamicResource Fluent.Ribbon.Images.Checked}"
+                   Visibility="Collapsed"
+                   SnapsToDevicePixels="True" />
             <TextBlock x:Name="gestureTextBlock"
                        Text="{TemplateBinding InputGestureText}"
                        Width="Auto"