From 0f0c27d039f3ffa8717ba00eaa37b9f024f582bd Mon Sep 17 00:00:00 2001
From: shoushou1106 <100991210+shoushou1106@users.noreply.github.com>
Date: Sat, 20 Jul 2024 22:58:10 +0800
Subject: [PATCH 1/3] Init

---
 FrostyEditor/Windows/ModSettingsWindow.xaml   |  32 +++--
 .../Windows/ModSettingsWindow.xaml.cs         | 124 +++++++++++++++++-
 2 files changed, 138 insertions(+), 18 deletions(-)

diff --git a/FrostyEditor/Windows/ModSettingsWindow.xaml b/FrostyEditor/Windows/ModSettingsWindow.xaml
index 287342905..7486f02d0 100644
--- a/FrostyEditor/Windows/ModSettingsWindow.xaml
+++ b/FrostyEditor/Windows/ModSettingsWindow.xaml
@@ -6,8 +6,8 @@
         xmlns:local="clr-namespace:FrostyEditor.Windows"
         xmlns:ctrl="clr-namespace:Frosty.Controls;assembly=FrostyControls"
         mc:Ignorable="d"
-        Title="Export Mod" Height="530" Width="600"
-        ResizeMode="NoResize" WindowStartupLocation="CenterScreen" SnapsToDevicePixels="True">
+        Title="Export Mod" MinHeight="530" Height="530" MinWidth="600" Width="600"
+        WindowStartupLocation="CenterScreen" SnapsToDevicePixels="True">
     
     <Grid Background="{StaticResource WindowBackground}">
         <Grid.RowDefinitions>
@@ -17,28 +17,35 @@
 
         <Grid Background="{StaticResource ListBackground}">
             <DockPanel LastChildFill="True">
-                <Grid Margin="5" DockPanel.Dock="Top">
-                    <Label Content="Please fill out each of the following sections. Only the first section is mandatory." FontFamily="Global User Interface" HorizontalAlignment="Center" VerticalAlignment="Center" FontWeight="Bold" FontSize="14"/>
-                </Grid>
+                <StackPanel Margin="5" DockPanel.Dock="Top">
+                    <Label HorizontalAlignment="Center" VerticalAlignment="Center">
+                        <TextBlock Text="Please fill out each of the following sections. Only the first 4 fields is mandatory."
+                                   FontWeight="Bold" FontSize="14" FontFamily="Global User Interface" TextWrapping="WrapWithOverflow" Foreground="{StaticResource FontColor}"
+                                   HorizontalAlignment="Center" VerticalAlignment="Center"/>
+                    </Label>
+                    <Label x:Name="ErrorMessageLabel" HorizontalAlignment="Center" VerticalAlignment="Center">
+                        <TextBlock Text="Test" TextWrapping="WrapWithOverflow" HorizontalAlignment="Center" VerticalAlignment="Center" Foreground="Red"/>
+                    </Label>
+                </StackPanel>
                 
-                <Border DockPanel.Dock="Top" Background="{StaticResource ControlBackground}" Height="2"/>
+                <Border DockPanel.Dock="Top" Background="{StaticResource ControlBackground}" Height="2" HorizontalAlignment="Stretch"/>
                 
                 <Grid DockPanel.Dock="Top" Margin="5">
                     <StackPanel>
                         <DockPanel LastChildFill="True">
-                            <Label Content="Title " FontFamily="Global User Interface" Width="75"/>
+                            <Label Content="Title" FontFamily="Global User Interface" Width="75"/>
                             <TextBox x:Name="modTitleTextBox" VerticalContentAlignment="Center" Text="Testing" Margin="1" BorderThickness="1"/>
                         </DockPanel>
                         <DockPanel LastChildFill="True">
-                            <Label Content="Author " FontFamily="Global User Interface" Width="75"/>
+                            <Label Content="Author" FontFamily="Global User Interface" Width="75"/>
                             <TextBox x:Name="modAuthorTextBox" VerticalContentAlignment="Center" Text="Testing" Margin="1" BorderThickness="1"/>
                         </DockPanel>
                         <DockPanel LastChildFill="True">
-                            <Label Content="Version " FontFamily="Global User Interface" Width="75"/>
+                            <Label Content="Version" FontFamily="Global User Interface" Width="75"/>
                             <TextBox Grid.Column="1" x:Name="modVersionTextBox" VerticalContentAlignment="Center" Text="Testing" Margin="1" BorderThickness="1"/>
                         </DockPanel>
                         <DockPanel LastChildFill="True">
-                            <Label Content="Category " FontFamily="Global User Interface" Width="75"/>
+                            <Label Content="Category" FontFamily="Global User Interface" Width="75"/>
                             <Grid>
                                 <Grid.ColumnDefinitions>
                                     <ColumnDefinition Width="0.5*"/>
