]> git.saurik.com Git - wxWidgets.git/blobdiff - src/generic/prntdlgg.cpp
projects / wxWidgets.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | inline | side by side
compilation warning fix
[wxWidgets.git] / src / generic / prntdlgg.cpp
diff --git a/src/generic/prntdlgg.cpp b/src/generic/prntdlgg.cpp
index 7aec75e74ba6820fd3015080eeb4fe6c21b101ea..28d12521593b7ac6950e3c2c2d0d4813fe25ea51 100644 (file)
--- a/src/generic/prntdlgg.cpp
+++ b/src/generic/prntdlgg.cpp
@@ -6,153 +6,242 @@
 // Created:     04/01/98
 // RCS-ID:      $Id$
 // Copyright:   (c) Julian Smart and Markus Holzem
 // Created:     04/01/98
 // RCS-ID:      $Id$
 // Copyright:   (c) Julian Smart and Markus Holzem
-// Licence:    wxWindows license
+// Licence:     wxWindows license
 /////////////////////////////////////////////////////////////////////////////
 
 /////////////////////////////////////////////////////////////////////////////
 
+// ============================================================================
+// declarations
+// ============================================================================
+
+// ----------------------------------------------------------------------------
+// headers
+// ----------------------------------------------------------------------------
+
 #ifdef __GNUG__
 #ifdef __GNUG__
-#pragma implementation "prntdlgg.h"
+    #pragma implementation "prntdlgg.h"
 #endif
 
 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
 
 #ifdef __BORLANDC__
 #endif
 
 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
 
 #ifdef __BORLANDC__
-#pragma hdrstop
+    #pragma hdrstop
 #endif
 
 #endif
 
-#include "wx/defs.h"
-
-#define WINDOWS_PRINTING    (wxTheApp->GetPrintMode() == wxPRINT_WINDOWS)
+#if wxUSE_PRINTING_ARCHITECTURE
 
 #ifndef WX_PRECOMP
 
 #ifndef WX_PRECOMP
-#include "wx/utils.h"
-#include "wx/dc.h"
-#include "wx/app.h"
-#include "wx/frame.h"
-#include "wx/stattext.h"
-#include "wx/button.h"
-#include "wx/checkbox.h"
-#include "wx/textctrl.h"
-#include "wx/radiobox.h"
-#include "wx/filedlg.h"
-#include "wx/choice.h"
-#include <wx/intl.h>
+    #include "wx/utils.h"
+    #include "wx/dc.h"
+    #include "wx/app.h"
+    #include "wx/frame.h"
+    #include "wx/stattext.h"
+    #include "wx/statbox.h"
+    #include "wx/button.h"
+    #include "wx/checkbox.h"
+    #include "wx/textctrl.h"
+    #include "wx/radiobox.h"
+    #include "wx/filedlg.h"
+    #include "wx/choice.h"
+    #include "wx/combobox.h"
+    #include "wx/intl.h"
+    #include "wx/sizer.h"
+#endif
+
+#if wxUSE_STATLINE
+  #include "wx/statline.h"
 #endif
 
 #include "wx/generic/prntdlgg.h"
 #endif
 
 #include "wx/generic/prntdlgg.h"
+
+#if wxUSE_POSTSCRIPT
+    #include "wx/generic/dcpsg.h"
+#endif
+
 #include "wx/printdlg.h"
 #include "wx/printdlg.h"
+#include "wx/paper.h"
+
+// For print paper things
+#include "wx/prntbase.h"
 
 #include <stdlib.h>
 #include <string.h>
 
 
 #include <stdlib.h>
 #include <string.h>
 
-#if !USE_SHARED_LIBRARY
-IMPLEMENT_CLASS(wxGenericPrintDialog, wxDialog)
-IMPLEMENT_CLASS(wxGenericPrintSetupDialog, wxDialog)
-IMPLEMENT_CLASS(wxGenericPageSetupDialog, wxDialog)
+// ----------------------------------------------------------------------------
+// wxWin macros
+// ----------------------------------------------------------------------------
+
 
 
-BEGIN_EVENT_TABLE(wxGenericPrintDialog, wxDialog)
-    EVT_BUTTON(wxID_OK, wxGenericPrintDialog::OnOK)
-    EVT_BUTTON(wxPRINTID_SETUP, wxGenericPrintDialog::OnSetup)
-    EVT_RADIOBOX(wxPRINTID_RANGE, wxGenericPrintDialog::OnRange)
-END_EVENT_TABLE()
+#if wxUSE_POSTSCRIPT
 
 
-BEGIN_EVENT_TABLE(wxGenericPageSetupDialog, wxDialog)
-    EVT_BUTTON(wxPRINTID_SETUP, wxGenericPageSetupDialog::OnPrinter)
-END_EVENT_TABLE()
+    IMPLEMENT_CLASS(wxGenericPrintDialog, wxDialog)
+    IMPLEMENT_CLASS(wxGenericPrintSetupDialog, wxDialog)
+
+    BEGIN_EVENT_TABLE(wxGenericPrintDialog, wxDialog)
+        EVT_BUTTON(wxID_OK, wxGenericPrintDialog::OnOK)
+        EVT_BUTTON(wxPRINTID_SETUP, wxGenericPrintDialog::OnSetup)
+        EVT_RADIOBOX(wxPRINTID_RANGE, wxGenericPrintDialog::OnRange)
+    END_EVENT_TABLE()
 #endif
 
 #endif
 
+    IMPLEMENT_CLASS(wxGenericPageSetupDialog, wxDialog)
+
+    BEGIN_EVENT_TABLE(wxGenericPageSetupDialog, wxDialog)
+        EVT_BUTTON(wxPRINTID_SETUP, wxGenericPageSetupDialog::OnPrinter)
+    END_EVENT_TABLE()
+
+// ----------------------------------------------------------------------------
+// global vars
+// ----------------------------------------------------------------------------
+
 extern wxPrintPaperDatabase *wxThePrintPaperDatabase;
 
 extern wxPrintPaperDatabase *wxThePrintPaperDatabase;
 
