Minor Reparent() and SetToolBar() fix for wxFrame.

[wxWidgets.git] / samples / forty / playerdg.cpp

diff --git a/samples/forty/playerdg.cpp b/samples/forty/playerdg.cpp
index aa671d0adcdfce8025c5e15a2d2b693cdbf0eeaf..fa3ea03d046635a980e8bb183a87e0f594a8fe4c 100644 (file)
--- a/samples/forty/playerdg.cpp
+++ b/samples/forty/playerdg.cpp
@@ -37,6 +37,7 @@ BEGIN_EVENT_TABLE(PlayerSelectionDialog, wxDialog)
        EVT_BUTTON(wxID_OK, PlayerSelectionDialog::ButtonCallback)
        EVT_BUTTON(wxID_CANCEL, PlayerSelectionDialog::ButtonCallback)
        EVT_LISTBOX(ID_LISTBOX, PlayerSelectionDialog::SelectCallback)
+    EVT_CLOSE(PlayerSelectionDialog::OnCloseWindow)
 END_EVENT_TABLE()
 
 PlayerSelectionDialog::PlayerSelectionDialog(
@@ -44,7 +45,7 @@ PlayerSelectionDialog::PlayerSelectionDialog(
                                                        ScoreFile* file
                                                        ) :
        wxDialog(parent, -1, "Player Selection",
-                       wxDefaultPosition, wxSize(250, 200),
+                       wxDefaultPosition, wxSize(320, 200),
                        wxDIALOG_MODAL | wxDEFAULT_DIALOG_STYLE),
        m_scoreFile(file)
 {
@@ -59,18 +60,13 @@ PlayerSelectionDialog::PlayerSelectionDialog(
                                                0, 0,
                                                wxLB_SINGLE
                                                );
-/*
-    Robert Roebling
     
-       int numPlayers = 0;
-       wxString* players = 0;
-       m_scoreFile->GetPlayerList(&players, numPlayers);
-       for (int i = 0; i < numPlayers; i++)
+       wxArrayString players;
+       m_scoreFile->GetPlayerList(players);
+       for (unsigned int i = 0; i < players.Count(); i++)
        {
                list->Append(players[i]);
        }
-       delete players;
-*/
 
        m_textField = new wxTextCtrl(this, -1, "", wxDefaultPosition, wxDefaultSize, 0);
 
@@ -133,7 +129,7 @@ PlayerSelectionDialog::~PlayerSelectionDialog()
 {
 }
 
-void PlayerSelectionDialog::OnSize(wxSizeEvent& event)
+void PlayerSelectionDialog::OnSize(wxSizeEvent& WXUNUSED(event))
 {
        Layout();
 }
@@ -147,22 +143,17 @@ const wxString& PlayerSelectionDialog::GetPlayersName()
        return m_player;
 }
 
-bool PlayerSelectionDialog::OnClose()
+void PlayerSelectionDialog::OnCloseWindow(wxCloseEvent& event)
 {
-       // hide the dialog
-       // NB don't return TRUE otherwise delete is called
        m_player = "";
-       Show(FALSE);
-       return FALSE;
+    EndModal(wxID_CANCEL);
 }
 
 void PlayerSelectionDialog::SelectCallback(wxCommandEvent& event)
 {
        if (event.GetEventType() == wxEVT_COMMAND_LISTBOX_SELECTED)
        {
-#ifdef __WXGTK__
-               if (event.IsSelection())
-#endif
+//         if (event.IsSelection())
                m_textField->SetValue(event.GetString());
        }
 }
@@ -181,17 +172,17 @@ void PlayerSelectionDialog::ButtonCallback(wxCommandEvent& event)
                        else
                        {
                                m_player = name;
-                               Show(FALSE);
+                               EndModal(wxID_OK);
                        }
                }
                else
                {
-                       wxMessageBox("Please enter your name", "Forty Thieves");
+                       wxMessageBox("Please enter your name", "Forty Thieves");
                }
        }
        else
        {
                m_player = "";
-               Show(FALSE);
+               EndModal(wxID_CANCEL);
        }
 }