Remove some out of date comments.

[wxWidgets.git] / src / generic / fontpickerg.cpp

diff --git a/src/generic/fontpickerg.cpp b/src/generic/fontpickerg.cpp
index 06a5c5c539fd8a0a3d9c290edd63756ffe770924..ffcbb91de357d3af9422660e6e6ea17395528686 100644 (file)
--- a/src/generic/fontpickerg.cpp
+++ b/src/generic/fontpickerg.cpp
@@ -35,7 +35,6 @@
 // implementation
 // ============================================================================
 
 // implementation
 // ============================================================================
 

-wxFontData wxGenericFontButton::ms_data;

 IMPLEMENT_DYNAMIC_CLASS(wxGenericFontButton, wxButton)
 
 // ----------------------------------------------------------------------------
 IMPLEMENT_DYNAMIC_CLASS(wxGenericFontButton, wxButton)
 
 // ----------------------------------------------------------------------------


@@ -48,8 +47,8 @@ bool wxGenericFontButton::Create( wxWindow *parent, wxWindowID id,
                         const wxValidator& validator, const wxString &name)
 {
     wxString label = (style & wxFNTP_FONTDESC_AS_LABEL) ?
                         const wxValidator& validator, const wxString &name)
 {
     wxString label = (style & wxFNTP_FONTDESC_AS_LABEL) ?

-                        wxEmptyString : // label will be updated by UpdateFont
-                        wxT("Choose font");
+                        wxString() : // label will be updated by UpdateFont
+                        _("Choose font");

 
     // create this button
     if (!wxButton::Create( parent, id, label, pos,
 
     // create this button
     if (!wxButton::Create( parent, id, label, pos,


@@ -60,11 +59,11 @@ bool wxGenericFontButton::Create( wxWindow *parent, wxWindowID id,
     }
 
     // and handle user clicks on it
     }
 
     // and handle user clicks on it

-    Connect(wxEVT_COMMAND_BUTTON_CLICKED,
+    Connect(GetId(), wxEVT_COMMAND_BUTTON_CLICKED,

             wxCommandEventHandler(wxGenericFontButton::OnButtonClick),
             NULL, this);
 
             wxCommandEventHandler(wxGenericFontButton::OnButtonClick),
             NULL, this);
 

-    m_selectedFont = initial;
+    m_selectedFont = initial.IsOk() ? initial : *wxNORMAL_FONT;

     UpdateFont();
     InitFontData();
 
     UpdateFont();
     InitFontData();
 


@@ -73,22 +72,22 @@ bool wxGenericFontButton::Create( wxWindow *parent, wxWindowID id,
 
 void wxGenericFontButton::InitFontData()
 {
 
 void wxGenericFontButton::InitFontData()
 {

-    ms_data.SetAllowSymbols(true);
-    ms_data.SetColour(*wxBLACK);
-    ms_data.EnableEffects(true);
+    m_data.SetAllowSymbols(true);
+    m_data.SetColour(*wxBLACK);
+    m_data.EnableEffects(true);

 }
 
 void wxGenericFontButton::OnButtonClick(wxCommandEvent& WXUNUSED(ev))
 {
 }
 
 void wxGenericFontButton::OnButtonClick(wxCommandEvent& WXUNUSED(ev))
 {

-    // update the wxFontData to be shown in the the dialog
-    ms_data.SetInitialFont(m_selectedFont);
+    // update the wxFontData to be shown in the dialog
+    m_data.SetInitialFont(m_selectedFont);

 
     // create the font dialog and display it
 
     // create the font dialog and display it

-    wxFontDialog dlg(this, ms_data);
+    wxFontDialog dlg(this, m_data);

     if (dlg.ShowModal() == wxID_OK)
     {
     if (dlg.ShowModal() == wxID_OK)
     {

-        ms_data = dlg.GetFontData();
-        SetSelectedFont(ms_data.GetChosenFont());
+        m_data = dlg.GetFontData();
+        SetSelectedFont(m_data.GetChosenFont());

 
         // fire an event
         wxFontPickerEvent event(this, GetId(), m_selectedFont);
 
         // fire an event
         wxFontPickerEvent event(this, GetId(), m_selectedFont);


@@ -98,10 +97,10 @@ void wxGenericFontButton::OnButtonClick(wxCommandEvent& WXUNUSED(ev))
 
 void wxGenericFontButton::UpdateFont()
 {
 
 void wxGenericFontButton::UpdateFont()
 {

-    if ( !m_selectedFont.Ok() )
+    if ( !m_selectedFont.IsOk() )

         return;
 
         return;
 

-    SetForegroundColour(ms_data.GetColour());
+    SetForegroundColour(m_data.GetColour());

 
     if (HasFlag(wxFNTP_USEFONT_FOR_LABEL))
     {
 
     if (HasFlag(wxFNTP_USEFONT_FOR_LABEL))
     {