Fixed a bug where skill templates would try to load the image for No Skill.

Fixed a bug where build template manager would delete saved template.

Daybreak/Controls/Templates/SkillTemplate.xaml

@@ -17,8 +17,11 @@
             <RowDefinition Height="*"></RowDefinition>
             <RowDefinition Height="auto"></RowDefinition>
         </Grid.RowDefinitions>
-        <Image VerticalAlignment="Stretch" HorizontalAlignment="Stretch"
-               Source="{Binding ElementName=_this, Path=ImageSource, Mode=OneWay}"></Image>
+        <Image VerticalAlignment="Top" HorizontalAlignment="Left"
+               Width="{Binding ElementName=_this, Path=ActualWidth, Mode=OneWay}"
+               Height="{Binding ElementName=_this, Path=ActualWidth, Mode=OneWay}"
+               Source="{Binding ElementName=_this, Path=ImageSource, Mode=OneWay}"
+               Stretch="UniformToFill"></Image>
         <Border BorderThickness="5" BorderBrush="Black"></Border>
         <Border BorderThickness="5" BorderBrush="{Binding ElementName=_this, Path=Foreground, Mode=OneWay}"
                 Opacity="{Binding ElementName=_this, Path=BorderOpacity, Mode=OneWay}"></Border>

Daybreak/Controls/Templates/SkillTemplate.xaml.cs

@@ -48,9 +48,9 @@ namespace Daybreak.Controls
 
         private void SkillTemplate_DataContextChanged(object sender, DependencyPropertyChangedEventArgs e)
         {
-            if (e.NewValue is Skill)
+            if (e.NewValue is Skill skill && skill != Skill.NoSkill)
             {
-                Task.Run(() => GetImageStream(e.NewValue.As<Skill>())).ContinueWith((previousTask) =>
+                Task.Run(() => GetImageStream(skill)).ContinueWith((previousTask) =>
                 {
                     this.Dispatcher.Invoke(() =>
                     {

Daybreak/Daybreak.csproj

@@ -9,7 +9,7 @@
     <CopyLocalLockFileAssemblies>false</CopyLocalLockFileAssemblies>
     <LangVersion>preview</LangVersion>
     <ApplicationIcon>Daybreak.ico</ApplicationIcon>
-    <Version>0.7.4</Version>
+    <Version>0.7.6</Version>
     <ApplicationManifest>app.manifest</ApplicationManifest>
   </PropertyGroup>
 

Daybreak/Services/BuildTemplates/BuildTemplateManager.cs

@@ -68,13 +68,12 @@ namespace Daybreak.Services.BuildTemplates
         public void SaveBuild(BuildEntry buildEntry)
         {
             var encodedBuild = this.EncodeTemplate(buildEntry.Build);
-            File.WriteAllText($"{BuildsPath}\\{buildEntry.Name}.txt", encodedBuild);
             if (string.IsNullOrWhiteSpace(buildEntry.PreviousName))
             {
-                return;
+                File.Delete($"{BuildsPath}\\{buildEntry.PreviousName}.txt");
             }
 
-            File.Delete($"{BuildsPath}\\{buildEntry.PreviousName}.txt");
+            File.WriteAllText($"{BuildsPath}\\{buildEntry.Name}.txt", encodedBuild);
         }
 
         public void RemoveBuild(BuildEntry buildEntry)