From 1b63f68eeebcdb07c90c73452a47b5cc6c9ceab9 Mon Sep 17 00:00:00 2001
From: Ben <ben-wallis@users.noreply.github.com>
Date: Mon, 6 Jul 2015 15:33:25 +0100
Subject: [PATCH] Fixed theme serialization/deserialization and added them
 toolbar icons

---
 Filtration.ObjectModel/ThemeEditor/Theme.cs   |   3 ++
 .../ThemeEditor/ThemeComponent.cs             |  10 ++++-
 .../ViewModels/ThemeEditorViewModel.cs        |   1 +
 .../Views/ThemeComponentControl.xaml          |  42 +++++++++---------
 .../Views/ThemeEditorView.xaml                |   4 +-
 Filtration/Filtration.csproj                  |   2 +
 Filtration/Resources/Icons/Add.ico            | Bin 0 -> 5430 bytes
 .../Resources/Icons/ThemeComponentDelete.ico  | Bin 0 -> 5430 bytes
 Filtration/Views/IconsDictionary.xaml         |   2 +
 Filtration/Views/MainWindow.xaml              |  16 +++----
 10 files changed, 46 insertions(+), 34 deletions(-)
 create mode 100644 Filtration/Resources/Icons/Add.ico
 create mode 100644 Filtration/Resources/Icons/ThemeComponentDelete.ico

diff --git a/Filtration.ObjectModel/ThemeEditor/Theme.cs b/Filtration.ObjectModel/ThemeEditor/Theme.cs
index 954a2c8..91f2722 100644
--- a/Filtration.ObjectModel/ThemeEditor/Theme.cs
+++ b/Filtration.ObjectModel/ThemeEditor/Theme.cs
@@ -2,6 +2,7 @@
 using System.Collections.Generic;
 using System.Linq;
 using System.Windows.Media;
+using System.Xml.Serialization;
 using Filtration.ObjectModel.Enums;
 
 namespace Filtration.ObjectModel.ThemeEditor
@@ -17,6 +18,8 @@ namespace Filtration.ObjectModel.ThemeEditor
         }
 
         public string Name { get; set; }
+
+        [XmlIgnore]
         public string FilePath { get; set; }
 
         public ThemeComponentCollection Components
diff --git a/Filtration.ObjectModel/ThemeEditor/ThemeComponent.cs b/Filtration.ObjectModel/ThemeEditor/ThemeComponent.cs
index 27ab695..dcfbe11 100644
--- a/Filtration.ObjectModel/ThemeEditor/ThemeComponent.cs
+++ b/Filtration.ObjectModel/ThemeEditor/ThemeComponent.cs
@@ -2,6 +2,7 @@
 using System.ComponentModel;
 using System.Runtime.CompilerServices;
 using System.Windows.Media;
+using System.Xml.Serialization;
 using Filtration.ObjectModel.Annotations;
 using Filtration.ObjectModel.Enums;
 
@@ -13,7 +14,12 @@ namespace Filtration.ObjectModel.ThemeEditor
         private Color _color;
         private EventHandler _themeComponentUpdatedEventHandler;
         private readonly object _eventLock = new object();
