From 70c3badc260cb8f497cf11be112459213b6ac961 Mon Sep 17 00:00:00 2001
From: Andreas Weizel <rpinski@awzhome.de>
Date: Sat, 8 Feb 2014 00:21:20 +0100
Subject: [PATCH] Some visual improvements in AddInManager's GUI (better text
 wrapping, buttons at bottom of list entries etc.).

---
 .../Project/Src/View/AddInManagerView.xaml    |   2 +-
 .../Project/Src/View/AddInsView.xaml          | 146 ++++++++----------
 .../Src/ViewModel/OfflineAddInViewModel.cs    |   4 +-
 3 files changed, 70 insertions(+), 82 deletions(-)

diff --git a/src/AddIns/Misc/AddInManager2/Project/Src/View/AddInManagerView.xaml b/src/AddIns/Misc/AddInManager2/Project/Src/View/AddInManagerView.xaml
index 3c10c88673..d62d1949c7 100644
--- a/src/AddIns/Misc/AddInManager2/Project/Src/View/AddInManagerView.xaml
+++ b/src/AddIns/Misc/AddInManager2/Project/Src/View/AddInManagerView.xaml
@@ -56,7 +56,7 @@
 		</DataTemplate>
 		
 		<DataTemplate x:Key="packageRepositoryItemTemplate" DataType="{x:Type model:PackageRepository}">
-			<TextBlock Text="{Binding Path=NameWithHighlight}">
+			<TextBlock Text="{Binding Path=NameWithHighlight}" ToolTip="{Binding Path=NameWithHighlight}" TextWrapping="Wrap">
 				<TextBlock.Style>
 					<Style TargetType="TextBlock">
 						<Style.Triggers>
diff --git a/src/AddIns/Misc/AddInManager2/Project/Src/View/AddInsView.xaml b/src/AddIns/Misc/AddInManager2/Project/Src/View/AddInsView.xaml
index 5ba8aafd61..d371d3cb2a 100644
--- a/src/AddIns/Misc/AddInManager2/Project/Src/View/AddInsView.xaml
+++ b/src/AddIns/Misc/AddInManager2/Project/Src/View/AddInsView.xaml
@@ -91,6 +91,8 @@
 		
 		<Style x:Key="addInListButtonStyle" TargetType="{x:Type Button}" BasedOn="{x:Static core:GlobalStyles.ButtonStyle}">
 			<Setter Property="Visibility" Value="Collapsed" />
+			<Setter Property="HorizontalAlignment" Value="Left" />
+			<Setter Property="Margin" Value="2,2,2,2" />
 			<Style.Triggers>
 				<MultiDataTrigger>
 					<MultiDataTrigger.Conditions>
@@ -128,16 +130,16 @@
 				<Grid.ColumnDefinitions>
 					<ColumnDefinition Width="60"/>
 					<ColumnDefinition Width="*"/>
-					<ColumnDefinition Width="Auto"/>
 				</Grid.ColumnDefinitions>
 				
-				<Grid.RowDefinitions>
+				<!--<Grid.RowDefinitions>
 					<RowDefinition Height="60"/>
-				</Grid.RowDefinitions>
+				</Grid.RowDefinitions>-->
 				
 				<Image
 					Grid.Column="0"
 					Height="32"
+					VerticalAlignment="Top"
 					Style="{StaticResource AddInImageStyle}" />
 				<Image
 					Grid.Column="0"
@@ -159,79 +161,66 @@
 					Width="16"
 					Style="{StaticResource InstalledTickImageStyle}"
 					Margin="36,0,0,0"/>