@@ -57,9 +64,8 @@
                 </Grid>
                 
                 <Grid DockPanel.Dock="Top" Margin="5">
-                    <StackPanel Orientation="Horizontal" HorizontalAlignment="Center">
-                        <ctrl:FrostyImageButton x:Name="iconImageButton" Title="ICON" OnValidate="FrostyImageButton_OnValidate" Margin="0,0,4,0"/>
-                        <Grid Height="88" Width="88" Margin="0,0,16,0"/>
+                    <ctrl:FrostyImageButton x:Name="iconImageButton" HorizontalAlignment="Left" Title="ICON" OnValidate="FrostyImageButton_OnValidate" Margin="16,0,0,0"/>
+                    <StackPanel Orientation="Horizontal" HorizontalAlignment="Right" Margin="0,0,16,0">
                         <ctrl:FrostyImageButton x:Name="ssImageButton1" Title="IMAGE" OnValidate="FrostyImageButton_OnValidate" Margin="0,0,4,0"/>
                         <ctrl:FrostyImageButton x:Name="ssImageButton2" Title="IMAGE" OnValidate="FrostyImageButton_OnValidate" Margin="0,0,4,0"/>
                         <ctrl:FrostyImageButton x:Name="ssImageButton3" Title="IMAGE" OnValidate="FrostyImageButton_OnValidate" Margin="0,0,4,0"/>
diff --git a/FrostyEditor/Windows/ModSettingsWindow.xaml.cs b/FrostyEditor/Windows/ModSettingsWindow.xaml.cs
index 0cfada3bd..ee9fc5bc6 100644
--- a/FrostyEditor/Windows/ModSettingsWindow.xaml.cs
+++ b/FrostyEditor/Windows/ModSettingsWindow.xaml.cs
@@ -6,6 +6,9 @@
 using System.Collections.Generic;
 using System;
 using System.Linq;
