Merge pull request #3 from intcooper/fix/first_start

Bug fix: the main window is now automatically opened on the first start.
rguida · May 21, 2024 · c12c752 · c12c752
2 parents bef3c84 + 5e2ec0b
commit c12c752
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 8 deletions.
diff --git a/MMExNotifier/App.xaml.cs b/MMExNotifier/App.xaml.cs
@@ -20,9 +20,17 @@ protected override void OnStartup(StartupEventArgs e)
             var view = new MainWindow();
             var viewModel = new MainViewModel(appConfiguration, new NotificationService(new ToastNotification()), new DatabaseService(appConfiguration));
 
+            view.Hide();
             view.DataContext = viewModel;
-            viewModel.OnClose += (s, e) => Application.Current.Dispatcher.Invoke(() => view.Close());
-            viewModel.OnOpen += (s, e) => Application.Current.Dispatcher.Invoke(() => { if (view.Visibility != Visibility.Visible) view.ShowDialog(); });
+            viewModel.OnClose += (s, e) => view.Dispatcher.Invoke(() => view.Close());
+            viewModel.OnOpen += (s, e) => view.Dispatcher.Invoke(() =>
+            {
+                if (view.Visibility != Visibility.Visible)
+                {
+                    view.Visibility = Visibility.Visible;
+                    view.Show();
+                }
+            });
             viewModel.Activate();
         }
     }

diff --git a/MMExNotifier/Views/MainWindow.xaml b/MMExNotifier/Views/MainWindow.xaml
@@ -20,7 +20,6 @@
     Foreground="{DynamicResource TextFillColorPrimaryBrush}"
     Icon="pack://application:,,,/Resources/MMExNotifier.png"
     ResizeMode="CanResizeWithGrip"
-    Visibility="Hidden"
     WindowBackdropType="Mica"
     WindowCornerPreference="Round"
     WindowStartupLocation="Manual"
@@ -278,11 +277,6 @@
                             </Button>
                         </Grid>
 
-                        <CheckBox
-                            Margin="20"
-                            Content="Run MMExNotifier on Windows logon"
-                            IsChecked="{Binding Path=AppSettings.RunAtLogon}" />
-
                         <Grid Margin="20,10">
                             <Grid.ColumnDefinitions>
                                 <ColumnDefinition Width="Auto" />
@@ -327,6 +321,11 @@
                                 Content="days ahead" />
                         </Grid>
 
+                        <CheckBox
+                            Margin="20"
+                            Content="Run MMExNotifier on Windows logon"
+                            IsChecked="{Binding Path=AppSettings.RunAtLogon}" />
+
                     </StackPanel>
                 </Border>
             </Grid>