-/*
- * Generic print dialog for non-Windows printing use.
- */
+#if wxUSE_POSTSCRIPT
 
 
+// ============================================================================
+// implementation
+// ============================================================================
 
 
-wxGenericPrintDialog::wxGenericPrintDialog(wxWindow *parent, wxPrintData* data):
-  wxDialog(parent, -1, _("Print"), wxPoint(0, 0), wxSize(600, 600), wxDEFAULT_DIALOG_STYLE|wxDIALOG_MODAL)
+// ----------------------------------------------------------------------------
+// Generic print dialog for non-Windows printing use.
+// ----------------------------------------------------------------------------
+
+wxGenericPrintDialog::wxGenericPrintDialog(wxWindow *parent,
+                                           wxPrintDialogData* data)
+                    : wxDialog(parent, -1, _("Print"),
+                               wxPoint(0, 0), wxSize(600, 600),
+                               wxDEFAULT_DIALOG_STYLE |
+                               wxDIALOG_MODAL |
+                               wxTAB_TRAVERSAL)
 {
 {
-  if ( data )
-    printData = *data;
-  
-  int buttonWidth = 65;
-  int buttonHeight = 25;
-  int spacing = 5;
-  int yPos = 5;
-  int xPos = 5;
-
-  wxButton *okButton = new wxButton(this, wxID_OK, _("OK"), wxPoint(5, yPos), wxSize(buttonWidth, buttonHeight));
-  (void) new wxButton(this, wxID_CANCEL, _("Cancel"), wxPoint(buttonWidth + 5 + spacing, yPos), wxSize(buttonWidth, buttonHeight));
-
-  setupButton = new wxButton(this, wxPRINTID_SETUP, _("Setup..."), wxPoint(buttonWidth*2 + 5 + 2*spacing, yPos), wxSize(buttonWidth, buttonHeight));
-
-  okButton->SetDefault();
-  okButton->SetFocus();
-
-  yPos += 35;
-
-  wxString choices[2];
-  choices[0] = _("All");
-  choices[1] = _("Pages");
-  
-  fromText = (wxTextCtrl*)NULL;
-  
-  if(printData.GetFromPage() != 0)
-  {
-     rangeRadioBox = new wxRadioBox(this, wxPRINTID_RANGE, _("Print Range"),
-                                    wxPoint(5, yPos), wxSize(-1, -1), 2, choices, 2);
-     rangeRadioBox->SetSelection(1);
-  }
-  
-  yPos += 60;
-  xPos = 5;
-  int staticWidth = 45;
-  int textWidth = 40;
-  spacing = 10;
-
-  if(printData.GetFromPage() != 0)
-  {
-     (void) new wxStaticText(this, wxPRINTID_STATIC, _("From:"), wxPoint(xPos, yPos));
-     xPos += staticWidth;
-
-     fromText = new wxTextCtrl(this, wxPRINTID_FROM, "", wxPoint(xPos, yPos), wxSize(textWidth, -1));
-     xPos += spacing + textWidth;
-     
-     (void) new wxStaticText(this, wxPRINTID_STATIC, _("To:"), wxPoint(xPos, yPos));
-     xPos += staticWidth;
-     
-     toText = new wxTextCtrl(this, wxPRINTID_TO, "", wxPoint(xPos, yPos), wxSize(textWidth, -1));
-     xPos += spacing + textWidth;
-  }
-  
-  (void) new wxStaticText(this, wxPRINTID_STATIC, _("Copies:"), wxPoint(xPos, yPos));
-  xPos += spacing + staticWidth;
-
-  noCopiesText = new wxTextCtrl(this, wxPRINTID_COPIES, "", wxPoint(xPos, yPos), wxSize(textWidth, -1));
-
-  yPos += 30;
-  xPos = 5;
-
-  printToFileCheckBox = new wxCheckBox(this, wxPRINTID_PRINTTOFILE, _("Print to File"), wxPoint(xPos, yPos));
-
-  Fit();
-  Centre(wxBOTH);
-
-  // Calls wxWindow::OnInitDialog and then wxGenericPrintDialog::TransferDataToWindow
-  InitDialog();
+    if ( data )
+        m_printDialogData = *data;
+
+    Init(parent);
 }
 
 }
 
-int wxGenericPrintDialog::ShowModal(void)
+wxGenericPrintDialog::wxGenericPrintDialog(wxWindow *parent,
+                                           wxPrintData* data)
+                    : wxDialog(parent, -1, _("Print"),
+                               wxPoint(0, 0), wxSize(600, 600),
+                               wxDEFAULT_DIALOG_STYLE |
+                               wxDIALOG_MODAL |
+                               wxTAB_TRAVERSAL)
 {
 {
-    if ( printData.GetSetupDialog() )
+    if ( data )
+        m_printDialogData = *data;
+
+    Init(parent);
+}
+
+void wxGenericPrintDialog::Init(wxWindow * WXUNUSED(parent))
+{
+  //    wxDialog::Create(parent, -1, _("Print"), wxPoint(0, 0), wxSize(600, 600),
+  //                     wxDEFAULT_DIALOG_STYLE | wxDIALOG_MODAL | wxTAB_TRAVERSAL);
+
+    wxBoxSizer *mainsizer = new wxBoxSizer( wxVERTICAL );
+
+    // 1) top row 
+        
+    wxStaticBoxSizer *topsizer = new wxStaticBoxSizer( 
+        new wxStaticBox( this, -1, _( "Printer options" ) ), wxHORIZONTAL );
+    m_printToFileCheckBox = new wxCheckBox( this, wxPRINTID_PRINTTOFILE, _("Print to File") );
+    topsizer->Add( m_printToFileCheckBox, 0, wxCENTER|wxALL, 5 );
+
+    topsizer->Add( 60,2,1 );
+
+    m_setupButton = new wxButton(this, wxPRINTID_SETUP, _("Setup...") );
+    topsizer->Add( m_setupButton, 0, wxCENTER|wxALL, 5 );
+    
+    mainsizer->Add( topsizer, 0, wxLEFT|wxTOP|wxRIGHT, 10 );
+    
+    // 2) middle row with radio box
+    
+    wxString *choices = new wxString[2];
+    choices[0] = _("All");
+    choices[1] = _("Pages");
+
+    m_fromText = (wxTextCtrl*)NULL;
+    m_toText = (wxTextCtrl*)NULL;
+    m_rangeRadioBox = (wxRadioBox *)NULL;
+    
+    if (m_printDialogData.GetFromPage() != 0)
+    {
+        m_rangeRadioBox = new wxRadioBox(this, wxPRINTID_RANGE, _("Print Range"),
+                                         wxDefaultPosition, wxDefaultSize,
+                                         2, choices,
+                                         1, wxRA_VERTICAL);
+        m_rangeRadioBox->SetSelection(1);
+       
+       mainsizer->Add( m_rangeRadioBox, 0, wxLEFT|wxTOP|wxRIGHT, 10 );
+    }
+
+    // 3) bottom row
+    
+    wxBoxSizer *bottomsizer = new wxBoxSizer( wxHORIZONTAL );
+
+    if (m_printDialogData.GetFromPage() != 0)
     {
     {
+        bottomsizer->Add( new wxStaticText(this, wxPRINTID_STATIC, _("From:") ), 0, wxCENTER|wxALL, 5 );
+        m_fromText = new wxTextCtrl(this, wxPRINTID_FROM, "", wxDefaultPosition, wxSize(40, -1));
+       bottomsizer->Add( m_fromText, 1, wxCENTER|wxRIGHT, 10 );
+
+        bottomsizer->Add( new wxStaticText(this, wxPRINTID_STATIC, _("To:") ), 0, wxCENTER|wxALL, 5);
+        m_toText = new wxTextCtrl(this, wxPRINTID_TO, "", wxDefaultPosition, wxSize(40, -1));
+       bottomsizer->Add( m_toText, 1, wxCENTER|wxRIGHT, 10 );
+    }
+
+    bottomsizer->Add( new wxStaticText(this, wxPRINTID_STATIC, _("Copies:") ), 0, wxCENTER|wxALL, 5 );
+    m_noCopiesText = new wxTextCtrl(this, wxPRINTID_COPIES, "", wxPoint(252, 130), wxSize(40, -1));
+    bottomsizer->Add( m_noCopiesText, 1, wxCENTER|wxRIGHT, 10 );
+
+    mainsizer->Add( bottomsizer, 0, wxTOP|wxLEFT|wxRIGHT, 12 );
+
+#if wxUSE_STATLINE
+    // 4) static line
+    mainsizer->Add( new wxStaticLine( this, -1 ), 0, wxEXPAND | wxLEFT|wxRIGHT|wxTOP, 10 );
+#endif
+
+    // 5) buttons
+
+    mainsizer->Add( CreateButtonSizer( wxOK|wxCANCEL), 0, wxCENTER|wxALL, 10 );
+
+    SetAutoLayout( TRUE );
+    SetSizer( mainsizer );
+
+    mainsizer->Fit( this );
+    Centre(wxBOTH);
+
+    // Calls wxWindow::OnInitDialog and then wxGenericPrintDialog::TransferDataToWindow
+    InitDialog();
+    delete[] choices;
+}
+
+int wxGenericPrintDialog::ShowModal()
+{
+    if ( m_printDialogData.GetSetupDialog() )
+    {
+        // Make sure wxPrintData object reflects the settings now, in case the setup dialog
+        // changes it. In fact there aren't any common settings at
+        // present, but there might be in future.
+        // TransferDataFromWindow();
+
         wxGenericPrintSetupDialog *genericPrintSetupDialog =
         wxGenericPrintSetupDialog *genericPrintSetupDialog =
-            new wxGenericPrintSetupDialog(GetParent(), wxThePrintSetupData);
+            new wxGenericPrintSetupDialog(this, & m_printDialogData.GetPrintData());
         int ret = genericPrintSetupDialog->ShowModal();
         if ( ret != wxID_CANCEL )
         {
         int ret = genericPrintSetupDialog->ShowModal();
         if ( ret != wxID_CANCEL )
         {
-            *wxThePrintSetupData = genericPrintSetupDialog->printData;
+            // Transfer settings to the global object (for compatibility) and to
+            // the print dialog's print data.
+            *wxThePrintSetupData = genericPrintSetupDialog->GetPrintData();
+            m_printDialogData.GetPrintData() = genericPrintSetupDialog->GetPrintData();
         }
         }
-        genericPrintSetupDialog->Close(TRUE);
+        genericPrintSetupDialog->Destroy();
+
+        // Restore the wxPrintData settings again (uncomment if any settings become common
+        // to both dialogs)
+        // TransferDataToWindow();
+
         return ret;
     }
     else
         return ret;
     }
     else
@@ -161,473 +250,567 @@ int wxGenericPrintDialog::ShowModal(void)
     }
 }
 
     }
 }
 
