* wxwindows: do not save the position of minimized windows. Closes #282.

   We cannot do better (i.e. save the position when the window is minimized)
   because we cannot be notified of such event with the wxWindows API.

modules/gui/wxwindows/bookmarks.cpp

@@ -294,7 +294,7 @@ BookmarksDialog::~BookmarksDialog()
 /*****************************************************************************
  * Private methods.
  *****************************************************************************/
-wxWindow *BookmarksDialog( intf_thread_t *p_intf, wxWindow *p_parent )
+wxFrame *BookmarksDialog( intf_thread_t *p_intf, wxWindow *p_parent )
 {
     return new class BookmarksDialog( p_intf, p_parent );
 }

modules/gui/wxwindows/dialogs.cpp

@@ -88,7 +88,7 @@ public:
     FileInfo            *p_fileinfo_dialog;
     WizardDialog        *p_wizard_dialog;
     wxFrame             *p_prefs_dialog;
-    wxWindow            *p_bookmarks_dialog;
+    wxFrame             *p_bookmarks_dialog;
     wxFileDialog        *p_file_generic_dialog;
     UpdateVLC           *p_updatevlc_dialog;
 };
@@ -200,7 +200,7 @@ DialogsProvider::~DialogsProvider()
 
 #define UPDATE(id,w)                                        \
   {                                                         \
-    if( w && w->IsShown() )                                 \
+    if( w && w->IsShown() && !w->IsIconized() )             \
         ws->SetSettings(  WindowSettings::id, true,         \
                           w->GetPosition(), w->GetSize() ); \
     else                                                    \

modules/gui/wxwindows/interface.cpp

@@ -335,8 +335,11 @@ Interface::~Interface()
 {
     WindowSettings *ws = p_intf->p_sys->p_window_settings;
 
-    ws->SetSettings( WindowSettings::ID_MAIN, true,
-                     GetPosition(), GetSize() );
+    if( !IsIconized() )
+    {
+        ws->SetSettings( WindowSettings::ID_MAIN, true,
+                         GetPosition(), GetSize() );
+    }
 
     if( video_window ) delete video_window;
 

modules/gui/wxwindows/wxwindows.h

@@ -157,7 +157,7 @@ struct intf_sys_t
 wxArrayString SeparateEntries( wxString );
 wxWindow *CreateVideoWindow( intf_thread_t *p_intf, wxWindow *p_parent );
 void UpdateVideoWindow( intf_thread_t *p_intf, wxWindow *p_window );
-wxWindow *BookmarksDialog( intf_thread_t *p_intf, wxWindow *p_parent );
+wxFrame *BookmarksDialog( intf_thread_t *p_intf, wxWindow *p_parent );
 wxWindow *CreateDialogsProvider( intf_thread_t *p_intf, wxWindow *p_parent );
 
 namespace wxvlc