Fix skill icon not unloading.

Moved all help icons on same side.

Daybreak/Controls/Templates/BuildTemplate.xaml

@@ -25,11 +25,11 @@
         <Grid Grid.Row="1">
             <Grid.ColumnDefinitions>
                 <ColumnDefinition Width="auto"></ColumnDefinition>
-                <ColumnDefinition Width="*"></ColumnDefinition>
                 <ColumnDefinition Width="auto"></ColumnDefinition>
+                <ColumnDefinition Width="*"></ColumnDefinition>
             </Grid.ColumnDefinitions>
-            <TextBlock Text="Primary Profession: " FontSize="16" Foreground="White"></TextBlock>
-            <ListView FontSize="16" Foreground="White" Grid.Column="1" BorderThickness="0"
+            <TextBlock Text="Primary Profession: " FontSize="16" Foreground="White" Grid.Column="1"></TextBlock>
+            <ListView FontSize="16" Foreground="White" Grid.Column="2" BorderThickness="0"
                       Background="Transparent" ItemsSource="{Binding ElementName=_this, Path=Professions, Mode=OneWay}"
                       SelectedItem="{Binding ElementName=_this, Path=PrimaryProfession, Mode=TwoWay}" Height="30"
                       SelectionMode="Single" ScrollViewer.VerticalScrollBarVisibility="Disabled"
@@ -43,17 +43,17 @@
                     <behaviors:ScrollIntoView></behaviors:ScrollIntoView>
                 </interactivity:Interaction.Behaviors>
             </ListView>
-            <local:HelpButton Grid.Column="2" Foreground="White" Width="20" Height="20" Margin="3"
+            <local:HelpButton Grid.Column="0" Foreground="White" Width="20" Height="20" Margin="3"
                               Clicked="HelpButtonPrimary_Clicked" Cursor="Hand"></local:HelpButton>
         </Grid>
         <Grid Grid.Row="2">
             <Grid.ColumnDefinitions>
                 <ColumnDefinition Width="auto"></ColumnDefinition>
-                <ColumnDefinition Width="*"></ColumnDefinition>
                 <ColumnDefinition Width="auto"></ColumnDefinition>
+                <ColumnDefinition Width="*"></ColumnDefinition>
             </Grid.ColumnDefinitions>
-            <TextBlock Text="Secondary Profession: " FontSize="16" Foreground="White"></TextBlock>
-            <ListView FontSize="16" Foreground="White" Grid.Column="1" BorderThickness="0"
+            <TextBlock Text="Secondary Profession: " FontSize="16" Foreground="White" Grid.Column="1"></TextBlock>
+            <ListView FontSize="16" Foreground="White" Grid.Column="2" BorderThickness="0"
                       Background="Transparent" ItemsSource="{Binding ElementName=_this, Path=Professions, Mode=OneWay}"
                       SelectedItem="{Binding ElementName=_this, Path=SecondaryProfession, Mode=TwoWay}" Height="30"
                       SelectionMode="Single" ScrollViewer.VerticalScrollBarVisibility="Disabled"
@@ -67,7 +67,7 @@
                     <behaviors:ScrollIntoView></behaviors:ScrollIntoView>
                 </interactivity:Interaction.Behaviors>
             </ListView>
-            <local:HelpButton Grid.Column="2" Foreground="White" Width="20" Height="20" Margin="3"
+            <local:HelpButton Grid.Column="0" Foreground="White" Width="20" Height="20" Margin="3"
                               Clicked="HelpButtonSecondary_Clicked" Cursor="Hand"></local:HelpButton>
         </Grid>
         <Grid Grid.Row="3">

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,15 +48,22 @@ namespace Daybreak.Controls
 
         private void SkillTemplate_DataContextChanged(object sender, DependencyPropertyChangedEventArgs e)
         {
-            if (e.NewValue is Skill)
+            if (e.NewValue is Skill skill)
             {
-                Task.Run(() => GetImageStream(e.NewValue.As<Skill>())).ContinueWith((previousTask) =>
+                if (skill != Skill.NoSkill)
                 {
-                    this.Dispatcher.Invoke(() =>
+                    Task.Run(() => GetImageStream(skill)).ContinueWith((previousTask) =>
                     {
-                        this.ImageSource = GetImageSource(previousTask.Result);
+                        this.Dispatcher.Invoke(() =>
+                        {
+                            this.ImageSource = GetImageSource(previousTask.Result);
+                        });
                     });
-                });
+                }
+                else if (this.ImageSource is not null)
+                {
+                    this.ImageSource = null;
+                }
             }
         }
 

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.7</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)