-        
+
+        public ThemeComponent()
+        {
+            
+        }
+
         public ThemeComponent(ThemeComponentType componentType, string componentName, Color componentColor)
         {
             if (componentName == null || componentColor == null)
@@ -51,7 +57,7 @@ namespace Filtration.ObjectModel.ThemeEditor
         public event EventHandler ThemeComponentDeleted;
 
         public string ComponentName { get; set; }
-        public ThemeComponentType ComponentType{ get; private set; }
+        public ThemeComponentType ComponentType{ get; set; }
 
         public Color Color
         {
diff --git a/Filtration.ThemeEditor/ViewModels/ThemeEditorViewModel.cs b/Filtration.ThemeEditor/ViewModels/ThemeEditorViewModel.cs
index 2c986b0..aab12ce 100644
--- a/Filtration.ThemeEditor/ViewModels/ThemeEditorViewModel.cs
+++ b/Filtration.ThemeEditor/ViewModels/ThemeEditorViewModel.cs
@@ -169,6 +169,7 @@ namespace Filtration.ThemeEditor.ViewModels
                 FilePath = saveDialog.FileName;
                 _themeProvider.SaveTheme(this, FilePath);
                 _filenameIsFake = false;
+                Title = Filename;
                 //RemoveDirtyFlag();
             }
             catch (Exception e)
diff --git a/Filtration.ThemeEditor/Views/ThemeComponentControl.xaml b/Filtration.ThemeEditor/Views/ThemeComponentControl.xaml
index 76a7a5c..e4c5853 100644
--- a/Filtration.ThemeEditor/Views/ThemeComponentControl.xaml
+++ b/Filtration.ThemeEditor/Views/ThemeComponentControl.xaml
@@ -19,10 +19,6 @@
             <RowDefinition Height="Auto" />
             <RowDefinition Height="25" />
         </Grid.RowDefinitions>
-        <Grid.ColumnDefinitions>
-            <ColumnDefinition Width="*" />
-            <ColumnDefinition Width="Auto" />
-        </Grid.ColumnDefinitions>
         <Grid.Resources>
             <DataTemplate x:Key="EditableComponentNameTemplate">
                 <TextBox Text="{Binding ComponentName}" />
@@ -31,23 +27,25 @@
                 <TextBlock Text="{Binding ComponentName}" ToolTip="{Binding ComponentName}" />
             </DataTemplate>
         </Grid.Resources>
-        <TextBlock Grid.Row="0"
-                   Grid.Column="1"
-                   Text="{Binding UsageCount, StringFormat='Usages: {0}'}"
-                   FontSize="10"
-                   Visibility="{Binding Path=DataContext.EditEnabled, RelativeSource={RelativeSource AncestorType={x:Type views:ThemeEditorView}}, Converter={StaticResource BooleanVisibilityConverter}}">
-            <TextBlock.Style>
-                <Style TargetType="TextBlock">
-                    <Style.Triggers>
-                        <DataTrigger Binding="{Binding UsageCount}" Value="0">
-                            <Setter Property="Foreground" Value="Red" />
-                        </DataTrigger>
-                    </Style.Triggers>
-                    <Setter Property="Foreground" Value="SteelBlue"></Setter>
-                </Style>
-            </TextBlock.Style>
-        </TextBlock>
-        <ContentControl Grid.Row="1" Grid.Column="0" Grid.ColumnSpan="2" Content="{Binding}">
+        <DockPanel LastChildFill="True">
+            <TextBlock DockPanel.Dock="Left"
+                       Text="{Binding UsageCount, StringFormat='Usages: {0}'}"
+                       FontSize="10"
+                       HorizontalAlignment="Right"
+                       Visibility="{Binding Path=DataContext.EditEnabled, RelativeSource={RelativeSource AncestorType={x:Type views:ThemeEditorView}}, Converter={StaticResource BooleanVisibilityConverter}}">
+                <TextBlock.Style>
+                    <Style TargetType="TextBlock">
+                        <Style.Triggers>
+                            <DataTrigger Binding="{Binding UsageCount}" Value="0">
+                                <Setter Property="Foreground" Value="Red" />
+                            </DataTrigger>
+                        </Style.Triggers>
+                        <Setter Property="Foreground" Value="SteelBlue" />
+                    </Style>
+                </TextBlock.Style>
+            </TextBlock>
+        </DockPanel>
+        <ContentControl Grid.Row="1" Content="{Binding}">
             <ContentControl.Style>
                 <Style TargetType="ContentControl">
                     <Style.Triggers>
@@ -62,6 +60,6 @@
             </ContentControl.Style>
         </ContentControl>
 
-        <xctk:ColorPicker Grid.Row="2" Grid.Column="0" Grid.ColumnSpan="2"  SelectedColor="{Binding Color}" />
+        <xctk:ColorPicker Grid.Row="2" SelectedColor="{Binding Color}" />
     </Grid>
 </UserControl>
diff --git a/Filtration.ThemeEditor/Views/ThemeEditorView.xaml b/Filtration.ThemeEditor/Views/ThemeEditorView.xaml
index 911a1e0..dc218a4 100644
--- a/Filtration.ThemeEditor/Views/ThemeEditorView.xaml
+++ b/Filtration.ThemeEditor/Views/ThemeEditorView.xaml
@@ -40,7 +40,7 @@
                         <Setter Property="Template">
                             <Setter.Value>
                                 <ControlTemplate TargetType="ListViewItem">
-                                    <Border Name="Border" Padding="2" SnapsToDevicePixels="true">
+                                    <Border Name="Border" BorderBrush="Black" Background="White"  BorderThickness="1" CornerRadius="3" Margin="2" SnapsToDevicePixels="true">
                                         <ContentPresenter />
                                     </Border>
                                     <ControlTemplate.Triggers>
@@ -62,7 +62,7 @@
                     <GroupStyle>
                         <GroupStyle.HeaderTemplate>
                             <DataTemplate>
-                                <TextBlock FontWeight="Bold" FontSize="15" Text="{Binding Path=Name, Converter={StaticResource ThemeComponentTypeToStringConverter}}"/>
+                                <TextBlock Margin="7,5,0,0" FontWeight="Bold" FontSize="15" Text="{Binding Path=Name, Converter={StaticResource ThemeComponentTypeToStringConverter}}"/>
                             </DataTemplate>
                         </GroupStyle.HeaderTemplate>
                     </GroupStyle>
diff --git a/Filtration/Filtration.csproj b/Filtration/Filtration.csproj
index d7eadc4..8d1aff4 100644
--- a/Filtration/Filtration.csproj
+++ b/Filtration/Filtration.csproj
@@ -427,6 +427,8 @@
     <Resource Include="Resources\Icons\ReplaceColors.ico" />
     <Resource Include="Resources\Icons\Theme.ico" />
     <Resource Include="Resources\Icons\filtration_icon.png" />
+    <Resource Include="Resources\Icons\Add.ico" />
+    <Resource Include="Resources\Icons\ThemeComponentDelete.ico" />
     <Content Include="Resources\ItemBaseTypes.txt">
       <CopyToOutputDirectory>Always</CopyToOutputDirectory>
     </Content>
diff --git a/Filtration/Resources/Icons/Add.ico b/Filtration/Resources/Icons/Add.ico
new file mode 100644
index 0000000000000000000000000000000000000000..f81e2cd76faebc9d92d967bd9c681015f41d8ac4
GIT binary patch
literal 5430
zcmeHK?Fqs#6g)q`pLT@&61TA9IDupM*Aa9KM{xuP5aLUVkRGjvq=5(tJlcC-_i`nH
z0t+nBIM~zSxCV9rT-)PY1DwsbH=n&)M(?)=VE%tum+Q&L_UqhnZFn!ep66AV_oMPT
zURu4HhvN#X{gz&@=9??-?PLUd9fo#a0kLuXQy1<TXV#j9`>0#t;;HMe72xy3IQQn5
zk8$PCs{rfGI4?WEe2go9UIkch#wqOFD;J)cF&7{At^DH(uui$~IJ>@E=9;JSPbr}N
z>Ko_|X&IG&`2Wp%q^YZ`nUAsVke0!GjMKX7YUX3CJEUbWALF#{x|;bI>kc)eBn5oD
So?GvQJ%G(OQ}tau;(P<hhx_^f

literal 0
HcmV?d00001

diff --git a/Filtration/Resources/Icons/ThemeComponentDelete.ico b/Filtration/Resources/Icons/ThemeComponentDelete.ico
new file mode 100644
index 0000000000000000000000000000000000000000..24a4b4871d965decb8f3ed357af13bb14a0e9798
GIT binary patch
literal 5430
zcmeI0Z)9X=6vj_F)e)9i`BavjlxV7yrIdb}RZ1yKX_QiyiAg4sOcIu*WD*;Ou&abp
zmZgLe%CanFxfM!SmavpkGb~|Qrk0pt<o0)xd*bZPoja2>Q~P0u<Gkm*&v~BnzW2R<
z5|JT!OD>KIepII47I|MpMz0>lzg!f#L+%4|3_%>_aU$f;`&TR$N2aEx*0Hbp<(`<B
zcnAFE^z`&NzEUoiL&hEKS}zW*1@{nppxDXD$@dtyBaVIIE8RGv+6RiK?=i#C!?M;#
z9M6a^*_`>P_SXorW_k0}Y&wqJQmK@)xmfM4#x`f(dUG6q=-x*8tw^`sV|^LcwqYFf
z^e1uP;-cxLd1}7HURqtGeK)Pf&ry3VUQaJQ&j)M0fYwi5m+@n`8k&P+J!$V$oBjU6
za8^&e1%6nK@fx<Te0pYL{ob}3^JvvPCepYJ#|HKsyQ#fky64Jk=jSQ$XYpDeNB2w-
z+_>iPx?c~`tSG0)4tgAhBKej`?jMnCL*&-B$nUpAf~-iL6Wq!lS)Dk@h^*E{GT>{}
zX{|nIMzO1yK93H6`rvu<do6chet!P);^N}+!oosOsZ_2HqDei&XgtC?XdFL}iAS)#
zrKP3Nj6S|R&mQ!CMB~2l!@cRb$$w<J=E*hb@$f$M*t3pRQ@P=7&(6-i@|;ml@=s0M
zyyj+RW-gpn)9T>)(Q(!+?>+sM4C6huTON1XG4(a$L*rSu^+ea_iDA8m<HUQlpFYsL
z>3O)G-=x)X4}6SP+t=6_u1Z>Md=8!)R;!-)ejJk&gDd(vKkoOw1oyqKqxTo=`@2bc
zNjZ8rzGj|>d<RQ?U!%3|YjjM_w@Er_Ie0eWJP}u$^(B4}Ce@&Q=JN~RE9b}hfwS$`
zq-PiGU7I%#rh$#5t+Ush>R>0%B)JZnCFRE+nWlNI(evDkzn^)y3TVABZtt0U?qeI1
z@9UegAAQBZH%1RT#-HZ(TJ+0%8eDXpS9@shfE&oSnm#+(hpjYy?u}1Rv!;Hr`p#ls
zchj_WPR|K;uJ3NvJd3^Trp;U@K1Xg%uUEh8OTw|2<E$l!bNd*by}7x$55eiZ*o@8T
zx6X;b8I!bpPWz_r2AunzFFpqIH+KB@s{U<BeB0L%H?h=bw%2COSHM4xYsF(UcDlV&
zqHoE2z6Sgzys6JZw98;lTuaZRV=zBs!@lpNv0fK&_ly&--*n}qbD`JTaB)40bziYk
zl2%#{-b>gI*x?z*KfylnHSF2vrjz8+)$j8E^gW8^2s~fw9>m`b%ndX5S6rL?p5lG4
zgBqhXhW(~J;Pq@Tcl%x(R1a@>|KA-5-ETV#8>?Zsy@CbRu!d!W!a=T{@8p}gR=$;O
z7CM=F^<ZeFdiZaKm7y>UGGSO(%N~taBs>tQw&Z%DDXm<6=pYkh!_^=cHi81MsuN`R
EFLU-7^8f$<

literal 0
HcmV?d00001

diff --git a/Filtration/Views/IconsDictionary.xaml b/Filtration/Views/IconsDictionary.xaml
index 4b4c6b6..f4fea97 100644
--- a/Filtration/Views/IconsDictionary.xaml
+++ b/Filtration/Views/IconsDictionary.xaml
@@ -29,4 +29,6 @@
     <Image Source="/Filtration;component/Resources/Icons/Paste.ico" x:Key="PasteIcon" x:Shared="False" />
     <Image Source="/Filtration;component/Resources/Icons/PasteStyle.ico" x:Key="PasteStyleIcon" x:Shared="False" />
     <Image Source="/Filtration;component/Resources/Icons/filtration_icon.png" x:Key="FiltrationIcon" x:Shared="False" />
+    <Image Source="/Filtration;component/Resources/Icons/Add.ico" x:Key="AddIcon" x:Shared="False" />
+    <Image Source="/Filtration;component/Resources/Icons/ThemeComponentDelete.ico" x:Key="ThemeComponentDeleteIcon" x:Shared="False" />
 </ResourceDictionary>
\ No newline at end of file
diff --git a/Filtration/Views/MainWindow.xaml b/Filtration/Views/MainWindow.xaml
index e0e3f27..a6975ca 100644
--- a/Filtration/Views/MainWindow.xaml
+++ b/Filtration/Views/MainWindow.xaml
@@ -10,7 +10,7 @@
         xmlns:views="clr-namespace:Filtration.Views"
         mc:Ignorable="d"
         d:DataContext="{d:DesignInstance Type=viewModels:MainWindowViewModel}"
-        Title="{Binding WindowTitle}" Height="768" Width="1100" BorderThickness="1" BorderBrush="Black" IsIconVisible="True" >
+        Title="{Binding WindowTitle}" Height="762" Width="1126" IsIconVisible="True" >
     <fluent:RibbonWindow.InputBindings>
         <KeyBinding Command="{Binding SaveCommand}" Modifiers="Control" Key="S" />
         <KeyBinding Command="{Binding OpenScriptCommand}" Modifiers="Control" Key="O" />
@@ -64,9 +64,9 @@
             </fluent:Ribbon.ContextualGroups>
             <fluent:RibbonTabItem Header="View">
                 <fluent:RibbonGroupBox Header="Tools">
-                    <fluent:ToggleButton Header="Section Browser" Width="150" SizeDefinition="Middle" Icon="{StaticResource AddSectionIcon}" IsChecked="{Binding AvalonDockWorkspaceViewModel.SectionBrowserViewModel.IsVisible}" />
-                    <fluent:ToggleButton Header="Block Group Browser" Width="150" SizeDefinition="Middle" Icon="{StaticResource BlockGroupBrowserIcon}" IsChecked="{Binding AvalonDockWorkspaceViewModel.BlockGroupBrowserViewModel.IsVisible}" />
-                    <fluent:ToggleButton Header="Block Output Preview" Width="150" SizeDefinition="Middle" Icon="{StaticResource BlockOutputPreviewIcon}" IsChecked="{Binding AvalonDockWorkspaceViewModel.BlockOutputPreviewViewModel.IsVisible}" />
+                    <fluent:ToggleButton Header="Section Browser" Width="150" Size="Middle" Icon="{StaticResource AddSectionIcon}" IsChecked="{Binding AvalonDockWorkspaceViewModel.SectionBrowserViewModel.IsVisible}"/>
+                    <fluent:ToggleButton Header="Block Group Browser" SizeDefinition="Middle" Icon="{StaticResource BlockGroupBrowserIcon}" IsChecked="{Binding AvalonDockWorkspaceViewModel.BlockGroupBrowserViewModel.IsVisible}" />
+                    <fluent:ToggleButton Header="Block Output Preview" SizeDefinition="Middle" Icon="{StaticResource BlockOutputPreviewIcon}" IsChecked="{Binding AvalonDockWorkspaceViewModel.BlockOutputPreviewViewModel.IsVisible}" />
                 </fluent:RibbonGroupBox>
             </fluent:RibbonTabItem>
             <fluent:RibbonTabItem x:Name="ScriptToolsTabItem" Header="Script Tools" Group="{Binding ElementName=ScriptToolsGroup}" Visibility="{Binding ActiveDocumentIsScript, Converter={StaticResource BooleanVisibilityConverter}, Mode=OneWay}" >
@@ -103,12 +103,12 @@
             </fluent:RibbonTabItem>
             <fluent:RibbonTabItem x:Name="ThemeToolsTabItem" Header="Theme Tools" Group="{Binding ElementName=ThemeToolsGroup}" Visibility="{Binding ActiveDocumentIsTheme, Converter={StaticResource BooleanVisibilityConverter}, Mode=OneWay}">
                 <fluent:RibbonGroupBox Header="Add Components">
-                    <fluent:Button SizeDefinition="Middle" Header="Add Text Color" Command="{Binding AddTextColorThemeComponentCommand}" />
-                    <fluent:Button SizeDefinition="Middle" Header="Add Background Color" Command="{Binding AddBackgroundColorThemeComponentCommand}" />
-                    <fluent:Button SizeDefinition="Middle" Header="Add Border Color" Command="{Binding AddBorderColorThemeComponentCommand}" />
+                    <fluent:Button SizeDefinition="Middle" Header="Add Text Color" Icon="{StaticResource AddIcon}" Command="{Binding AddTextColorThemeComponentCommand}" />
+                    <fluent:Button SizeDefinition="Middle" Header="Add Background Color" Icon="{StaticResource AddIcon}" Command="{Binding AddBackgroundColorThemeComponentCommand}" />
+                    <fluent:Button SizeDefinition="Middle" Header="Add Border Color" Icon="{StaticResource AddIcon}" Command="{Binding AddBorderColorThemeComponentCommand}" />
                 </fluent:RibbonGroupBox>
                 <fluent:RibbonGroupBox Header="Delete">
-                    <fluent:Button Header="Delete Theme Component" Command="{Binding DeleteThemeComponentCommand}" />
+                    <fluent:Button Header="Delete Theme Component" Icon="{StaticResource ThemeComponentDeleteIcon}" LargeIcon="{StaticResource ThemeComponentDeleteIcon}" Command="{Binding DeleteThemeComponentCommand}" />
                 </fluent:RibbonGroupBox>
             </fluent:RibbonTabItem>
         </fluent:Ribbon>