-wxGenericPrintDialog::~wxGenericPrintDialog(void)
+wxGenericPrintDialog::~wxGenericPrintDialog()
 {
 }
 
 void wxGenericPrintDialog::OnOK(wxCommandEvent& WXUNUSED(event))
 {
 {
 }
 
 void wxGenericPrintDialog::OnOK(wxCommandEvent& WXUNUSED(event))
 {
-  TransferDataFromWindow();
-
-  // There are some interactions between the global setup data
-  // and the standard print dialog. The global printing 'mode'
-  // is determined by whether the user checks Print to file
-  // or not.
-  if (printData.GetPrintToFile())
-  {
-    wxThePrintSetupData->SetPrinterMode(PS_FILE);
-
-    char *f = wxFileSelector(_("PostScript file"),
-        wxPathOnly(wxThePrintSetupData->GetPrinterFile()),
-        wxFileNameFromPath(wxThePrintSetupData->GetPrinterFile()),
-        "ps", "*.ps", 0, this);
-    if (f)
-      wxThePrintSetupData->SetPrinterFile(f);
+    TransferDataFromWindow();
+
+    // There are some interactions between the global setup data
+    // and the standard print dialog. The global printing 'mode'
+    // is determined by whether the user checks Print to file
+    // or not.
+    if (m_printDialogData.GetPrintToFile())
+    {
+        m_printDialogData.GetPrintData().SetPrintMode(wxPRINT_MODE_FILE);
+        wxThePrintSetupData->SetPrinterMode(wxPRINT_MODE_FILE);
+
+        wxString f = wxFileSelector(_("PostScript file"),
+            wxPathOnly(wxThePrintSetupData->GetPrinterFile()),
+            wxFileNameFromPath(wxThePrintSetupData->GetPrinterFile()),
+            wxT("ps"), wxT("*.ps"), 0, this);
+        if ( f.IsEmpty() )
+            return;
+
+        m_printDialogData.GetPrintData().SetFilename(f);
+        wxThePrintSetupData->SetPrinterFile(f);
+    }
     else
     else
-      return;
-  }
-  else
-    wxThePrintSetupData->SetPrinterMode(PS_PRINTER);
-  
-  EndModal(wxID_OK);
+    {
+        m_printDialogData.GetPrintData().SetPrintMode(wxPRINT_MODE_PRINTER);
+        wxThePrintSetupData->SetPrinterMode(wxPRINT_MODE_PRINTER);
+    }
+
+    EndModal(wxID_OK);
 }
 
 void wxGenericPrintDialog::OnRange(wxCommandEvent& event)
 {
 }
 
 void wxGenericPrintDialog::OnRange(wxCommandEvent& event)
 {
-  if (!fromText) return;
-
-  if (event.GetInt() == 1)
-  {
-    fromText->Enable(FALSE);
-    toText->Enable(FALSE);
-  }
-  else if (event.GetInt() == 0)
-  {
-    fromText->Enable(TRUE);
-    toText->Enable(TRUE);
-  }
+    if (!m_fromText) return;
+
+    if (event.GetInt() == 0)
+    {
+        m_fromText->Enable(FALSE);
+        m_toText->Enable(FALSE);
+    }
+    else if (event.GetInt() == 1)
+    {
+        m_fromText->Enable(TRUE);
+        m_toText->Enable(TRUE);
+    }
 }
 
 void wxGenericPrintDialog::OnSetup(wxCommandEvent& WXUNUSED(event))
 {
 }
 
 void wxGenericPrintDialog::OnSetup(wxCommandEvent& WXUNUSED(event))
 {
-  wxGenericPrintSetupDialog *genericPrintSetupDialog =
-          new wxGenericPrintSetupDialog(this, wxThePrintSetupData);
-  int ret = genericPrintSetupDialog->ShowModal();
-  if ( ret != wxID_CANCEL )
-  {
-    *wxThePrintSetupData = genericPrintSetupDialog->printData;
-    printData.SetOrientation(wxThePrintSetupData->GetPrinterOrientation());
-  }
-
-  genericPrintSetupDialog->Close(TRUE);
+   *wxThePrintSetupData = m_printDialogData.GetPrintData();
+    wxGenericPrintSetupDialog *genericPrintSetupDialog =
+        new wxGenericPrintSetupDialog(this, wxThePrintSetupData);
+    int ret = genericPrintSetupDialog->ShowModal();
+    if ( ret != wxID_CANCEL )
+    {
+        *wxThePrintSetupData = genericPrintSetupDialog->GetPrintData();
+        m_printDialogData = genericPrintSetupDialog->GetPrintData();
+    }
+
+    genericPrintSetupDialog->Close(TRUE);
 }
 
 }
 
