* Added confirmation messages when saving or resetting settings

CodeDead · CodeDead · commit 67ad92e13e12 · 2017-06-28T04:21:57.000+02:00
diff --git a/Advanced PortChecker/Classes/PortChecker.cs b/Advanced PortChecker/Classes/PortChecker.cs
@@ -162,8 +162,6 @@ private static bool IsUdpOpen(string address, int port, int timeout)
                     udpClient.Client.ReceiveTimeout = timeout;
                     udpClient.Client.SendTimeout = timeout;
 
-
-
                     udpClient.Connect(address, port);
                     return true;
                 }
diff --git a/Advanced PortChecker/Windows/SettingsWindow.xaml b/Advanced PortChecker/Windows/SettingsWindow.xaml
@@ -19,30 +19,35 @@
         </Grid.RowDefinitions>
         <TabControl>
             <TabItem Header="General">
-                <Grid >
+                <Grid HorizontalAlignment="Center" VerticalAlignment="Center">
                     <Grid.RowDefinitions>
                         <RowDefinition Height="Auto"></RowDefinition>
+                        <RowDefinition Height="Auto"></RowDefinition>
                     </Grid.RowDefinitions>
-                    <Grid.ColumnDefinitions>
-                        <ColumnDefinition Width="Auto"></ColumnDefinition>
-                        <ColumnDefinition Width="Auto"></ColumnDefinition>
-                        <ColumnDefinition Width="Auto"></ColumnDefinition>
-                    </Grid.ColumnDefinitions>
-                    <Label Content="Connection time-out:"></Label>
-                    <syncfusion:IntegerTextBox x:Name="IntTimeOut" Grid.Column="1" MinValue="1"></syncfusion:IntegerTextBox>
-                    <Label Grid.Column="2" Content="milliseconds"></Label>
+                    <Grid>
+                        <CheckBox x:Name="ChbAutoUpdate">Automatically check for updates</CheckBox>
+                    </Grid>
+                    <Grid Grid.Row="1">
+                        <Grid.RowDefinitions>
+                            <RowDefinition Height="Auto"></RowDefinition>
+                        </Grid.RowDefinitions>
+                        <Grid.ColumnDefinitions>
+                            <ColumnDefinition Width="Auto"></ColumnDefinition>
+                            <ColumnDefinition Width="Auto"></ColumnDefinition>
+                            <ColumnDefinition Width="Auto"></ColumnDefinition>
+                        </Grid.ColumnDefinitions>
+                        <Label Content="Connection time-out:"></Label>
+                        <syncfusion:IntegerTextBox x:Name="IntTimeOut" Grid.Column="1" MinValue="1"></syncfusion:IntegerTextBox>
+                        <Label Grid.Column="2" Content="milliseconds"></Label>
+                    </Grid>
                 </Grid>
             </TabItem>
             <TabItem Header="Theme">
-                <Grid>
+                <Grid HorizontalAlignment="Center" VerticalAlignment="Center">
                     <Grid.RowDefinitions>
                         <RowDefinition Height="Auto"></RowDefinition>
-                        <RowDefinition Height="Auto"></RowDefinition>
                     </Grid.RowDefinitions>
-                    <Grid HorizontalAlignment="Center">
-                        <CheckBox x:Name="ChbAutoUpdate">Automatically check for updates</CheckBox>
-                    </Grid>
-                    <Grid Grid.Row="1">
+                    <Grid>
                         <Grid.RowDefinitions>
                             <RowDefinition Height="Auto"></RowDefinition>
                             <RowDefinition Height="Auto"></RowDefinition>
diff --git a/Advanced PortChecker/Windows/SettingsWindow.xaml.cs b/Advanced PortChecker/Windows/SettingsWindow.xaml.cs
@@ -65,6 +65,8 @@ private void BtnSave_Click(object sender, RoutedEventArgs e)
 
                 _mw.ChangeVisualStyle();
                 ChangeVisualStyle();
+
+                MessageBox.Show("All settings have been saved!", "Advanced PortChecker", MessageBoxButton.OK, MessageBoxImage.Information);
             }
             catch (Exception ex)
             {
@@ -76,6 +78,11 @@ private void BtnReset_Click(object sender, RoutedEventArgs e)
         {
             try
             {
+                if (MessageBox.Show("Are you sure you want to reset all settings?", "Advanced PortChecker", MessageBoxButton.YesNo, MessageBoxImage.Question) != MessageBoxResult.Yes)
+                {
+                    return;
+                }
+
                 Properties.Settings.Default.Reset();
                 Properties.Settings.Default.Save();
 

Original file line number	Diff line number	Diff line change
`@@ -162,8 +162,6 @@ private static bool IsUdpOpen(string address, int port, int timeout)`
`162`	`162`	`udpClient.Client.ReceiveTimeout = timeout;`
`163`	`163`	`udpClient.Client.SendTimeout = timeout;`
`164`	`164`
`165`		`-`
`166`		`-`
`167`	`165`	`udpClient.Connect(address, port);`
`168`	`166`	`return true;`
`169`	`167`	`}`
Original file line number	Diff line number	Diff line change
`@@ -65,6 +65,8 @@ private void BtnSave_Click(object sender, RoutedEventArgs e)`
`65`	`65`
`66`	`66`	`_mw.ChangeVisualStyle();`
`67`	`67`	`ChangeVisualStyle();`
	`68`	`+`
	`69`	`+ MessageBox.Show("All settings have been saved!", "Advanced PortChecker", MessageBoxButton.OK, MessageBoxImage.Information);`
`68`	`70`	`}`
`69`	`71`	`catch (Exception ex)`
`70`	`72`	`{`
`@@ -76,6 +78,11 @@ private void BtnReset_Click(object sender, RoutedEventArgs e)`
`76`	`78`	`{`
`77`	`79`	`try`
`78`	`80`	`{`
	`81`	`+ if (MessageBox.Show("Are you sure you want to reset all settings?", "Advanced PortChecker", MessageBoxButton.YesNo, MessageBoxImage.Question) != MessageBoxResult.Yes)`
	`82`	`+ {`
	`83`	`+ return;`
	`84`	`+ }`
	`85`	`+`
`79`	`86`	`Properties.Settings.Default.Reset();`
`80`	`87`	`Properties.Settings.Default.Save();`
`81`	`88`