From f637c131c4d28e3b617f5e0701cc94eb9569d620 Mon Sep 17 00:00:00 2001
From: Diogo Trindade <diogotr7@gmail.com>
Date: Fri, 20 Nov 2020 21:27:30 +0000
Subject: [PATCH] Added reset layer property button

---
 .../Profile/LayerProperties/LayerProperty.cs  |  8 ++++--
 .../Tree/TreePropertyView.xaml                | 28 +++++++++++++------
 .../Tree/TreePropertyViewModel.cs             |  6 ++++
 3 files changed, 30 insertions(+), 12 deletions(-)

diff --git a/src/Artemis.Core/Models/Profile/LayerProperties/LayerProperty.cs b/src/Artemis.Core/Models/Profile/LayerProperties/LayerProperty.cs
index fd9cab497..74ad46048 100644
--- a/src/Artemis.Core/Models/Profile/LayerProperties/LayerProperty.cs
+++ b/src/Artemis.Core/Models/Profile/LayerProperties/LayerProperty.cs
@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.Collections.ObjectModel;
 using System.Linq;
@@ -208,8 +208,10 @@ namespace Artemis.Core
             if (_disposed)
                 throw new ObjectDisposedException("LayerProperty");
 
-            BaseValue = DefaultValue;
-            CurrentValue = DefaultValue;
+            string json = JsonConvert.SerializeObject(DefaultValue, new JsonSerializerSettings { TypeNameHandling = TypeNameHandling.All });
+            
+            BaseValue = JsonConvert.DeserializeObject<T>(json);
+            CurrentValue = JsonConvert.DeserializeObject<T>(json);
         }
 
         private void ReapplyUpdate()
diff --git a/src/Artemis.UI/Screens/ProfileEditor/LayerProperties/Tree/TreePropertyView.xaml b/src/Artemis.UI/Screens/ProfileEditor/LayerProperties/Tree/TreePropertyView.xaml
index 8e29a9019..c0c51ccb0 100644
--- a/src/Artemis.UI/Screens/ProfileEditor/LayerProperties/Tree/TreePropertyView.xaml
+++ b/src/Artemis.UI/Screens/ProfileEditor/LayerProperties/Tree/TreePropertyView.xaml
@@ -1,4 +1,4 @@
-<UserControl x:Class="Artemis.UI.Screens.ProfileEditor.LayerProperties.Tree.TreePropertyView"
+<UserControl x:Class="Artemis.UI.Screens.ProfileEditor.LayerProperties.Tree.TreePropertyView"
              xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
              xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
              xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
@@ -48,16 +48,17 @@
                 <ColumnDefinition Width="*" />
                 <ColumnDefinition Width="Auto" />
                 <ColumnDefinition Width="Auto" />
+                <ColumnDefinition Width="Auto" />
             </Grid.ColumnDefinitions>
             <ToggleButton Grid.Column="0"
                           Style="{StaticResource MaterialDesignFlatToggleButton}"
                           ToolTip="Toggle key-framing"
-                          Width="18"
-                          Height="18"
+                          Width="24"
+                          Height="24"
                           IsChecked="{Binding KeyframesEnabled}"
                           IsEnabled="{Binding LayerProperty.KeyframesSupported}"
                           VerticalAlignment="Center" Padding="-25">
-                <materialDesign:PackIcon Kind="Stopwatch" Height="13" Width="13" />
+                <materialDesign:PackIcon Kind="Stopwatch" Height="16" Width="16" />
             </ToggleButton>
 
             <TextBlock Grid.Column="1"
@@ -68,7 +69,7 @@
                        Text="{Binding LayerProperty.PropertyDescription.Name}"
                        ToolTip="{Binding LayerProperty.PropertyDescription.Description}"
                        HorizontalAlignment="Left" />
-
+            
             <ContentControl Grid.Column="2" Margin="20 0" s:View.Model="{Binding PropertyInputViewModel, IsAsync=True}">
                 <ContentControl.Resources>
                     <Style TargetType="TextBlock">
@@ -78,17 +79,26 @@
                 </ContentControl.Resources>
             </ContentControl>
 
-            <shared:LockableToggleButton Grid.Column="3"
+            <Button Grid.Column="3"
+                    Command="{s:Action ResetToDefault}" 
+                    Style="{StaticResource MaterialDesignIconButton}"
+                    ToolTip="Reset the property to its default value."
+                    Width="24"
+                    Height="24">
+                <materialDesign:PackIcon Kind="BackupRestore" Height="16" Width="16" />
+            </Button>
+            
+            <shared:LockableToggleButton Grid.Column="4"
                                          Style="{StaticResource DataBindingsButton}"
                                          ToolTip="Change the property's data binding"
-                                         Width="20"
-                                         Height="20"
+                                         Width="24"
+                                         Height="24"
                                          VerticalAlignment="Center"
                                          IsLocked="True"
                                          IsEnabled="{Binding LayerProperty.DataBindingsSupported}"
                                          IsChecked="{Binding HasDataBinding, Mode=OneWay}"
                                          Click="{s:Action ActivateDataBindingViewModel}">
-                <materialDesign:PackIcon Kind="VectorLink" Height="13" Width="13" />
+                <materialDesign:PackIcon Kind="VectorLink" Height="16" Width="16" />
             </shared:LockableToggleButton>
         </Grid>
 
diff --git a/src/Artemis.UI/Screens/ProfileEditor/LayerProperties/Tree/TreePropertyViewModel.cs b/src/Artemis.UI/Screens/ProfileEditor/LayerProperties/Tree/TreePropertyViewModel.cs
index 89f0b0370..bde10777a 100644
--- a/src/Artemis.UI/Screens/ProfileEditor/LayerProperties/Tree/TreePropertyViewModel.cs
+++ b/src/Artemis.UI/Screens/ProfileEditor/LayerProperties/Tree/TreePropertyViewModel.cs
@@ -49,6 +49,12 @@ namespace Artemis.UI.Screens.ProfileEditor.LayerProperties.Tree
                 _profileEditorService.ChangeSelectedDataBinding(LayerProperty);
         }
 
+        public void ResetToDefault()
+        {
+            LayerProperty.ApplyDefaultValue();
+            _profileEditorService.UpdateSelectedProfileElement();
+        }
+
         private void ApplyKeyframesEnabled(bool enable)
         {
             // If enabling keyframes for the first time, add a keyframe with the current value at the current position