From a2e081ccdbef19915d7f7ba2ad0b5ec9efee3e1e Mon Sep 17 00:00:00 2001
From: Andre Beging <git@troogs.de>
Date: Sat, 16 Sep 2017 15:47:11 +0200
Subject: [PATCH] Visual improvements

---
 Converters/AmountToColorConverter.cs | 4 ++--
 Helper/PrintHelper.cs                | 4 ++++
 View/MainView.xaml                   | 4 ++--
 View/MainView.xaml.cs                | 4 ++--
 ViewModel/MainViewModel.cs           | 2 +-
 5 files changed, 11 insertions(+), 7 deletions(-)

diff --git a/Converters/AmountToColorConverter.cs b/Converters/AmountToColorConverter.cs
index c6f9887..3017f99 100644
--- a/Converters/AmountToColorConverter.cs
+++ b/Converters/AmountToColorConverter.cs
@@ -42,9 +42,9 @@ namespace DebtMgr.Converters
             }
 
             if (doubleValue < 0.001 && doubleValue > -0.001)
-                return new SolidColorBrush(Colors.Green);
+                return new SolidColorBrush(Colors.Black);
 
-            if(doubleValue > 0)
+            if (doubleValue > 0)
                 return new SolidColorBrush(Colors.Green);
 
             if (doubleValue < 0)
diff --git a/Helper/PrintHelper.cs b/Helper/PrintHelper.cs
index 6d616af..533fe95 100644
--- a/Helper/PrintHelper.cs
+++ b/Helper/PrintHelper.cs
@@ -34,6 +34,10 @@ namespace DebtMgr.Helper
                 96, // These decides the dpi factors 
                 96,// The can be changed when we'll have preview options.
                 PixelFormats.Pbgra32);
+
+
+
+
             bitmap.Render(uiElement);
 
             SaveUsingBitmapTargetRenderer(fileName, bitmap, encoder);
diff --git a/View/MainView.xaml b/View/MainView.xaml
index 8831618..9e52e0c 100644
--- a/View/MainView.xaml
+++ b/View/MainView.xaml
@@ -89,7 +89,7 @@
                             <MenuItem Header="Add _Charge" Command="{Binding AddChargeContextMenuCommand}" />
                             <MenuItem Header="Add _Deposit" Command="{Binding AddDepositContextMenuCommand}" />
                             <Separator></Separator>
-                            <MenuItem Header="_Print" Click="PrintContextMenu_OnClick" IsEnabled="False" />
+                            <MenuItem Header="_Print" Click="PrintContextMenu_OnClick" IsEnabled="False"/>
                         </ContextMenu>
                     </ListBox.Resources>
                     <ListBox.ContextMenu>
@@ -104,7 +104,7 @@
                 <Separator Style="{StaticResource {x:Static ToolBar.SeparatorStyleKey}}" />
             </StackPanel>
             
-            <Grid Grid.Column="2">
+            <Grid Grid.Column="2" Name="RightPanel">
                 <Grid.RowDefinitions>
                     <RowDefinition Height="Auto"></RowDefinition>
                     <RowDefinition Height="Auto"></RowDefinition>
diff --git a/View/MainView.xaml.cs b/View/MainView.xaml.cs
index 102c717..b7d87b3 100644
--- a/View/MainView.xaml.cs
+++ b/View/MainView.xaml.cs
@@ -108,9 +108,9 @@ namespace DebtMgr.View
 
         private void PrintContextMenu_OnClick(object sender, RoutedEventArgs e)
         {
-            //var targetElement = TransactionHistoryListView;
+            var targetElement = RightPanel;
 
-            //PrintHelper.SaveUsingEncoder("test.png", targetElement);
+            PrintHelper.SaveUsingEncoder("test.png", targetElement);
             return;
         }
     }
diff --git a/ViewModel/MainViewModel.cs b/ViewModel/MainViewModel.cs
index 8e44cc8..25f4c17 100644
--- a/ViewModel/MainViewModel.cs
+++ b/ViewModel/MainViewModel.cs
@@ -517,7 +517,7 @@ namespace DebtMgr.ViewModel
             // Remember selection
             var rememberSelection = PersonListViewSelectedItem?.Id;
 
-            var personList = App.Database.GetAllWithChildren<Person>();
+            var personList = App.Database.GetAllWithChildren<Person>().OrderBy(x => x.FirstName).ThenBy(x => x.LastName).ToList();
             PersonListViewItemSource = personList;
 
             var overallBalance = personList.Sum(x => x.Total);