From abe059917c8b01b544a8a730feece11dd4e75830 Mon Sep 17 00:00:00 2001 From: Martin Wiethan <47688561+Marterich@users.noreply.github.com> Date: Wed, 2 Apr 2025 22:54:44 +0200 Subject: [PATCH 01/15] Optimize UI Performance (#3277) * Optimize UI Load performance * remove timers * Formatting * Remove unused Collapsed Checkbox Style from inputXML.xaml --- config/themes.json | 3 +- .../private/Initalize-InstallAppEntry.ps1 | 166 ++++++------------ .../Initialize-InstallCategoryAppList.ps1 | 21 ++- scripts/main.ps1 | 6 +- xaml/inputXML.xaml | 89 ++++++++-- 5 files changed, 147 insertions(+), 138 deletions(-) diff --git a/config/themes.json b/config/themes.json index 40b0cf64..724cb6af 100644 --- a/config/themes.json +++ b/config/themes.json @@ -35,7 +35,8 @@ "CheckboxMouseOverColor": "#999999", "ButtonBorderThickness": "1", "ButtonMargin": "1", - "ButtonCornerRadius": "2" + "ButtonCornerRadius": "2", + "AppTileImageSize": "40" }, "Light": { "AppInstallUnselectedColor": "#F0F0F0", diff --git a/functions/private/Initalize-InstallAppEntry.ps1 b/functions/private/Initalize-InstallAppEntry.ps1 index 47268a18..09aa732f 100644 --- a/functions/private/Initalize-InstallAppEntry.ps1 +++ b/functions/private/Initalize-InstallAppEntry.ps1 @@ -5,27 +5,19 @@ function Initialize-InstallAppEntry { Used to as part of the Install Tab UI generation .PARAMETER TargetElement The Element into which the Apps should be placed - .PARAMETER AppKey + .PARAMETER appKey The Key of the app inside the $sync.configs.applicationsHashtable #> param( [Windows.Controls.WrapPanel]$TargetElement, - $AppKey + $appKey ) - $App = $sync.configs.applicationsHashtable.$AppKey + # Create the outer Border for the application type $border = New-Object Windows.Controls.Border - $border.BorderBrush = [Windows.Media.Brushes]::Gray - $border.SetResourceReference([Windows.Controls.Control]::BorderThicknessProperty, "AppTileBorderThickness") - $border.CornerRadius = 5 - $border.SetResourceReference([Windows.Controls.Control]::PaddingProperty, "AppTileMargins") - $border.SetResourceReference([Windows.Controls.Control]::WidthProperty, "AppTileWidth") - $border.VerticalAlignment = "Top" - $border.SetResourceReference([Windows.Controls.Control]::MarginProperty, "AppTileMargins") - $border.Cursor = [System.Windows.Input.Cursors]::Hand - $border.SetResourceReference([Windows.Controls.Control]::BackgroundProperty, "AppInstallUnselectedColor") - $border.Tag = $Appkey - $border.ToolTip = $App.description + $border.Style = $sync.Form.Resources.AppTileBorderStyle + $border.Tag = $appKey + $border.ToolTip = $Apps.$appKey.description $border.Add_MouseUp({ $childCheckbox = ($this.Child.Children | Where-Object {$_.Template.TargetType -eq [System.Windows.Controls.Checkbox]})[0] $childCheckBox.isChecked = -not $childCheckbox.IsChecked @@ -47,12 +39,8 @@ function Initialize-InstallAppEntry { # Create the CheckBox, vertically centered $checkBox = New-Object Windows.Controls.CheckBox - $checkBox.Name = $AppKey - $checkBox.Background = "Transparent" - $checkBox.HorizontalAlignment = "Left" - $checkBox.VerticalAlignment = "Center" - $checkBox.SetResourceReference([Windows.Controls.Control]::MarginProperty, "AppTileMargins") - $checkBox.SetResourceReference([Windows.Controls.Control]::StyleProperty, "CollapsedCheckBoxStyle") + $checkBox.Name = $appKey + $checkbox.Style = $sync.Form.Resources.AppTileCheckboxStyle $checkbox.Add_Checked({ Invoke-WPFSelectedAppsUpdate -type "Add" -checkbox $this $borderElement = $this.Parent.Parent @@ -64,7 +52,7 @@ function Initialize-InstallAppEntry { $borderElement = $this.Parent.Parent $borderElement.SetResourceReference([Windows.Controls.Control]::BackgroundProperty, "AppInstallUnselectedColor") }) - $sync.$($checkBox.Name) = $checkBox + # Create a StackPanel for the image and name $imageAndNamePanel = New-Object Windows.Controls.StackPanel $imageAndNamePanel.Orientation = "Horizontal" @@ -73,29 +61,15 @@ function Initialize-InstallAppEntry { # Create the Image and set a placeholder $image = New-Object Windows.Controls.Image # $image.Name = "wpfapplogo" + $App.Name - $image.Width = 40 - $image.Height = 40 - $image.Margin = New-Object Windows.Thickness(0, 0, 10, 0) + $image.Style = $sync.Form.Resources.AppTileImageStyle $image.Source = $noimage # Ensure $noimage is defined in your script - # Clip the image corners - $image.Clip = New-Object Windows.Media.RectangleGeometry - $image.Clip.Rect = New-Object Windows.Rect(0, 0, $image.Width, $image.Height) - $image.Clip.RadiusX = 5 - $image.Clip.RadiusY = 5 - $image.SetResourceReference([Windows.Controls.Control]::VisibilityProperty, "AppTileCompactVisibility") - $imageAndNamePanel.Children.Add($image) | Out-Null # Create the TextBlock for the application name $appName = New-Object Windows.Controls.TextBlock - $appName.Text = $App.Content - $appName.SetResourceReference([Windows.Controls.Control]::FontSizeProperty, "AppTileFontSize") - $appName.FontWeight = [Windows.FontWeights]::Bold - $appName.SetResourceReference([Windows.Controls.Control]::ForegroundProperty, "MainForegroundColor") - $appName.VerticalAlignment = "Center" - $appName.SetResourceReference([Windows.Controls.Control]::MarginProperty, "AppTileMargins") - $appName.Background = "Transparent" + $appName.Style = $sync.Form.Resources.AppTileNameStyle + $appName.Text = $Apps.$appKey.content $imageAndNamePanel.Children.Add($appName) | Out-Null # Add the image and name panel to the Checkbox @@ -107,91 +81,53 @@ function Initialize-InstallAppEntry { # Create the StackPanel for the buttons and dock it to the right $buttonPanel = New-Object Windows.Controls.StackPanel - $buttonPanel.Orientation = "Horizontal" - $buttonPanel.HorizontalAlignment = "Right" - $buttonPanel.VerticalAlignment = "Center" - $buttonPanel.SetResourceReference([Windows.Controls.Control]::MarginProperty, "AppTileMargins") - $buttonPanel.SetResourceReference([Windows.Controls.Control]::VisibilityProperty, "AppTileCompactVisibility") + $buttonPanel.Style = $sync.Form.Resources.AppTileButtonPanelStyle [Windows.Controls.DockPanel]::SetDock($buttonPanel, [Windows.Controls.Dock]::Right) - # Create the "Install" button - $installButton = New-Object Windows.Controls.Button - $installButton.Width = 45 - $installButton.Height = 35 - $installButton.Margin = New-Object Windows.Thickness(0, 0, 10, 0) + # Define the button properties + $buttons = @( + [PSCustomObject]@{ Name = "Install"; Description = "Install or Upgrade the application"; Tooltip = "Install or Upgrade the application"; Icon = [char]0xE118 }, + [PSCustomObject]@{ Name = "Uninstall"; Description = "Uninstall the application"; Tooltip = "Uninstall the application"; Icon = [char]0xE74D }, + [PSCustomObject]@{ Name = "Info"; Description = "Open the application's website in your default browser"; Tooltip = "Open the application's website in your default browser"; Icon = [char]0xE946 } + ) - $installIcon = New-Object Windows.Controls.TextBlock - $installIcon.Text = [char]0xE118 # Install Icon - $installIcon.FontFamily = "Segoe MDL2 Assets" - $installIcon.FontSize = 20 - $installIcon.SetResourceReference([Windows.Controls.Control]::ForegroundProperty, "MainForegroundColor") - $installIcon.Background = "Transparent" - $installIcon.HorizontalAlignment = "Center" - $installIcon.VerticalAlignment = "Center" + # Iterate over each button and create it + foreach ($button in $buttons) { + $newButton = New-Object Windows.Controls.Button + $newButton.Style = $sync.Form.Resources.AppTileButtonStyle + $newButton.Content = $button.Icon + $newButton.ToolTip = $button.Tooltip + $buttonPanel.Children.Add($newButton) | Out-Null - $installButton.Content = $installIcon - $installButton.ToolTip = "Install or Upgrade the application" - $buttonPanel.Children.Add($installButton) | Out-Null - - # Add Click event for the "Install" button - $installButton.Add_Click({ - $appKey = $this.Parent.Parent.Parent.Tag - $appObject = $sync.configs.applicationsHashtable.$appKey - Invoke-WPFInstall -PackagesToInstall $appObject - }) - - # Create the "Uninstall" button - $uninstallButton = New-Object Windows.Controls.Button - $uninstallButton.Width = 45 - $uninstallButton.Height = 35 - - $uninstallIcon = New-Object Windows.Controls.TextBlock - $uninstallIcon.Text = [char]0xE74D # Uninstall Icon - $uninstallIcon.FontFamily = "Segoe MDL2 Assets" - $uninstallIcon.FontSize = 20 - $uninstallIcon.SetResourceReference([Windows.Controls.Control]::ForegroundProperty, "MainForegroundColor") - $uninstallIcon.Background = "Transparent" - $uninstallIcon.HorizontalAlignment = "Center" - $uninstallIcon.VerticalAlignment = "Center" - - $uninstallButton.Content = $uninstallIcon - $buttonPanel.Children.Add($uninstallButton) | Out-Null - - $uninstallButton.ToolTip = "Uninstall the application" - $uninstallButton.Add_Click({ - $appKey = $this.Parent.Parent.Parent.Tag - $appObject = $sync.configs.applicationsHashtable.$appKey - Invoke-WPFUnInstall -PackagesToUninstall $appObject - }) - - # Create the "Info" button - $infoButton = New-Object Windows.Controls.Button - $infoButton.Width = 45 - $infoButton.Height = 35 - $infoButton.Margin = New-Object Windows.Thickness(10, 0, 0, 0) - - $infoIcon = New-Object Windows.Controls.TextBlock - $infoIcon.Text = [char]0xE946 # Info Icon - $infoIcon.FontFamily = "Segoe MDL2 Assets" - $infoIcon.FontSize = 20 - $infoIcon.SetResourceReference([Windows.Controls.Control]::ForegroundProperty, "MainForegroundColor") - $infoIcon.Background = "Transparent" - $infoIcon.HorizontalAlignment = "Center" - $infoIcon.VerticalAlignment = "Center" - - $infoButton.Content = $infoIcon - $infoButton.ToolTip = "Open the application's website in your default browser" - $buttonPanel.Children.Add($infoButton) | Out-Null - - $infoButton.Add_Click({ - $appKey = $this.Parent.Parent.Parent.Tag - $appObject = $sync.configs.applicationsHashtable.$appKey - Start-Process $appObject.link - }) + switch ($button.Name) { + "Install" { + $newButton.Add_Click({ + $appKey = $this.Parent.Parent.Parent.Tag + $appObject = $sync.configs.applicationsHashtable.$appKey + Invoke-WPFInstall -PackagesToInstall $appObject + }) + } + "Uninstall" { + $newButton.Add_Click({ + $appKey = $this.Parent.Parent.Parent.Tag + $appObject = $sync.configs.applicationsHashtable.$appKey + Invoke-WPFUnInstall -PackagesToUninstall $appObject + }) + } + "Info" { + $newButton.Add_Click({ + $appKey = $this.Parent.Parent.Parent.Tag + $appObject = $sync.configs.applicationsHashtable.$appKey + Start-Process $appObject.link + }) + } + } + } # Add the button panel to the DockPanel $dockPanel.Children.Add($buttonPanel) | Out-Null # Add the border to the corresponding Category $TargetElement.Children.Add($border) | Out-Null + return $checkbox } diff --git a/functions/private/Initialize-InstallCategoryAppList.ps1 b/functions/private/Initialize-InstallCategoryAppList.ps1 index 6f2b5b7c..47c6d72e 100644 --- a/functions/private/Initialize-InstallCategoryAppList.ps1 +++ b/functions/private/Initialize-InstallCategoryAppList.ps1 @@ -53,8 +53,17 @@ function Initialize-InstallCategoryAppList { $TargetElement.Dispatcher.Invoke([System.Windows.Threading.DispatcherPriority]::Background, [action]{ $TargetElement.Items.Clear() # Remove the loading message - $categories = $Apps.Values | Select-Object -ExpandProperty category -Unique | Sort-Object - foreach ($category in $categories) { + + # Pre-group apps by category + $appsByCategory = @{} + foreach ($appKey in $Apps.Keys) { + $category = $Apps.$appKey.Category + if (-not $appsByCategory.ContainsKey($category)) { + $appsByCategory[$category] = @() + } + $appsByCategory[$category] += $appKey + } + foreach ($category in $($appsByCategory.Keys | Sort-Object)) { Add-Category -Category $category -TargetElement $TargetElement $wrapPanel = New-Object Windows.Controls.WrapPanel $wrapPanel.Orientation = "Horizontal" @@ -64,9 +73,9 @@ function Initialize-InstallCategoryAppList { $wrapPanel.Visibility = [Windows.Visibility]::Collapsed $wrapPanel.Tag = "CategoryWrapPanel_$category" $null = $TargetElement.Items.Add($wrapPanel) - $Apps.Keys | Where-Object { $Apps.$_.Category -eq $category } | Sort-Object | ForEach-Object { - Initialize-InstallAppEntry -TargetElement $wrapPanel -AppKey $_ - } + $appsByCategory[$category] | ForEach-Object { + $sync.$_ = $(Initialize-InstallAppEntry -TargetElement $wrapPanel -AppKey $_) } - }) + } + }) } diff --git a/scripts/main.ps1 b/scripts/main.ps1 index fe12165a..830567db 100644 --- a/scripts/main.ps1 +++ b/scripts/main.ps1 @@ -46,7 +46,6 @@ class GenericException : Exception { GenericException($Message) : base($Message) {} } - $inputXML = $inputXML -replace 'mc:Ignorable="d"', '' -replace "x:N", 'N' -replace '^ + + + + + + - -