-bool wxGenericPrintDialog::TransferDataToWindow(void)
+bool wxGenericPrintDialog::TransferDataToWindow()
 {
 {
-   char buf[10];
-
-   if(printData.GetFromPage() != 0)
-   {
-      if (printData.GetEnablePageNumbers())
-      {
-         fromText->Enable(TRUE);
-         toText->Enable(TRUE);
-         
-         sprintf(buf, "%d", printData.GetFromPage());
-         fromText->SetValue(buf);
-         sprintf(buf, "%d", printData.GetToPage());
-         toText->SetValue(buf);
-         
-         if (printData.GetAllPages())
-            rangeRadioBox->SetSelection(0);
-         else
-            rangeRadioBox->SetSelection(1);
-      }
-      else
-      {
-         fromText->Enable(FALSE);
-         toText->Enable(FALSE);
-         rangeRadioBox->SetSelection(0);
-         rangeRadioBox->wxRadioBox::Enable(1, FALSE);
-      }
-   }
-  sprintf(buf, "%d", printData.GetNoCopies());
-  noCopiesText->SetValue(buf);
-
-  printToFileCheckBox->SetValue(printData.GetPrintToFile());
-  printToFileCheckBox->Enable(printData.GetEnablePrintToFile());
-  return TRUE;
+    char buf[10];
+
+    if(m_printDialogData.GetFromPage() != 0)
+    {
+       if(m_fromText)
+       {
+          if (m_printDialogData.GetEnablePageNumbers())
+          {
+             m_fromText->Enable(TRUE);
+             m_toText->Enable(TRUE);
+             sprintf(buf, "%d", m_printDialogData.GetFromPage());
+             m_fromText->SetValue(buf);
+             sprintf(buf, "%d", m_printDialogData.GetToPage());
+             m_toText->SetValue(buf);
+             if(m_rangeRadioBox)
+                if (m_printDialogData.GetAllPages())
+                   m_rangeRadioBox->SetSelection(0);
+                else
+                   m_rangeRadioBox->SetSelection(1);
+          }
+          else
+          {
+             m_fromText->Enable(FALSE);
+             m_toText->Enable(FALSE);
+             if(m_rangeRadioBox)
+             {
+                m_rangeRadioBox->SetSelection(0);
+                m_rangeRadioBox->wxRadioBox::Enable(1, FALSE);
+             }
+          }
+       }
+    }
+    sprintf(buf, "%d", m_printDialogData.GetNoCopies());
+    m_noCopiesText->SetValue(buf);
+
+    m_printToFileCheckBox->SetValue(m_printDialogData.GetPrintToFile());
+    m_printToFileCheckBox->Enable(m_printDialogData.GetEnablePrintToFile());
+    return TRUE;
 }
 
 }
 
-bool wxGenericPrintDialog::TransferDataFromWindow(void)
+bool wxGenericPrintDialog::TransferDataFromWindow()
 {
 {
-   if(printData.GetFromPage() != -1)
-   {
-      if (printData.GetEnablePageNumbers())
-      {
-         printData.SetFromPage(atoi(fromText->GetValue()));
-         printData.SetToPage(atoi(toText->GetValue()));
-      }
-      if (rangeRadioBox->GetSelection() == 0)
-         printData.SetAllPages(TRUE);
-      else
-         printData.SetAllPages(FALSE);
-   }
-   else
-   { // continuous printing
-      printData.SetFromPage(1);
-      printData.SetToPage(32000);
-   }
-   printData.SetNoCopies(atoi(noCopiesText->GetValue()));
-   printData.SetPrintToFile(printToFileCheckBox->GetValue());
-
-  return TRUE;
+    if(m_printDialogData.GetFromPage() != -1)
+    {
+        if (m_printDialogData.GetEnablePageNumbers())
+        {
+           if(m_fromText) m_printDialogData.SetFromPage(wxAtoi(m_fromText->GetValue()));
+           if(m_toText)   m_printDialogData.SetToPage(wxAtoi(m_toText->GetValue()));
+        }
+        if(m_rangeRadioBox)
+        {
+           if (m_rangeRadioBox->GetSelection() == 0)
+              m_printDialogData.SetAllPages(TRUE);
+           else
+              m_printDialogData.SetAllPages(FALSE);
+        }
+    }
+    else
+    { // continuous printing
+        m_printDialogData.SetFromPage(1);
+        m_printDialogData.SetToPage(32000);
+    }
+    m_printDialogData.SetNoCopies(wxAtoi(m_noCopiesText->GetValue()));
+    m_printDialogData.SetPrintToFile(m_printToFileCheckBox->GetValue());
+
+    return TRUE;
 }
 
 }
 
-wxDC *wxGenericPrintDialog::GetPrintDC(void)
+/*
+TODO: collate and noCopies should be duplicated across dialog data and print data objects
+(slightly different semantics on Windows but let's ignore this for a bit).
+*/
+
+wxDC *wxGenericPrintDialog::GetPrintDC()
 {
 {
-  return new wxPostScriptDC(wxThePrintSetupData->GetPrinterFile(), FALSE, (wxWindow *) NULL);
+  //    return new wxPostScriptDC(wxThePrintSetupData->GetPrinterFile(), FALSE, (wxWindow *) NULL);
+  return new wxPostScriptDC(GetPrintDialogData().GetPrintData());
 }
 
 }
 
-/*
- * Generic print setup dialog
- */
+// ----------------------------------------------------------------------------
+// Generic print setup dialog
+// ----------------------------------------------------------------------------
 
 
+wxGenericPrintSetupDialog::wxGenericPrintSetupDialog(wxWindow *parent, wxPrintData* data):
+wxDialog(parent, -1, _("Print Setup"), wxPoint(0, 0), wxSize(600, 600), wxDEFAULT_DIALOG_STYLE|wxDIALOG_MODAL|wxTAB_TRAVERSAL)
+{
+    Init(data);
+}
+
+// Convert wxPrintSetupData to standard wxPrintData object
 wxGenericPrintSetupDialog::wxGenericPrintSetupDialog(wxWindow *parent, wxPrintSetupData* data):
 wxGenericPrintSetupDialog::wxGenericPrintSetupDialog(wxWindow *parent, wxPrintSetupData* data):
