[DYN-5087] Update package details view to include all info (#14851)

* Fix PostDiff job

* Update ShortcutToolbar.xaml.cs

* Add details

* Revert "Update ShortcutToolbar.xaml.cs"

This reverts commit 0a671b2.

* Update ShortcutToolbar.xaml.cs

Author: zeusongit

src/DynamoCoreWpf/Controls/ShortcutToolbar.xaml.cs

@@ -61,7 +61,7 @@ public ShortcutToolbar(DynamoViewModel dynamoViewModel)
                 authManager.LoginStateChanged += AuthChangeHandler;
                 if (authManager.LoginState == LoginState.LoggedIn)
                 {
-                    if(loginMenu.Items.Count == 0)
+                    if (loginMenu.Items.Count == 0)
                     {
                         loginMenu.Items.Add(logoutOption);
                     }

src/DynamoCoreWpf/Properties/Resources.Designer.cs

@@ -3885,7 +3885,19 @@ In certain complex graphs or host program scenarios, Automatic mode may cause in
   <data name="SignOutConfirmationDialogTitle" xml:space="preserve">
     <value>Sign out of Dynamo</value>
   </data>
+  <data name="PackageDetailsGroup" xml:space="preserve">
+    <value>GROUP</value>
+  </data>
+  <data name="PackageDetailsKeywords" xml:space="preserve">
+    <value>KEYWORDS</value>
+  </data>
+  <data name="PackageDetailsSize" xml:space="preserve">
+    <value>Size</value>
+  </data>
+  <data name="PackageDetailsLinks" xml:space="preserve">
+    <value>LINKS</value>
+  </data>
   <data name="ResetChangesWarningPopupMessage" xml:space="preserve">
     <value>Your changes will be lost if you proceed.</value>
   </data>
-</root>
+</root>

src/DynamoCoreWpf/Properties/Resources.en-US.resx

@@ -3872,6 +3872,18 @@ In certain complex graphs or host program scenarios, Automatic mode may cause in
   <data name="SignOutConfirmationDialogTitle" xml:space="preserve">
     <value>Sign out of Dynamo</value>
   </data>
+  <data name="PackageDetailsGroup" xml:space="preserve">
+    <value>GROUP</value>
+  </data>
+  <data name="PackageDetailsKeywords" xml:space="preserve">
+    <value>KEYWORDS</value>
+  </data>
+  <data name="PackageDetailsSize" xml:space="preserve">
+    <value>Size</value>
+  </data>
+  <data name="PackageDetailsLinks" xml:space="preserve">
+    <value>LINKS</value>
+  </data>
   <data name="ResetChangesWarningPopupMessage" xml:space="preserve">
     <value>Your changes will be lost if you proceed.</value>
   </data>

src/DynamoCoreWpf/Properties/Resources.resx

@@ -2076,6 +2076,53 @@ public object ConvertBack(object value, Type targetType, object parameter, Cultu
         }
     }
 
+    public class CopyrightInfoTooltipConverter : IMultiValueConverter
+    {
+        public object Convert(object[] values, Type targetType, object parameter, CultureInfo culture)
+        {
+            if (values[0] == DependencyProperty.UnsetValue || values[1] == DependencyProperty.UnsetValue)
+            {
+                return null;
+            }
+            if (values != null && values.Count() > 0)
+            {
+                var cph = string.IsNullOrEmpty((string)values[0]) ? "N/A" : (string)values[0];
+                var cpy = string.IsNullOrEmpty((string)values[1]) ? "N/A" : (string)values[1];
+                var tooltip = Resources.PackageDetailsCopyRightHolder + ": " + cph + Environment.NewLine +
+                    Resources.PackageDetailsCopyRightYear + ": " + cpy;
+                return tooltip;
+            }
+            return "";
+        }
+
+        public object[] ConvertBack(object value, Type[] targetTypes, object parameter, CultureInfo culture)
+        {
+            throw new NotImplementedException();
+        }
+    }
+
+    public class PackageDetailsLinkCollapseOnEmpty : IMultiValueConverter
+    {
+        public object Convert(object[] values, Type targetType, object parameter, CultureInfo culture)
+        {
+            if (values[0] == DependencyProperty.UnsetValue || values[1] == DependencyProperty.UnsetValue)
+            {
+                return null;
+            }
+            if (values != null && values.Count() > 0)
+            {
+                if (!string.IsNullOrEmpty((string)values[0]) || !string.IsNullOrEmpty((string)values[1]))
+                return Visibility.Visible; ;
+            }
+            return Visibility.Collapsed;
+        }
+
+        public object[] ConvertBack(object value, Type[] targetTypes, object parameter, CultureInfo culture)
+        {
+            throw new NotImplementedException();
+        }
+    }
+
     public sealed class WarningLevelToColorConverter : IValueConverter
     {
         public object Convert(object value, Type targetType, object parameter, CultureInfo culture)

src/DynamoCoreWpf/UI/Converters.cs

@@ -25,6 +25,7 @@ public class PackageDetailItem : NotificationObject
         private bool canInstall;
         private bool isEnabledForInstall;
         private string packageName;
+        private string packageSize;
         private PackageLoader PackageLoader { get; }
 
         #endregion
@@ -135,6 +136,20 @@ public List<string> Packages
             }
         }
 
+        /// <summary>
+        /// The size of the current package version.
+        /// //TODO: Point this property to the package version size after it has been added to the db.
+        /// </summary>
+        public string PackageSize
+        {
+            get => packageSize;
+            set
+            {
+                packageSize = value;
+                RaisePropertyChanged(nameof(PackageSize));
+            }
+        }
+
         /// <summary>
         /// Returs true if package version is not already installed,
         /// false if already installed.
@@ -181,6 +196,8 @@ public PackageDetailItem(string packageName, PackageVersion packageVersion, bool
             this.CopyRightYear = PackageVersion.copyright_year;
             this.CanInstall = canInstall;
             this.IsEnabledForInstall = isEnabledForInstall && canInstall;
+            //TODO: point this property to the package version size after it has been added to the db.
+            this.PackageSize = Dynamo.Properties.Resources.NoneString;
 
 
             // To avoid displaying package self-dependencies.