+using System.Text;
+using System.Windows.Controls;
+using FrostySdk.IO;
 
 namespace FrostyEditor.Windows
 {
@@ -38,6 +41,8 @@ public ModSettingsWindow(FrostyProject inProject = null)
 
         private void ModSettingsWindow_Loaded(object sender, System.Windows.RoutedEventArgs e)
         {
+            ErrorMessageLabel.Visibility = System.Windows.Visibility.Collapsed;
+
             modCategoryComboBox.ItemsSource = categories;
             
             modTitleTextBox.Text = ModSettings.Title;
@@ -72,20 +77,129 @@ private void cancelButton_Click(object sender, System.Windows.RoutedEventArgs e)
 
         private void saveButton_Click(object sender, System.Windows.RoutedEventArgs e)
         {
-            if (modTitleTextBox.Text == "" || modAuthorTextBox.Text == "" || modCategoryTextBox.Text == "" || modVersionTextBox.Text == "")
+            List<string> errors = new List<string>();
+
+            // Check Title
+            if (String.IsNullOrEmpty(modTitleTextBox.Text))
             {
-                FrostyMessageBox.Show("Title, Author, Category and Version are mandatory fields", "Frosty Editor");
-                return;
+                errors.Add("Title are mandatory fields");
+                modTitleTextBox.BorderBrush = System.Windows.Media.Brushes.Red;
+            }
+            else if (modTitleTextBox.Text.Any(c => c > sbyte.MaxValue))
+            {
+                errors.Add("Title can only contain ASCII characters.");
+                modTitleTextBox.BorderBrush = System.Windows.Media.Brushes.Red;
+            }
+            else
+            {
+                modTitleTextBox.BorderBrush = (System.Windows.Media.Brush)FindResource("ControlBackground");
+            }
+
+            // Check Author
+            if (String.IsNullOrEmpty(modAuthorTextBox.Text))
+            {
+                errors.Add("Author are mandatory fields");
+                modAuthorTextBox.BorderBrush = System.Windows.Media.Brushes.Red;
+            }
+            else if (modAuthorTextBox.Text.Any(c => c > sbyte.MaxValue))
+            {
+                errors.Add("Author can only contain ASCII characters.");
+                modAuthorTextBox.BorderBrush = System.Windows.Media.Brushes.Red;
+            }
+            else
+            {
+                modAuthorTextBox.BorderBrush = (System.Windows.Media.Brush)FindResource("ControlBackground");
             }
 
+            // Check Category
+            if (String.IsNullOrEmpty(modCategoryTextBox.Text))
+            {
+                errors.Add("Category are mandatory fields");
+                modCategoryTextBox.BorderBrush = System.Windows.Media.Brushes.Red;
+            }
+            else if (modCategoryTextBox.Text.Any(c => c > sbyte.MaxValue))
+            {
+                errors.Add("Category can only contain ASCII characters.");
+                modCategoryTextBox.BorderBrush = System.Windows.Media.Brushes.Red;
+            }
+            else
+            {
+                modCategoryTextBox.BorderBrush = (System.Windows.Media.Brush)FindResource("ControlBackground");
+            }
+
+            // Check Version
+            if (String.IsNullOrEmpty(modVersionTextBox.Text))
+            {
+                errors.Add("Version are mandatory fields");
+                modVersionTextBox.BorderBrush = System.Windows.Media.Brushes.Red;
+            }
+            else if (modVersionTextBox.Text.Any(c => c > sbyte.MaxValue))
+            {
+                errors.Add("Version can only contain ASCII characters.");
+                modVersionTextBox.BorderBrush = System.Windows.Media.Brushes.Red;
+            }
+            else
+            {
+                modVersionTextBox.BorderBrush = (System.Windows.Media.Brush)FindResource("ControlBackground");
+            }
+
+            // Check Link
             string[] approvedDomains = { "nexusmods.com", "moddb.com" };
+            if (!String.IsNullOrEmpty(modPageLinkTextBox.Text))
+            {
+                if (!Uri.IsWellFormedUriString(modPageLinkTextBox.Text, UriKind.RelativeOrAbsolute))
+                {
+                    errors.Add("Link needs to be valid");
+                    modPageLinkTextBox.BorderBrush = System.Windows.Media.Brushes.Red;
+                }
+                else if (!approvedDomains.Any(modPageLinkTextBox.Text.Contains))
+                {
+                    errors.Add("Link needs to be nexusmods.com or moddb.com");
+                    modPageLinkTextBox.BorderBrush = System.Windows.Media.Brushes.Red;
+                }
+                else
+                {
+                    if (Uri.TryCreate(modPageLinkTextBox.Text, UriKind.Absolute, out Uri uriResult))
+                        modPageLinkTextBox.Text = uriResult.AbsoluteUri;
 
-            if (modPageLinkTextBox.Text != "" && (!Uri.IsWellFormedUriString(modPageLinkTextBox.Text, UriKind.RelativeOrAbsolute) || !approvedDomains.Any(modPageLinkTextBox.Text.Contains)))
+                    if (modPageLinkTextBox.Text.Any(c => c > sbyte.MaxValue))
+                    {
+                        errors.Add("Link can only contain ASCII characters.");
+                        modPageLinkTextBox.BorderBrush = System.Windows.Media.Brushes.Red;
+                    }
+                    else
+                    {
+                        modPageLinkTextBox.BorderBrush = (System.Windows.Media.Brush)FindResource("ControlBackground");
+                    }
+                }
+            }
+            else
+            {
+                modPageLinkTextBox.BorderBrush = (System.Windows.Media.Brush)FindResource("ControlBackground");
+            }
+
+            // Check Description
+            if (!String.IsNullOrEmpty(modDescriptionTextBox.Text) &&
+                !modDescriptionTextBox.Text.Any(c => c > sbyte.MaxValue))
+            {
+                errors.Add("Description can only contain ASCII characters.");
+                modDescriptionTextBox.BorderBrush = System.Windows.Media.Brushes.Red;
+            }
+            else
+            {
+                modDescriptionTextBox.BorderBrush = (System.Windows.Media.Brush)FindResource("ControlBackground");
+            }
+
+            // Check if error exits
+            if (errors.Any())
             {
-                FrostyMessageBox.Show("Link needs to be valid", "Frosty Editor");
+                ErrorMessageLabel.Visibility = System.Windows.Visibility.Visible;
+                (ErrorMessageLabel.Content as TextBlock).Text = String.Join(Environment.NewLine, errors);
                 return;
             }
 
+            ErrorMessageLabel.Visibility = System.Windows.Visibility.Collapsed;
+
             ModSettings.Title = modTitleTextBox.Text;
             ModSettings.Author = modAuthorTextBox.Text;
             ModSettings.Category = modCategoryTextBox.Text;

From 8680b0b29896814da6f859881713cac96e8d2211 Mon Sep 17 00:00:00 2001
From: shoushou1106 <100991210+shoushou1106@users.noreply.github.com>
Date: Sat, 20 Jul 2024 22:59:52 +0800
Subject: [PATCH 2/3] Clean up

---
 FrostyEditor/Windows/ModSettingsWindow.xaml.cs | 2 --
 1 file changed, 2 deletions(-)

diff --git a/FrostyEditor/Windows/ModSettingsWindow.xaml.cs b/FrostyEditor/Windows/ModSettingsWindow.xaml.cs
index ee9fc5bc6..a866e82e3 100644
--- a/FrostyEditor/Windows/ModSettingsWindow.xaml.cs
+++ b/FrostyEditor/Windows/ModSettingsWindow.xaml.cs
@@ -6,9 +6,7 @@
 using System.Collections.Generic;
 using System;
 using System.Linq;
-using System.Text;
 using System.Windows.Controls;
-using FrostySdk.IO;
 
 namespace FrostyEditor.Windows
 {

From 499a4ae320803e57339803b1743d1535948c5c8e Mon Sep 17 00:00:00 2001
From: shoushou <shoushou@shoushou1106.org>
Date: Sun, 24 Nov 2024 19:18:26 -0800
Subject: [PATCH 3/3] Fix grammar

---
 .../Windows/ModSettingsWindow.xaml.cs         | 20 +++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/FrostyEditor/Windows/ModSettingsWindow.xaml.cs b/FrostyEditor/Windows/ModSettingsWindow.xaml.cs
index a866e82e3..80670137d 100644
--- a/FrostyEditor/Windows/ModSettingsWindow.xaml.cs
+++ b/FrostyEditor/Windows/ModSettingsWindow.xaml.cs
@@ -80,12 +80,12 @@ private void saveButton_Click(object sender, System.Windows.RoutedEventArgs e)
             // Check Title
             if (String.IsNullOrEmpty(modTitleTextBox.Text))
             {
-                errors.Add("Title are mandatory fields");
+                errors.Add("Title is a mandatory field");
                 modTitleTextBox.BorderBrush = System.Windows.Media.Brushes.Red;
             }
             else if (modTitleTextBox.Text.Any(c => c > sbyte.MaxValue))
             {
-                errors.Add("Title can only contain ASCII characters.");
+                errors.Add("Title can only contain ASCII characters");
                 modTitleTextBox.BorderBrush = System.Windows.Media.Brushes.Red;
             }
             else
@@ -96,12 +96,12 @@ private void saveButton_Click(object sender, System.Windows.RoutedEventArgs e)
             // Check Author
             if (String.IsNullOrEmpty(modAuthorTextBox.Text))
             {
-                errors.Add("Author are mandatory fields");
+                errors.Add("Author is a mandatory field");
                 modAuthorTextBox.BorderBrush = System.Windows.Media.Brushes.Red;
             }
             else if (modAuthorTextBox.Text.Any(c => c > sbyte.MaxValue))
             {
-                errors.Add("Author can only contain ASCII characters.");
+                errors.Add("Author can only contain ASCII characters");
                 modAuthorTextBox.BorderBrush = System.Windows.Media.Brushes.Red;
             }
             else
@@ -112,12 +112,12 @@ private void saveButton_Click(object sender, System.Windows.RoutedEventArgs e)
             // Check Category
             if (String.IsNullOrEmpty(modCategoryTextBox.Text))
             {
-                errors.Add("Category are mandatory fields");
+                errors.Add("Category is a mandatory field");
                 modCategoryTextBox.BorderBrush = System.Windows.Media.Brushes.Red;
             }
             else if (modCategoryTextBox.Text.Any(c => c > sbyte.MaxValue))
             {
-                errors.Add("Category can only contain ASCII characters.");
+                errors.Add("Category can only contain ASCII characters");
                 modCategoryTextBox.BorderBrush = System.Windows.Media.Brushes.Red;
             }
             else
@@ -128,12 +128,12 @@ private void saveButton_Click(object sender, System.Windows.RoutedEventArgs e)
             // Check Version
             if (String.IsNullOrEmpty(modVersionTextBox.Text))
             {
-                errors.Add("Version are mandatory fields");
+                errors.Add("Version is a mandatory field");
                 modVersionTextBox.BorderBrush = System.Windows.Media.Brushes.Red;
             }
             else if (modVersionTextBox.Text.Any(c => c > sbyte.MaxValue))
             {
-                errors.Add("Version can only contain ASCII characters.");
+                errors.Add("Version can only contain ASCII characters");
                 modVersionTextBox.BorderBrush = System.Windows.Media.Brushes.Red;
             }
             else
@@ -162,7 +162,7 @@ private void saveButton_Click(object sender, System.Windows.RoutedEventArgs e)
 
                     if (modPageLinkTextBox.Text.Any(c => c > sbyte.MaxValue))
                     {
-                        errors.Add("Link can only contain ASCII characters.");
+                        errors.Add("Link can only contain ASCII characters");
                         modPageLinkTextBox.BorderBrush = System.Windows.Media.Brushes.Red;
                     }
                     else
@@ -180,7 +180,7 @@ private void saveButton_Click(object sender, System.Windows.RoutedEventArgs e)
             if (!String.IsNullOrEmpty(modDescriptionTextBox.Text) &&
                 !modDescriptionTextBox.Text.Any(c => c > sbyte.MaxValue))
             {
-                errors.Add("Description can only contain ASCII characters.");
+                errors.Add("Description can only contain ASCII characters");
                 modDescriptionTextBox.BorderBrush = System.Windows.Media.Brushes.Red;
             }
             else