-  wxDialog(parent, -1, _("Print Setup"), wxPoint(0, 0), wxSize(600, 600), wxDEFAULT_DIALOG_STYLE|wxDIALOG_MODAL)
+wxDialog(parent, -1, _("Print Setup"), wxPoint(0, 0), wxSize(600, 600), wxDEFAULT_DIALOG_STYLE|wxDIALOG_MODAL|wxTAB_TRAVERSAL)
 {
 {
-  if ( data )
-    printData = *data;
+    wxPrintData printData;
+    if (data)
+        printData = * data;
+    else
+        printData = * wxThePrintSetupData;
 
 
-  int buttonWidth = 65;
-  int buttonHeight = 25;
-  int spacing = 5;
-  int yPos = 5;
-  int xPos = 5;
+    Init(& printData);
+}
 
 
-  wxButton *okButton = new wxButton(this, wxID_OK, _("OK"), wxPoint(xPos, yPos), wxSize(buttonWidth, buttonHeight));
-  xPos += buttonWidth + spacing;
-  (void) new wxButton(this, wxID_CANCEL, _("Cancel"), wxPoint(xPos, yPos), wxSize(buttonWidth, buttonHeight));
+void wxGenericPrintSetupDialog::Init(wxPrintData* data)
+{
+    if ( data )
+        m_printData = *data;
 
 
-  okButton->SetDefault();
-  okButton->SetFocus();
+    int staticBoxWidth = 300;
 
 
-  yPos += 35;
-  xPos = 5;
+    (void) new wxStaticBox(this, wxPRINTID_STATIC, _("Paper size"), wxPoint(10, 10), wxSize(staticBoxWidth, 60) );
 
 
-  paperTypeChoice = CreatePaperTypeChoice(&xPos, &yPos);
+    int xPos = 20;
+    int yPos = 30;
+    m_paperTypeChoice = CreatePaperTypeChoice(&xPos, &yPos);
 
 
-  wxString choices[2];
-  choices[0] = _("Portrait");
-  choices[1] = _("Landscape");
+    wxString *choices =  new wxString[2];
+    choices[0] = _("Portrait");
+    choices[1] = _("Landscape");
 
 
-  orientationRadioBox = new wxRadioBox(this, wxPRINTID_ORIENTATION, _("Orientation"),
-    wxPoint(xPos, yPos), wxSize(-1, -1), 2, choices, 2);
-  orientationRadioBox->SetSelection(0);
+    m_orientationRadioBox = new wxRadioBox(this, wxPRINTID_ORIENTATION, _("Orientation"),
+        wxPoint(10, 80), wxSize(-1, -1), 2, choices, 1, wxRA_VERTICAL );
+    m_orientationRadioBox->SetSelection(0);
 
 
-  xPos += 200;
+    (void) new wxStaticBox(this, wxPRINTID_STATIC, _("Options"), wxPoint(10, 130), wxSize(staticBoxWidth, 50) );
 
 
-  colourCheckBox = new wxCheckBox(this, wxPRINTID_PRINTCOLOUR, _("Print in colour"), wxPoint(xPos, yPos));
+    int colourYPos = 145;
 
 
-  xPos = 5;
-  yPos += 60;
+#ifdef __WXMOTIF__
+    colourYPos = 150;
+#endif
 
 
-  int staticWidth = 100;
-  int textWidth = 120;
-  spacing = 10;
+    m_colourCheckBox = new wxCheckBox(this, wxPRINTID_PRINTCOLOUR, _("Print in colour"), wxPoint(15, colourYPos));
 
 
-  (void) new wxStaticText(this, wxPRINTID_STATIC, _("Printer command:"), wxPoint(xPos, yPos));
-  xPos += staticWidth;
+    (void) new wxStaticBox(this, wxPRINTID_STATIC, _("Print spooling"), wxPoint(330, 10), wxSize(200,170) );
 
 
-  printerCommandText = new wxTextCtrl(this, wxPRINTID_COMMAND, "", wxPoint(xPos, yPos), wxSize(textWidth, -1));
-  xPos += textWidth + spacing;
+    (void) new wxStaticText(this, wxPRINTID_STATIC, _("Printer command:"), wxPoint(340, 30));
 
 
-  (void) new wxStaticText(this, wxPRINTID_STATIC, _("Printer options:"), wxPoint(xPos, yPos));
-  xPos += staticWidth;
+    m_printerCommandText = new wxTextCtrl(this, wxPRINTID_COMMAND, "", wxPoint(360, 55), wxSize(150, -1));
 
 
-  printerOptionsText = new wxTextCtrl(this, wxPRINTID_OPTIONS, "", wxPoint(xPos, yPos), wxSize(textWidth, -1));
+    (void) new wxStaticText(this, wxPRINTID_STATIC, _("Printer options:"), wxPoint(340, 110));
 
 
-  Fit();
-  Centre(wxBOTH);
+    m_printerOptionsText = new wxTextCtrl(this, wxPRINTID_OPTIONS, "", wxPoint(360, 135), wxSize(150, -1));
 
 
-  InitDialog();
-}
+    wxButton *okButton = new wxButton(this, wxID_OK, _("OK"), wxPoint(130, 200), wxSize(80, -1));
+    (void) new wxButton(this, wxID_CANCEL, _("Cancel"), wxPoint(320, 200), wxSize(80, -1));
 
 
-wxGenericPrintSetupDialog::~wxGenericPrintSetupDialog(void)
-{
-}
+    okButton->SetDefault();
+    okButton->SetFocus();
 
 
-bool wxGenericPrintSetupDialog::TransferDataToWindow(void)
-{
-  if (printerCommandText && printData.GetPrinterCommand())
-    printerCommandText->SetValue(printData.GetPrinterCommand());
-  if (printerOptionsText && printData.GetPrinterOptions())
-    printerOptionsText->SetValue(printData.GetPrinterOptions());
-  if (colourCheckBox)
-    colourCheckBox->SetValue(printData.GetColour());
-
-  if (orientationRadioBox)
-  {
-    if (printData.GetPrinterOrientation() == PS_PORTRAIT)
-      orientationRadioBox->SetSelection(0);
-    else
-      orientationRadioBox->SetSelection(1);
-  }
-  return TRUE;
+    Fit();
+    Centre(wxBOTH);
+
+    InitDialog();
+    delete[] choices;
 }
 
 }
 
-bool wxGenericPrintSetupDialog::TransferDataFromWindow(void)
+wxGenericPrintSetupDialog::~wxGenericPrintSetupDialog()
 {
 {
-  if (printerCommandText)
-    printData.SetPrinterCommand(WXSTRINGCAST printerCommandText->GetValue());
-  if (printerOptionsText)
-    printData.SetPrinterOptions(WXSTRINGCAST printerOptionsText->GetValue());
-  if (colourCheckBox)
-    printData.SetColour(colourCheckBox->GetValue());
-  if (orientationRadioBox)
-  {
-    int sel = orientationRadioBox->GetSelection();
-    if (sel == 0)
-      printData.SetPrinterOrientation(PS_PORTRAIT);
-    else
-      printData.SetPrinterOrientation(PS_LANDSCAPE);
-  }
-  if (paperTypeChoice)
-  {
-    wxString val(paperTypeChoice->GetStringSelection());
-    if (!val.IsNull() && val != "")
-      printData.SetPaperName((char *)(const char *)val);
-  }
-  return TRUE;
 }
 
 }
 
-wxChoice *wxGenericPrintSetupDialog::CreatePaperTypeChoice(int *x, int *y)
+bool wxGenericPrintSetupDialog::TransferDataToWindow()
 {
 {
-  if (!wxThePrintPaperDatabase)
-  {
-    wxThePrintPaperDatabase = new wxPrintPaperDatabase;
-    wxThePrintPaperDatabase->CreateDatabase();
-  }
-  int n = wxThePrintPaperDatabase->Number();
-  wxString *choices = new wxString [n];
-  int sel = 0;
-  int i;
-  for (i = 0; i < n; i++)
-  {
-    wxPrintPaperType *paper = (wxPrintPaperType *)wxThePrintPaperDatabase->Nth(i)->Data();
-    choices[i] = paper->pageName;
-    if (printData.GetPaperName() && choices[i] == printData.GetPaperName())
-      sel = i;
-  }
-
-  (void) new wxStaticText(this, wxPRINTID_STATIC, _("Paper size"), wxPoint(*x, *y));
-  *y += 25;
-
-  wxChoice *choice = new wxChoice(this, wxPRINTID_PAPERSIZE, wxPoint(*x, *y), wxSize(300, -1), n,
-    choices);
-  *y += 35;
-  delete[] choices;
-
-  choice->SetSelection(sel);
-  return choice;
+    if (m_printerCommandText && m_printData.GetPrinterCommand())
+        m_printerCommandText->SetValue(m_printData.GetPrinterCommand());
+    if (m_printerOptionsText && m_printData.GetPrinterOptions())
+        m_printerOptionsText->SetValue(m_printData.GetPrinterOptions());
+    if (m_colourCheckBox)
+        m_colourCheckBox->SetValue(m_printData.GetColour());
+
+    if (m_orientationRadioBox)
+    {
+        if (m_printData.GetOrientation() == wxPORTRAIT)
+            m_orientationRadioBox->SetSelection(0);
+        else
+            m_orientationRadioBox->SetSelection(1);
+    }
+    return TRUE;
 }
 
 }
 