-				<StackPanel
-					Grid.Column="1"
-					Margin="4, 0">
-					<TextBlock
-						x:Name="nameTextBlock"
-						Text="{Binding Path=Name}"
-						FontWeight="Bold"/>
-					<TextBlock
-						x:Name="summaryTextBlock"
-						Text="{Binding Path=Summary}"
-						TextWrapping="Wrap"
-						TextTrimming="CharacterEllipsis"
-						MaxHeight="40"/>
-				</StackPanel>
-				<Button
-					x:Name="addPackageButton"
-					Grid.Column="2"
-					Content="{core:Localize AddInManager.ActionInstall}"
-					IsDefault="True"
-					Command="{Binding Path=AddPackageCommand}"
-					VerticalAlignment="Top"
-					Style="{StaticResource addInListButtonStyle}"
-					Margin="2,2,2,2"/>
-				<Button
-					x:Name="updatePackageButton"
-					Grid.Column="2"
-					Content="{core:Localize AddInManager.ActionUpdate}"
-					IsDefault="True"
-					Command="{Binding Path=UpdatePackageCommand}"
-					VerticalAlignment="Top"
-					Style="{StaticResource addInListButtonStyle}"
-					Margin="2,2,2,2"/>
-				<Button
-					x:Name="disablePackageButton"
-					Grid.Column="2"
-					Content="{core:Localize AddInManager.ActionDisable}"
-					IsDefault="True"
-					Command="{Binding Path=DisablePackageCommand}"
-					VerticalAlignment="Top"
-					Style="{StaticResource addInListButtonStyle}"
-					Margin="2,2,2,2"/>
-				<Button
-					x:Name="removePackageButton"
-					Grid.Column="2"
-					Content="{core:Localize AddInManager.ActionUninstall}"
-					Command="{Binding Path=RemovePackageCommand}"
-					VerticalAlignment="Bottom"
-					Style="{StaticResource addInListButtonStyle}"
-					Margin="2,2,2,2"/>
-				<Button
-					x:Name="cancelInstallationButton"
-					Grid.Column="2"
-					Content="{core:Localize AddInManager.ActionCancelInstallation}"
-					Command="{Binding Path=CancelInstallationCommand}"
-					VerticalAlignment="Bottom"
-					Style="{StaticResource addInListButtonStyle}"
-					Margin="2,2,2,2"/>
-				<Button
-					x:Name="cancelUpdateButton"
-					Grid.Column="2"
-					Content="{core:Localize AddInManager.ActionCancelUpdate}"
-					Command="{Binding Path=CancelUpdateCommand}"
-					VerticalAlignment="Bottom"
-					Style="{StaticResource addInListButtonStyle}"
-					Margin="2,2,2,2"/>
-				<Button
-					x:Name="cancelUninstallationButton"
-					Grid.Column="2"
-					Content="{core:Localize AddInManager.ActionCancelDeinstallation}"
-					Command="{Binding Path=CancelUninstallationCommand}"
-					VerticalAlignment="Bottom"
-					Style="{StaticResource addInListButtonStyle}"
-					Margin="2,2,2,2"/>
+				
+				<Grid Grid.Column="1" Margin="0,0,0,0" HorizontalAlignment="Left">
+					<Grid.RowDefinitions>
+						<RowDefinition Height="Auto" />
+						<RowDefinition Height="Auto" />
+					</Grid.RowDefinitions>
+					<StackPanel Grid.Row="0" Margin="4,4,4,4" HorizontalAlignment="Left">
+						<TextBlock
+							x:Name="nameTextBlock"
+							Text="{Binding Path=Name}"
+							FontWeight="Bold"
+							TextWrapping="Wrap"
+							Margin="0,0,0,6"/>
+						<TextBlock
+							x:Name="summaryTextBlock"
+							Text="{Binding Path=Summary}"
+							TextWrapping="Wrap"
+							TextTrimming="CharacterEllipsis"/>
+					</StackPanel>
+					<StackPanel Grid.Row="1" Margin="2,2,2,2" Orientation="Horizontal">
+						<Button
+							x:Name="addPackageButton"
+							Content="{core:Localize AddInManager.ActionInstall}"
+							IsDefault="True"
+							Command="{Binding Path=AddPackageCommand}"
+							Style="{StaticResource addInListButtonStyle}"/>
+						<Button
+							x:Name="updatePackageButton"
+							Content="{core:Localize AddInManager.ActionUpdate}"
+							IsDefault="True"
+							Command="{Binding Path=UpdatePackageCommand}"
+							Style="{StaticResource addInListButtonStyle}"/>
+						<Button
+							x:Name="disablePackageButton"
+							Content="{core:Localize AddInManager.ActionDisable}"
+							IsDefault="True"
+							Command="{Binding Path=DisablePackageCommand}"
+							Style="{StaticResource addInListButtonStyle}"/>
+						<Button
+							x:Name="removePackageButton"
+							Content="{core:Localize AddInManager.ActionUninstall}"
+							Command="{Binding Path=RemovePackageCommand}"
+							Style="{StaticResource addInListButtonStyle}"/>
+						<Button
+							x:Name="cancelInstallationButton"
+							Content="{core:Localize AddInManager.ActionCancelInstallation}"
+							Command="{Binding Path=CancelInstallationCommand}"
+							Style="{StaticResource addInListButtonStyle}"/>
+						<Button
+							x:Name="cancelUpdateButton"
+							Content="{core:Localize AddInManager.ActionCancelUpdate}"
+							Command="{Binding Path=CancelUpdateCommand}"
+							Style="{StaticResource addInListButtonStyle}"/>
+						<Button
+							x:Name="cancelUninstallationButton"
+							Content="{core:Localize AddInManager.ActionCancelDeinstallation}"
+							Command="{Binding Path=CancelUninstallationCommand}"
+							Style="{StaticResource addInListButtonStyle}"/>
+					</StackPanel>
+				</Grid>
 			</Grid>
 			
 			<DataTemplate.Triggers>
@@ -468,12 +457,11 @@
 	
 	<Grid>
 		<Grid.ColumnDefinitions>
-			<ColumnDefinition MinWidth="100"/>
+			<ColumnDefinition MinWidth="200"/>
 			<ColumnDefinition Width="Auto"/>
 			<ColumnDefinition Width=".6*" MinWidth="100"/>
 		</Grid.ColumnDefinitions>
 		
-		
 		<Grid Grid.Column="0">
 			<Grid>
 				<Grid.RowDefinitions>
@@ -495,7 +483,7 @@
 						<RowDefinition Height="Auto"/>
 					</Grid.RowDefinitions>
 					
-					<!--				<TextBlock
+					<!-- <TextBlock
 						Grid.Row="0"
 						Grid.Column="0"
 						VerticalAlignment="Center"
diff --git a/src/AddIns/Misc/AddInManager2/Project/Src/ViewModel/OfflineAddInViewModel.cs b/src/AddIns/Misc/AddInManager2/Project/Src/ViewModel/OfflineAddInViewModel.cs
index 51eeba9062..ea6d0523ec 100644
--- a/src/AddIns/Misc/AddInManager2/Project/Src/ViewModel/OfflineAddInViewModel.cs
+++ b/src/AddIns/Misc/AddInManager2/Project/Src/ViewModel/OfflineAddInViewModel.cs
@@ -267,8 +267,8 @@ namespace ICSharpCode.AddInManager2.ViewModel
 		{
 			get
 			{
-				// Disabling is only possible if this AddIn has an identity!
-				return (Id != null);
+				// Disabling is only possible if this AddIn has an identity and is not the AddInManager itself
+				return (Id != null) && (_addIn.Manifest.PrimaryIdentity != "ICSharpCode.AddInManager2");
 			}
 		}