-/*
- * Generic page setup dialog
- */
-
-void wxGenericPageSetupDialog::OnPrinter(wxCommandEvent& WXUNUSED(event))
+bool wxGenericPrintSetupDialog::TransferDataFromWindow()
 {
 {
-  if (wxTheApp->GetPrintMode() == wxPRINT_POSTSCRIPT)
-  {
-    wxGenericPrintSetupDialog *genericPrintSetupDialog =
-          new wxGenericPrintSetupDialog(this, wxThePrintSetupData);
-    int ret = genericPrintSetupDialog->ShowModal();
-    if (ret == wxID_OK)
-      *wxThePrintSetupData = genericPrintSetupDialog->GetPrintData();
+    if (m_printerCommandText)
+        m_printData.SetPrinterCommand(m_printerCommandText->GetValue());
+    if (m_printerOptionsText)
+        m_printData.SetPrinterOptions(m_printerOptionsText->GetValue());
+    if (m_colourCheckBox)
+        m_printData.SetColour(m_colourCheckBox->GetValue());
+    if (m_orientationRadioBox)
+    {
+        int sel = m_orientationRadioBox->GetSelection();
+        if (sel == 0)
+            m_printData.SetOrientation(wxPORTRAIT);
+        else
+            m_printData.SetOrientation(wxLANDSCAPE);
+    }
+    if (m_paperTypeChoice)
+    {
+        wxString val(m_paperTypeChoice->GetStringSelection());
+        if (!val.IsNull() && val != "")
+            m_printData.SetPaperId(wxThePrintPaperDatabase->ConvertNameToId(val));
+    }
 
 
-    genericPrintSetupDialog->Close(TRUE);
-  }
-#ifdef __WXMSW__
-  else
-  {
-    wxPrintData data;
-    data.SetSetupDialog(TRUE);
-    wxPrintDialog printDialog(this, & data);
-    printDialog.Show(TRUE);
-  }
-#endif
+    // This is for backward compatibility only
+    *wxThePrintSetupData = GetPrintData();
+    return TRUE;
 }
 
 }
 
-wxGenericPageSetupDialog::wxGenericPageSetupDialog(wxWindow *parent, wxPageSetupData* data):
-  wxDialog(parent, -1, _("Page Setup"), wxPoint(0, 0), wxSize(600, 600), wxDIALOG_MODAL|wxDEFAULT_DIALOG_STYLE)
+wxComboBox *wxGenericPrintSetupDialog::CreatePaperTypeChoice(int *x, int *y)
 {
 {
-  if ( data )
-    pageData = *data;
+/* Should not be necessary
+    if (!wxThePrintPaperDatabase)
+    {
+        wxThePrintPaperDatabase = new wxPrintPaperDatabase;
+        wxThePrintPaperDatabase->CreateDatabase();
+    }
+*/
+    int n = wxThePrintPaperDatabase->Number();
+    wxString *choices = new wxString [n];
+    int sel = 0;
+    int i;
+    for (i = 0; i < n; i++)
+    {
+        wxPrintPaperType *paper = (wxPrintPaperType *)wxThePrintPaperDatabase->Nth(i)->Data();
+        choices[i] = paper->GetName();
+        if (m_printData.GetPaperId() == paper->GetId())
+            sel = i;
+    }
 
 
-  int buttonWidth = 75;
-  int buttonHeight = 25;
-  int spacing = 5;
-  int yPos = 5;
-  int xPos = 5;
+    int width = 250;
 
 
-  wxButton *okButton = new wxButton(this, wxID_OK, _("OK"), wxPoint(5, yPos), wxSize(buttonWidth, buttonHeight));
-  (void) new wxButton(this, wxID_CANCEL, _("Cancel"), wxPoint(buttonWidth + 5 + spacing, yPos), wxSize(buttonWidth, buttonHeight));
+    wxComboBox *choice = new wxComboBox(this, wxPRINTID_PAPERSIZE,
+                                        _("Paper Size"),
+                                        wxPoint(*x, *y), wxSize(width, -1), n,
+        choices);
 
 
-  printerButton = new wxButton(this, wxPRINTID_SETUP, _("Printer..."), wxPoint(buttonWidth*2 + 5 + 2*spacing, yPos), wxSize(buttonWidth, buttonHeight));
+    //    SetFont(thisFont);
 
 
-  if ( !pageData.GetEnablePrinter() )
-    printerButton->Enable(FALSE);
+    delete[] choices;
 
 
-//  if (printData.GetEnableHelp())
-//  wxButton *helpButton = new wxButton(this, (wxFunction)wxGenericPageSetupHelpProc, _("Help"), -1, -1, buttonWidth, buttonHeight);
+    choice->SetSelection(sel);
+    return choice;
+}
+#endif // wxUSE_POSTSCRIPT
 
 
-  okButton->SetDefault();
-  okButton->SetFocus();
+// ----------------------------------------------------------------------------
+// Generic page setup dialog
+// ----------------------------------------------------------------------------
 
 
-  xPos = 5;
-  yPos += 35;
+void wxGenericPageSetupDialog::OnPrinter(wxCommandEvent& WXUNUSED(event))
+{
+    // We no longer query GetPrintMode, so we can eliminate the need
+    // to call SetPrintMode.
+    // This has the limitation that we can't explicitly call the PostScript
+    // print setup dialog from the generic Page Setup dialog under Windows,
+    // but since this choice would only happen when trying to do PostScript
+    // printing under Windows (and only in 16-bit Windows which
+    // doesn't have a Windows-specific page setup dialog) it's worth it.
+
+    // First save the current settings, so the wxPrintData object is up to date.
+    TransferDataFromWindow();
+
+    // Transfer the current print settings from this dialog to the page setup dialog.
+    wxPrintDialogData data;
+    data = GetPageSetupData().GetPrintData();
+    data.SetSetupDialog(TRUE);
+    wxPrintDialog *printDialog = new wxPrintDialog(this, & data);
+    printDialog->ShowModal();
 
 
-  paperTypeChoice = CreatePaperTypeChoice(&xPos, &yPos);
+    // Transfer the page setup print settings from the page dialog to this dialog again, in case
+    // the page setup dialog changed something.
+    GetPageSetupData().GetPrintData() = printDialog->GetPrintDialogData().GetPrintData();
+    GetPageSetupData().CalculatePaperSizeFromId(); // Make sure page size reflects the id in wxPrintData
 
 
-  xPos = 5;
+    printDialog->Destroy();
 
 
-  wxString choices[2];
-  choices[0] = _("Portrait");
-  choices[1] = _("Landscape");
-  orientationRadioBox = new wxRadioBox(this, wxPRINTID_ORIENTATION, _("Orientation"),
-    wxPoint(xPos, yPos), wxSize(-1, -1), 2, choices, 2);
-  orientationRadioBox->SetSelection(0);
+    // Now update the dialog in case the page setup dialog changed some of our settings.
+    TransferDataToWindow();
+}
 
 
-  xPos = 5;
-  yPos += 60;
+wxGenericPageSetupDialog::wxGenericPageSetupDialog(wxWindow *parent, wxPageSetupData* data):
+wxDialog(parent, -1, _("Page Setup"), wxPoint(0, 0), wxSize(600, 600), wxDIALOG_MODAL|wxDEFAULT_DIALOG_STYLE|wxTAB_TRAVERSAL)
+{
+    if (data)
+        m_pageData = *data;
+       
+    int textWidth = 80;
+       
+    wxBoxSizer *mainsizer = new wxBoxSizer( wxVERTICAL );
+    
+    // 1) top
+    wxStaticBoxSizer *topsizer = new wxStaticBoxSizer( 
+      new wxStaticBox(this,wxPRINTID_STATIC, _("Paper size")), wxHORIZONTAL );
+    
+    int n = wxThePrintPaperDatabase->Number();
+    wxString *choices = new wxString [n];
+    int i;
+    for (i = 0; i < n; i++)
+    {
+        wxPrintPaperType *paper = (wxPrintPaperType *)wxThePrintPaperDatabase->Nth(i)->Data();
+        choices[i] = paper->GetName();
+    }
 
 
-  int staticWidth = 110;
-  int textWidth = 60;
-  spacing = 10;
+    m_paperTypeChoice = new wxComboBox(this, wxPRINTID_PAPERSIZE, _("Paper Size"),
+                                        wxDefaultPosition, wxSize(300, -1), n, choices);
+    topsizer->Add( m_paperTypeChoice, 1, wxEXPAND|wxALL, 5 );
+//  m_paperTypeChoice->SetSelection(sel);
+
+    mainsizer->Add( topsizer, 0, wxTOP|wxLEFT|wxRIGHT | wxEXPAND, 10 );
+
+    // 2) middle sizer with radio box
+
+    wxString *choices2 = new wxString[2];
+    choices2[0] = _("Portrait");
+    choices2[1] = _("Landscape");
+    m_orientationRadioBox = new wxRadioBox(this, wxPRINTID_ORIENTATION, _("Orientation"),
+        wxDefaultPosition, wxDefaultSize, 2, choices2, 2);
+    m_orientationRadioBox->SetSelection(0);
+
+    mainsizer->Add( m_orientationRadioBox, 0, wxTOP|wxLEFT|wxRIGHT, 10 );
+
+    // 3) margins
+
+    wxBoxSizer *table = new wxBoxSizer( wxHORIZONTAL );
+
+    wxBoxSizer *column1 = new wxBoxSizer( wxVERTICAL );
+    column1->Add( new wxStaticText(this, wxPRINTID_STATIC, _("Left margin (mm):")),1,wxALL|wxALIGN_RIGHT,5 );
+    column1->Add( new wxStaticText(this, wxPRINTID_STATIC, _("Top margin (mm):")),1,wxALL|wxALIGN_RIGHT,5 );
+    table->Add( column1, 0, wxALL | wxEXPAND, 5 );
+    
+    wxBoxSizer *column2 = new wxBoxSizer( wxVERTICAL );
+    m_marginLeftText = new wxTextCtrl(this, wxPRINTID_LEFTMARGIN, "", wxDefaultPosition, wxSize(textWidth, -1));
+    m_marginTopText = new wxTextCtrl(this, wxPRINTID_TOPMARGIN, "", wxDefaultPosition, wxSize(textWidth, -1));
+    column2->Add( m_marginLeftText, 1, wxALL, 5 );
+    column2->Add( m_marginTopText, 1, wxALL, 5 );
+    table->Add( column2, 0, wxRIGHT|wxTOP|wxBOTTOM | wxEXPAND, 5 );
+    
+    wxBoxSizer *column3 = new wxBoxSizer( wxVERTICAL );
+    column3->Add( new wxStaticText(this, wxPRINTID_STATIC, _("Right margin (mm):")),1,wxALL|wxALIGN_RIGHT,5 );
+    column3->Add( new wxStaticText(this, wxPRINTID_STATIC, _("Bottom margin (mm):")),1,wxALL|wxALIGN_RIGHT,5 );
+    table->Add( column3, 0, wxALL | wxEXPAND, 5 );
+    
+    wxBoxSizer *column4 = new wxBoxSizer( wxVERTICAL );
+    m_marginRightText = new wxTextCtrl(this, wxPRINTID_RIGHTMARGIN, "", wxDefaultPosition, wxSize(textWidth, -1));
+    m_marginBottomText = new wxTextCtrl(this, wxPRINTID_BOTTOMMARGIN, "", wxDefaultPosition, wxSize(textWidth, -1));
+    column4->Add( m_marginRightText, 1, wxALL, 5 );
+    column4->Add( m_marginBottomText, 1, wxALL, 5 );
+    table->Add( column4, 0, wxRIGHT|wxTOP|wxBOTTOM | wxEXPAND, 5 );
+
+    mainsizer->Add( table, 0 );
+
+#if wxUSE_STATLINE
+    // 5) static line
+    mainsizer->Add( new wxStaticLine( this, -1 ), 0, wxEXPAND | wxLEFT|wxRIGHT|wxTOP, 10 );
+#endif
 
 
-  (void) new wxStaticText(this, wxPRINTID_STATIC, _("Left margin (mm):"), wxPoint(xPos, yPos));
-  xPos += staticWidth;
+    // 6) buttons
+    
+    wxSizer* buttonsizer = CreateButtonSizer( wxOK|wxCANCEL);
+    m_printerButton = new wxButton(this, wxPRINTID_SETUP, _("Printer...") );
+    buttonsizer->Add( m_printerButton, 0, wxLEFT|wxRIGHT, 10 );
+    if ( !m_pageData.GetEnablePrinter() )
+        m_printerButton->Enable(FALSE);
+    //  if (m_printData.GetEnableHelp())
+    //  wxButton *helpButton = new wxButton(this, (wxFunction)wxGenericPageSetupHelpProc, _("Help"), -1, -1, buttonWidth, buttonHeight);
+    mainsizer->Add( buttonsizer, 0, wxCENTER|wxALL, 10 );
 
 
-  marginLeftText = new wxTextCtrl(this, wxPRINTID_LEFTMARGIN, "", wxPoint(xPos, yPos), wxSize(textWidth, -1));
-  xPos += textWidth + spacing;
 
 
-  (void) new wxStaticText(this, wxPRINTID_STATIC, _("Right margin (mm):"), wxPoint(xPos, yPos));
-  xPos += staticWidth;
+    SetAutoLayout( TRUE );
+    SetSizer( mainsizer );
 
 
-  marginRightText = new wxTextCtrl(this, wxPRINTID_RIGHTMARGIN, "", wxPoint(xPos, yPos), wxSize(textWidth, -1));
-  xPos += textWidth + spacing;
+    mainsizer->Fit( this );
+    Centre(wxBOTH);
 
 
-  yPos += 35;
-  xPos = 5;
+    InitDialog();
+    
+    delete[] choices;
+    delete [] choices2;
+}
 
 
-  (void) new wxStaticText(this, wxPRINTID_STATIC, _("Top margin (mm):"), wxPoint(xPos, yPos));
-  xPos += staticWidth;
+wxGenericPageSetupDialog::~wxGenericPageSetupDialog()
+{
+}
 
 
-  marginTopText = new wxTextCtrl(this, wxPRINTID_TOPMARGIN, "", wxPoint(xPos, yPos), wxSize(textWidth, -1));
-  xPos += textWidth + spacing;
+bool wxGenericPageSetupDialog::TransferDataToWindow()
+{
+    if (m_marginLeftText)
+        m_marginLeftText->SetValue(IntToString((int) m_pageData.GetMarginTopLeft().x));
+    if (m_marginTopText)
+        m_marginTopText->SetValue(IntToString((int) m_pageData.GetMarginTopLeft().y));
+    if (m_marginRightText)
+        m_marginRightText->SetValue(IntToString((int) m_pageData.GetMarginBottomRight().x));
+    if (m_marginBottomText)
+        m_marginBottomText->SetValue(IntToString((int) m_pageData.GetMarginBottomRight().y));
+
+    if (m_orientationRadioBox)
+    {
+        if (m_pageData.GetPrintData().GetOrientation() == wxPORTRAIT)
+            m_orientationRadioBox->SetSelection(0);
+        else
+            m_orientationRadioBox->SetSelection(1);
+    }
 
 
-  (void) new wxStaticText(this, wxPRINTID_STATIC, _("Bottom margin (mm):"), wxPoint(xPos, yPos));
-  xPos += staticWidth;
+    // Find the paper type from either the current paper size in the wxPageSetupDialogData, or
+    // failing that, the id in the wxPrintData object.
 
 
-  marginBottomText = new wxTextCtrl(this, wxPRINTID_BOTTOMMARGIN, "", wxPoint(xPos, yPos), wxSize(textWidth, -1));
+    wxPrintPaperType* type = wxThePrintPaperDatabase->FindPaperType(
+             wxSize(m_pageData.GetPaperSize().x * 10, m_pageData.GetPaperSize().y * 10));
 
 
-  Fit();
-  Centre(wxBOTH);
+    if (!type && m_pageData.GetPrintData().GetPaperId() != wxPAPER_NONE)
+        type = wxThePrintPaperDatabase->FindPaperType(m_pageData.GetPrintData().GetPaperId());
 
 
-  InitDialog();
-}
+    if (type)
+    {
+        m_paperTypeChoice->SetStringSelection(type->GetName());
+    }
 
 
-wxGenericPageSetupDialog::~wxGenericPageSetupDialog(void)
-{
+    return TRUE;
 }
 
 }
 
-bool wxGenericPageSetupDialog::TransferDataToWindow(void)
+bool wxGenericPageSetupDialog::TransferDataFromWindow()
 {
 {
-  if (marginLeftText)
-    marginLeftText->SetValue(IntToString((int) pageData.GetMarginTopLeft().x));
-  if (marginTopText)
-    marginTopText->SetValue(IntToString((int) pageData.GetMarginTopLeft().y));
-  if (marginRightText)
-    marginRightText->SetValue(IntToString((int) pageData.GetMarginBottomRight().x));
-  if (marginBottomText)
-    marginBottomText->SetValue(IntToString((int) pageData.GetMarginBottomRight().y));
-
-  if (orientationRadioBox)
-  {
-    if (pageData.GetOrientation() == wxPORTRAIT)
-      orientationRadioBox->SetSelection(0);
-    else
-      orientationRadioBox->SetSelection(1);
-  }
-  return TRUE;
-}
+    if (m_marginLeftText && m_marginTopText)
+        m_pageData.SetMarginTopLeft(wxPoint(wxAtoi((const wxChar *)m_marginLeftText->GetValue()),wxAtoi((const wxChar *)m_marginTopText->GetValue())));
+    if (m_marginRightText && m_marginBottomText)
+        m_pageData.SetMarginBottomRight(wxPoint(wxAtoi((const wxChar *)m_marginRightText->GetValue()),wxAtoi((const wxChar *)m_marginBottomText->GetValue())));
 
 
-bool wxGenericPageSetupDialog::TransferDataFromWindow(void)
-{
-  if (marginLeftText && marginTopText)
-    pageData.SetMarginTopLeft(wxPoint(atoi((const char *)marginLeftText->GetValue()),atoi((const char *)marginTopText->GetValue())));
-  if (marginRightText && marginBottomText)
-    pageData.SetMarginBottomRight(wxPoint(atoi((const char *)marginRightText->GetValue()),atoi((const char *)marginBottomText->GetValue())));
-
-  if (orientationRadioBox)
-  {
-    int sel = orientationRadioBox->GetSelection();
-    if (sel == 0)
-    {
-      wxThePrintSetupData->SetPrinterOrientation(wxPORTRAIT);
-      pageData.SetOrientation(wxPORTRAIT);
-    }
-    else
+    if (m_orientationRadioBox)
     {
     {
-      wxThePrintSetupData->SetPrinterOrientation(wxLANDSCAPE);
-      pageData.SetOrientation(wxLANDSCAPE);
+        int sel = m_orientationRadioBox->GetSelection();
+        if (sel == 0)
+        {
+#if wxUSE_POSTSCRIPT
+            wxThePrintSetupData->SetPrinterOrientation(wxPORTRAIT);
+#endif
+            m_pageData.GetPrintData().SetOrientation(wxPORTRAIT);
+        }
+        else
+        {
+#if wxUSE_POSTSCRIPT
+            wxThePrintSetupData->SetPrinterOrientation(wxLANDSCAPE);
+#endif
+            m_pageData.GetPrintData().SetOrientation(wxLANDSCAPE);
+        }
     }
     }
-  }
-  if (paperTypeChoice)
-  {
-    wxString val(paperTypeChoice->GetStringSelection());
-    if (!val.IsNull() && val != "")
+    if (m_paperTypeChoice)
     {
     {
-        wxPrintPaperType* paper = wxThePrintPaperDatabase->FindPaperType((char*) (const char *)val);
-        if ( paper )
+        wxString val(m_paperTypeChoice->GetStringSelection());
+        if (!val.IsNull() && val != "")
         {
         {
-            pageData.SetPaperSize(wxPoint(paper->widthMM, paper->heightMM));
+            wxPrintPaperType* paper = wxThePrintPaperDatabase->FindPaperType(val);
+            if ( paper )
+            {
+                m_pageData.SetPaperSize(wxSize(paper->GetWidth()/10, paper->GetHeight()/10));
+                m_pageData.GetPrintData().SetPaperId(paper->GetId());
+            }
         }
     }
         }
     }
-  }
-  return TRUE;
+
+    return TRUE;
 }
 
 }
 
-wxChoice *wxGenericPageSetupDialog::CreatePaperTypeChoice(int *x, int *y)
+wxComboBox *wxGenericPageSetupDialog::CreatePaperTypeChoice(int *x, int *y)
 {
 {
-  if (!wxThePrintPaperDatabase)
-  {
-    wxThePrintPaperDatabase = new wxPrintPaperDatabase;
-    wxThePrintPaperDatabase->CreateDatabase();
-  }
-  int n = wxThePrintPaperDatabase->Number();
-  wxString *choices = new wxString [n];
-  int sel = 0;
-  int i;
-  for (i = 0; i < n; i++)
-  {
-    wxPrintPaperType *paper = (wxPrintPaperType *)wxThePrintPaperDatabase->Nth(i)->Data();
-    choices[i] = paper->pageName;
-    if (pageData.GetPaperSize().x == paper->widthMM && pageData.GetPaperSize().y == paper->heightMM)
-      sel = i;
-  }
-
-  (void) new wxStaticText(this, wxPRINTID_STATIC, _("Paper size"), wxPoint(*x, *y));
-  *y += 25;
-
-  wxChoice *choice = new wxChoice(this, wxPRINTID_PAPERSIZE, wxPoint(*x, *y), wxSize(300, -1), n,
-    choices);
-  *y += 35;
-  delete[] choices;
-
-  choice->SetSelection(sel);
-  return choice;
-}
+/*
+    if (!wxThePrintPaperDatabase)
+    {
+        wxThePrintPaperDatabase = new wxPrintPaperDatabase;
+        wxThePrintPaperDatabase->CreateDatabase();
+    }
+*/
+
+    int n = wxThePrintPaperDatabase->Number();
+    wxString *choices = new wxString [n];
+    int i;
+    for (i = 0; i < n; i++)
+    {
+        wxPrintPaperType *paper = (wxPrintPaperType *)wxThePrintPaperDatabase->Nth(i)->Data();
+        choices[i] = paper->GetName();
+    }
 
 
+    (void) new wxStaticText(this, wxPRINTID_STATIC, _("Paper size"), wxPoint(*x, *y));
+    *y += 25;
 
 
+    wxComboBox *choice = new wxComboBox(this, wxPRINTID_PAPERSIZE,
+                                        _("Paper Size"),
+                                        wxPoint(*x, *y), wxSize(300, -1), n,
+        choices);
+    *y += 35;
+    delete[] choices;
+
+//    choice->SetSelection(sel);
+    return choice;
+}
+
+#endif
wxWidgets (Impactor)