GetItem() doesn't ignore column number any more

[wxWidgets.git] / utils / dialoged / src / reseditr.cpp

diff --git a/utils/dialoged/src/reseditr.cpp b/utils/dialoged/src/reseditr.cpp
index 6dd1d8987df442875a47e42273319223b98eb6ca..34a5dd365c693f0b723df98322cbc073fd6f420f 100644 (file)
--- a/utils/dialoged/src/reseditr.cpp
+++ b/utils/dialoged/src/reseditr.cpp
@@ -44,14 +44,14 @@
 #include <math.h>
 #include <string.h>
 
 #include <math.h>
 #include <string.h>
 

+#if wxUSE_IOSTREAMH

 #if defined(__WXMSW__) && !defined(__GNUWIN32__)
 #include <strstrea.h>
 #else
 #include <strstream.h>
 #endif
 #if defined(__WXMSW__) && !defined(__GNUWIN32__)
 #include <strstrea.h>
 #else
 #include <strstream.h>
 #endif

-
-#ifdef __WXMSW__
-#include <windows.h>
+#else
+#include <strstream>

 #endif
 
 #ifdef __WXMSW__
 #endif
 
 #ifdef __WXMSW__


@@ -67,7 +67,7 @@
 static void ObjectMenuProc(wxMenu& menu, wxCommandEvent& event);
 wxResourceManager *wxResourceManager::sm_currentResourceManager = NULL;
 
 static void ObjectMenuProc(wxMenu& menu, wxCommandEvent& event);
 wxResourceManager *wxResourceManager::sm_currentResourceManager = NULL;
 

-#ifdef __WXGTK__
+#if defined(__WXGTK__) || defined(__WXMOTIF__)

 #include "bitmaps/load.xpm"
 #include "bitmaps/save.xpm"
 #include "bitmaps/new.xpm"
 #include "bitmaps/load.xpm"
 #include "bitmaps/save.xpm"
 #include "bitmaps/new.xpm"


@@ -82,6 +82,11 @@ wxResourceManager *wxResourceManager::sm_currentResourceManager = NULL;
 #include "bitmaps/toback.xpm"
 #include "bitmaps/help.xpm"
 #include "bitmaps/wxwin.xpm"
 #include "bitmaps/toback.xpm"
 #include "bitmaps/help.xpm"
 #include "bitmaps/wxwin.xpm"

+
+#include "bitmaps/dialog.xpm"
+#include "bitmaps/folder1.xpm"
+#include "bitmaps/folder2.xpm"
+#include "bitmaps/buttonsm.xpm"

 #endif
 
 /*
 #endif
 
 /*


@@ -143,13 +148,14 @@ wxResourceManager::~wxResourceManager()
 bool wxResourceManager::Initialize()
 {
   // Set up the resource filename for each platform.
 bool wxResourceManager::Initialize()
 {
   // Set up the resource filename for each platform.

+  // TODO: This shold be replaced by wxConfig usage.

 #ifdef __WXMSW__
   // dialoged.ini in the Windows directory
 #ifdef __WXMSW__
   // dialoged.ini in the Windows directory

-  char buf[256];
-  GetWindowsDirectory(buf, 256);
-  strcat(buf, "\\dialoged.ini");
-  m_optionsResourceFilename = buf;
-#elif defined(__WXGTK__)
+  wxString windowsDir = wxGetOSDirectory();
+  windowsDir += "\\dialoged.ini" ;
+
+  m_optionsResourceFilename = windowsDir;
+#elif defined(__WXGTK__) || defined(__WXMOTIF__)

   wxGetHomeDir( &m_optionsResourceFilename );
   m_optionsResourceFilename += "/.dialogedrc";
 #else
   wxGetHomeDir( &m_optionsResourceFilename );
   m_optionsResourceFilename += "/.dialogedrc";
 #else


@@ -172,7 +178,7 @@ bool wxResourceManager::Initialize()
 #ifdef __WXMSW__
     m_bitmapImage = new wxBitmap("WXWINBMP", wxBITMAP_TYPE_BMP_RESOURCE);
 #endif
 #ifdef __WXMSW__
     m_bitmapImage = new wxBitmap("WXWINBMP", wxBITMAP_TYPE_BMP_RESOURCE);
 #endif

-#ifdef __WXGTK__
+#if defined(__WXGTK__) || defined(__WXMOTIF__)

     m_bitmapImage = new wxBitmap( wxwin_xpm );
 #endif
   }
     m_bitmapImage = new wxBitmap( wxwin_xpm );
 #endif
   }


@@ -183,11 +189,16 @@ bool wxResourceManager::Initialize()
   wxIcon icon2("FOLDER1_ICON", wxBITMAP_TYPE_ICO_RESOURCE, 16, 16);
   wxIcon icon3("FOLDER2_ICON", wxBITMAP_TYPE_ICO_RESOURCE, 16, 16);
   wxIcon icon4("BUTTONSM_ICON", wxBITMAP_TYPE_ICO_RESOURCE, 16, 16);
   wxIcon icon2("FOLDER1_ICON", wxBITMAP_TYPE_ICO_RESOURCE, 16, 16);
   wxIcon icon3("FOLDER2_ICON", wxBITMAP_TYPE_ICO_RESOURCE, 16, 16);
   wxIcon icon4("BUTTONSM_ICON", wxBITMAP_TYPE_ICO_RESOURCE, 16, 16);

+#else
+  wxIcon icon1( dialog_xpm );    
+  wxIcon icon2( folder1_xpm );    
+  wxIcon icon3( folder2_xpm );    
+  wxIcon icon4( buttonsm_xpm );    
+#endif

   m_imageList.Add(icon1);
   m_imageList.Add(icon2);
   m_imageList.Add(icon3);
   m_imageList.Add(icon4);
   m_imageList.Add(icon1);
   m_imageList.Add(icon2);
   m_imageList.Add(icon3);
   m_imageList.Add(icon4);

-#endif

 
   m_symbolTable.AddStandardSymbols();
 
 
   m_symbolTable.AddStandardSymbols();
 


@@ -308,8 +319,8 @@ bool wxResourceManager::ShowResourceEditor(bool show, wxWindow *WXUNUSED(parent)
     c->right.SameAs      (m_editorFrame, wxRight, 0);
     c->bottom.SameAs     (m_editorFrame, wxBottom, 0);
     c->width.Unconstrained();
     c->right.SameAs      (m_editorFrame, wxRight, 0);
     c->bottom.SameAs     (m_editorFrame, wxBottom, 0);
     c->width.Unconstrained();

-#ifdef __WXGTK__
-    c->height.Absolute(140);
+#if defined(__WXGTK__) || defined(__WXMOTIF__)
+    c->height.Absolute(120);

 #else
     c->height.Absolute(60);
 #endif
 #else
     c->height.Absolute(60);
 #endif


@@ -326,10 +337,8 @@ bool wxResourceManager::ShowResourceEditor(bool show, wxWindow *WXUNUSED(parent)
   else
   {
     wxFrame *fr = m_editorFrame;
   else
   {
     wxFrame *fr = m_editorFrame;

-    if (m_editorFrame->OnClose())
+    if (m_editorFrame->Close())

     {
     {

-      fr->Show(FALSE);
-      delete fr;

       m_editorFrame = NULL;
       m_editorPanel = NULL;
     }
       m_editorFrame = NULL;
       m_editorPanel = NULL;
     }


@@ -380,7 +389,7 @@ bool wxResourceManager::Save(const wxString& filename)
 bool wxResourceManager::SaveAs()
 {
   wxString s(wxFileSelector("Save resource file", wxPathOnly(WXSTRINGCAST m_currentFilename), wxFileNameFromPath(WXSTRINGCAST m_currentFilename),
 bool wxResourceManager::SaveAs()
 {
   wxString s(wxFileSelector("Save resource file", wxPathOnly(WXSTRINGCAST m_currentFilename), wxFileNameFromPath(WXSTRINGCAST m_currentFilename),

-    "wxr", "*.wxr", wxSAVE | wxOVERWRITE_PROMPT));
+    "wxr", "*.wxr", wxSAVE | wxOVERWRITE_PROMPT, wxTheApp->GetTopWindow()));

     
   if (s.IsNull() || s == "")
     return FALSE;
     
   if (s.IsNull() || s == "")
     return FALSE;


@@ -423,7 +432,7 @@ bool wxResourceManager::New(bool loadFromFile, const wxString& filename)
     wxString str = filename;
     if (str == wxString(""))
     {
     wxString str = filename;
     if (str == wxString(""))
     {

-      wxString f(wxFileSelector("Open resource file", NULL, NULL, "wxr", "*.wxr", 0, NULL));
+      wxString f(wxFileSelector("Open resource file", NULL, NULL, "wxr", "*.wxr", 0, wxTheApp->GetTopWindow()));

       if (!f.IsNull() && f != "")
         str = f;
       else
       if (!f.IsNull() && f != "")
         str = f;
       else


@@ -515,7 +524,7 @@ bool wxResourceManager::DisassociateWindows()
 {
   m_resourceTable.BeginFind();
   wxNode *node;
 {
   m_resourceTable.BeginFind();
   wxNode *node;

-  while (node = m_resourceTable.Next())
+  while ((node = m_resourceTable.Next()))

   {
     wxItemResource *res = (wxItemResource *)node->Data();
     DisassociateResource(res);
   {
     wxItemResource *res = (wxItemResource *)node->Data();
     DisassociateResource(res);


@@ -541,7 +550,7 @@ void wxResourceManager::AssociateResource(wxItemResource *resource, wxWindow *wi
     else
     {
       char buf[200];
     else
     {
       char buf[200];

-      sprintf(buf, "AssociateResource: cannot find child window %s", child->GetName() ? child->GetName() : "(unnamed)");
+      sprintf(buf, "AssociateResource: cannot find child window %s", child->GetName() ? (const char*) child->GetName() : "(unnamed)");

       wxMessageBox(buf, "Dialog Editor problem", wxOK);
     }
 
       wxMessageBox(buf, "Dialog Editor problem", wxOK);
     }
 


@@ -556,18 +565,15 @@ bool wxResourceManager::DisassociateResource(wxItemResource *resource)
     return FALSE;
 
   // Disassociate children of window
     return FALSE;
 
   // Disassociate children of window

-  if (win->GetChildren())
+  wxNode *node = win->GetChildren().First();
+  while (node)

   {
   {

-    wxNode *node = win->GetChildren()->First();
-    while (node)
-    {
-      wxWindow *child = (wxWindow *)node->Data();
-      if (child->IsKindOf(CLASSINFO(wxControl)))
-       DisassociateResource(child);
-      node = node->Next();
-    }
+    wxWindow *child = (wxWindow *)node->Data();
+    if (child->IsKindOf(CLASSINFO(wxControl)))
+           DisassociateResource(child);
+    node = node->Next();

   }
   }

-  
+

   RemoveSelection(win);
   m_resourceAssociations.Delete((long)resource);
   return TRUE;
   RemoveSelection(win);
   m_resourceAssociations.Delete((long)resource);
   return TRUE;


@@ -595,7 +601,7 @@ bool wxResourceManager::SaveInfoAndDeleteHandler(wxWindow* win)
         win->PopEventHandler();
 
         // Now reset all child event handlers
         win->PopEventHandler();
 
         // Now reset all child event handlers

-        wxNode *node = win->GetChildren()->First();
+        wxNode *node = win->GetChildren().First();

         while ( node )
         {
                wxWindow *child = (wxWindow *)node->Data();
         while ( node )
         {
                wxWindow *child = (wxWindow *)node->Data();


@@ -644,12 +650,12 @@ wxItemResource *wxResourceManager::FindResourceForWindow(wxWindow *win)
 {
   m_resourceAssociations.BeginFind();
   wxNode *node;
 {
   m_resourceAssociations.BeginFind();
   wxNode *node;

-  while (node = m_resourceAssociations.Next())
+  while ((node = m_resourceAssociations.Next()))

   {
     wxWindow *w = (wxWindow *)node->Data();
     if (w == win)
     {
   {
     wxWindow *w = (wxWindow *)node->Data();
     if (w == win)
     {

-      return (wxItemResource *)node->key.integer;
+      return (wxItemResource *)node->GetKeyInteger();

     }
   }
   return NULL;
     }
   }
   return NULL;


@@ -675,8 +681,10 @@ void wxResourceManager::MakeUniqueName(char *prefix, char *buf)
 
 wxFrame *wxResourceManager::OnCreateEditorFrame(const char *title)
 {
 
 wxFrame *wxResourceManager::OnCreateEditorFrame(const char *title)
 {

+  /*

   int frameWidth = 420;
   int frameHeight = 300;
   int frameWidth = 420;
   int frameHeight = 300;

+  */

   
   wxResourceEditorFrame *frame = new wxResourceEditorFrame(this, NULL, title,
     wxPoint(m_resourceEditorWindowSize.x, m_resourceEditorWindowSize.y),
   
   wxResourceEditorFrame *frame = new wxResourceEditorFrame(this, NULL, title,
     wxPoint(m_resourceEditorWindowSize.x, m_resourceEditorWindowSize.y),


@@ -687,8 +695,7 @@ wxFrame *wxResourceManager::OnCreateEditorFrame(const char *title)
 
   frame->SetAutoLayout(TRUE);
 #ifdef __WXMSW__
 
   frame->SetAutoLayout(TRUE);
 #ifdef __WXMSW__

-  wxIcon *icon = new wxIcon("DIALOGEDICON");
-  frame->SetIcon(icon);
+  frame->SetIcon(wxIcon("DIALOGEDICON"));

 #endif
   return frame;
 }
 #endif
   return frame;
 }


@@ -729,7 +736,11 @@ wxResourceEditorScrolledWindow *wxResourceManager::OnCreateEditorPanel(wxFrame *
 {
   wxResourceEditorScrolledWindow *panel = new wxResourceEditorScrolledWindow(parent, wxDefaultPosition, wxDefaultSize,
 //    wxSUNKEN_BORDER|wxCLIP_CHILDREN);
 {
   wxResourceEditorScrolledWindow *panel = new wxResourceEditorScrolledWindow(parent, wxDefaultPosition, wxDefaultSize,
 //    wxSUNKEN_BORDER|wxCLIP_CHILDREN);

+#ifdef __WXMOTIF__
+    wxBORDER);
+#else

     wxSUNKEN_BORDER);
     wxSUNKEN_BORDER);

+#endif

 
   panel->SetScrollbars(10, 10, 100, 100);
 
 
   panel->SetScrollbars(10, 10, 100, 100);
 


@@ -754,7 +765,7 @@ wxToolBar *wxResourceManager::OnCreateToolBar(wxFrame *parent)
   wxBitmap ToolbarToFrontBitmap("TOFRONTTOOL");
   wxBitmap ToolbarHelpBitmap("HELPTOOL");
 #endif
   wxBitmap ToolbarToFrontBitmap("TOFRONTTOOL");
   wxBitmap ToolbarHelpBitmap("HELPTOOL");
 #endif

-#ifdef __WXGTK__
+#if defined(__WXGTK__) || defined(__WXMOTIF__)

   wxBitmap ToolbarLoadBitmap( load_xpm );
   wxBitmap ToolbarSaveBitmap( save_xpm);
   wxBitmap ToolbarNewBitmap( new_xpm );
   wxBitmap ToolbarLoadBitmap( load_xpm );
   wxBitmap ToolbarSaveBitmap( save_xpm);
   wxBitmap ToolbarNewBitmap( new_xpm );


@@ -785,48 +796,48 @@ wxToolBar *wxResourceManager::OnCreateToolBar(wxFrame *parent)
 #endif
   int currentX = gap;
   toolbar->AddSeparator();
 #endif
   int currentX = gap;
   toolbar->AddSeparator();

-  toolbar->AddTool(TOOLBAR_NEW, ToolbarNewBitmap, (wxBitmap *)NULL,
-                   FALSE, (float)currentX, -1, NULL, "New dialog");
+  toolbar->AddTool(TOOLBAR_NEW, ToolbarNewBitmap, wxNullBitmap,
+                   FALSE, currentX, -1, NULL, "New dialog");

   currentX += width + dx;
   currentX += width + dx;

-  toolbar->AddTool(TOOLBAR_LOAD_FILE, ToolbarLoadBitmap, (wxBitmap *)NULL,
-                   FALSE, (float)currentX, -1, NULL, "Load");
+  toolbar->AddTool(TOOLBAR_LOAD_FILE, ToolbarLoadBitmap, wxNullBitmap,
+                   FALSE, currentX, -1, NULL, "Load");

   currentX += width + dx;
   currentX += width + dx;

-  toolbar->AddTool(TOOLBAR_SAVE_FILE, ToolbarSaveBitmap, (wxBitmap *)NULL,
-                   FALSE, (float)currentX, -1, NULL, "Save");
+  toolbar->AddTool(TOOLBAR_SAVE_FILE, ToolbarSaveBitmap, wxNullBitmap,
+                   FALSE, currentX, -1, NULL, "Save");

   currentX += width + dx + gap;
   toolbar->AddSeparator();
   currentX += width + dx + gap;
   toolbar->AddSeparator();

-  toolbar->AddTool(TOOLBAR_FORMAT_HORIZ, ToolbarVertBitmap, (wxBitmap *)NULL,
-                   FALSE, (float)currentX, -1, NULL, "Horizontal align");
+  toolbar->AddTool(TOOLBAR_FORMAT_HORIZ, ToolbarVertBitmap, wxNullBitmap,
+                   FALSE, currentX, -1, NULL, "Horizontal align");

   currentX += width + dx;
   currentX += width + dx;

-  toolbar->AddTool(TOOLBAR_FORMAT_VERT_TOP_ALIGN, ToolbarAlignTBitmap, (wxBitmap *)NULL,
-                   FALSE, (float)currentX, -1, NULL, "Top align");
+  toolbar->AddTool(TOOLBAR_FORMAT_VERT_TOP_ALIGN, ToolbarAlignTBitmap, wxNullBitmap,
+                   FALSE, currentX, -1, NULL, "Top align");

   currentX += width + dx;
   currentX += width + dx;

-  toolbar->AddTool(TOOLBAR_FORMAT_VERT_BOT_ALIGN, ToolbarAlignBBitmap, (wxBitmap *)NULL,
-                   FALSE, (float)currentX, -1, NULL, "Bottom align");
+  toolbar->AddTool(TOOLBAR_FORMAT_VERT_BOT_ALIGN, ToolbarAlignBBitmap, wxNullBitmap,
+                   FALSE, currentX, -1, NULL, "Bottom align");

   currentX += width + dx;
   currentX += width + dx;

-  toolbar->AddTool(TOOLBAR_FORMAT_VERT, ToolbarHorizBitmap, (wxBitmap *)NULL,
-                   FALSE, (float)currentX, -1, NULL, "Vertical align");
+  toolbar->AddTool(TOOLBAR_FORMAT_VERT, ToolbarHorizBitmap, wxNullBitmap,
+                   FALSE, currentX, -1, NULL, "Vertical align");

   currentX += width + dx;
   currentX += width + dx;

-  toolbar->AddTool(TOOLBAR_FORMAT_HORIZ_LEFT_ALIGN, ToolbarAlignLBitmap, (wxBitmap *)NULL,
-                   FALSE, (float)currentX, -1, NULL, "Left align");
+  toolbar->AddTool(TOOLBAR_FORMAT_HORIZ_LEFT_ALIGN, ToolbarAlignLBitmap, wxNullBitmap,
+                   FALSE, currentX, -1, NULL, "Left align");

   currentX += width + dx;
   currentX += width + dx;

-  toolbar->AddTool(TOOLBAR_FORMAT_HORIZ_RIGHT_ALIGN, ToolbarAlignRBitmap, (wxBitmap *)NULL,
-                   FALSE, (float)currentX, -1, NULL, "Right align");
+  toolbar->AddTool(TOOLBAR_FORMAT_HORIZ_RIGHT_ALIGN, ToolbarAlignRBitmap, wxNullBitmap,
+                   FALSE, currentX, -1, NULL, "Right align");

   currentX += width + dx;
   currentX += width + dx;

-  toolbar->AddTool(TOOLBAR_COPY_SIZE, ToolbarCopySizeBitmap, (wxBitmap *)NULL,
-                   FALSE, (float)currentX, -1, NULL, "Copy size");
+  toolbar->AddTool(TOOLBAR_COPY_SIZE, ToolbarCopySizeBitmap, wxNullBitmap,
+                   FALSE, currentX, -1, NULL, "Copy size");

   currentX += width + dx + gap;
   toolbar->AddSeparator();
   currentX += width + dx + gap;
   toolbar->AddSeparator();

-  toolbar->AddTool(TOOLBAR_TO_FRONT, ToolbarToFrontBitmap, (wxBitmap *)NULL,
-                   FALSE, (float)currentX, -1, NULL, "To front");
+  toolbar->AddTool(TOOLBAR_TO_FRONT, ToolbarToFrontBitmap, wxNullBitmap,
+                   FALSE, currentX, -1, NULL, "To front");

   currentX += width + dx;
   currentX += width + dx;

-  toolbar->AddTool(TOOLBAR_TO_BACK, ToolbarToBackBitmap, (wxBitmap *)NULL,
-                   FALSE, (float)currentX, -1, NULL, "To back");
+  toolbar->AddTool(TOOLBAR_TO_BACK, ToolbarToBackBitmap, wxNullBitmap,
+                   FALSE, currentX, -1, NULL, "To back");

   currentX += width + dx + gap;
 
   toolbar->AddSeparator();
   currentX += width + dx + gap;
 
   toolbar->AddSeparator();

-  toolbar->AddTool(TOOLBAR_HELP, ToolbarHelpBitmap, (wxBitmap *)NULL,
-                   FALSE, (float)currentX, -1, NULL, "Help");
+  toolbar->AddTool(TOOLBAR_HELP, ToolbarHelpBitmap, wxNullBitmap,
+                   FALSE, currentX, -1, NULL, "Help");

   currentX += width + dx;
   
   toolbar->Realize();
   currentX += width + dx;
   
   toolbar->Realize();


@@ -842,15 +853,11 @@ void wxResourceManager::UpdateResourceList()
   m_editorResourceTree->SetInvalid(TRUE);
   m_editorResourceTree->DeleteAllItems();
 
   m_editorResourceTree->SetInvalid(TRUE);
   m_editorResourceTree->DeleteAllItems();
 

-  long id = m_editorResourceTree->InsertItem(0, "Dialogs"
-#ifdef __WXMSW__
-     , 1, 2
-#endif
-   );
+  long id = m_editorResourceTree->AddRoot("Dialogs", 1, 2);

 
   m_resourceTable.BeginFind();
   wxNode *node;
 
   m_resourceTable.BeginFind();
   wxNode *node;

-  while (node = m_resourceTable.Next())
+  while ((node = m_resourceTable.Next()))

   {
     wxItemResource *res = (wxItemResource *)node->Data();
     wxString resType(res->GetType());
   {
     wxItemResource *res = (wxItemResource *)node->Data();
     wxString resType(res->GetType());


@@ -859,7 +866,7 @@ void wxResourceManager::UpdateResourceList()
       AddItemsRecursively(id, res);
     }
   }
       AddItemsRecursively(id, res);
     }
   }

-  m_editorResourceTree->ExpandItem(id, wxTREE_EXPAND_EXPAND);
+  m_editorResourceTree->Expand(id);

   m_editorResourceTree->SetInvalid(FALSE);
 }
 
   m_editorResourceTree->SetInvalid(FALSE);
 }
 


@@ -875,13 +882,9 @@ void wxResourceManager::AddItemsRecursively(long parent, wxItemResource *resourc
   else
     imageId = 3;
   
   else
     imageId = 3;
   

-  long id = m_editorResourceTree->InsertItem(parent, theString
-#ifdef __WXMSW__
-     , imageId
-#endif
-   );
+  long id = m_editorResourceTree->AppendItem(parent, theString, imageId );

 
 

-  m_editorResourceTree->SetItemData(id, (long) resource);
+  m_editorResourceTree->SetItemData(id, new wxResourceTreeData(resource));

 
   if (strcmp(resource->GetType(), "wxBitmap") != 0)
   {
 
   if (strcmp(resource->GetType(), "wxBitmap") != 0)
   {


@@ -893,7 +896,7 @@ void wxResourceManager::AddItemsRecursively(long parent, wxItemResource *resourc
       node = node->Next();
     }
   }
       node = node->Next();
     }
   }

-  m_editorResourceTree->ExpandItem(id, wxTREE_EXPAND_EXPAND);
+//  m_editorResourceTree->ExpandItem(id, wxTREE_EXPAND_EXPAND);

 }
 
 bool wxResourceManager::EditSelectedResource()
 }
 
 bool wxResourceManager::EditSelectedResource()


@@ -901,7 +904,8 @@ bool wxResourceManager::EditSelectedResource()
   int sel = m_editorResourceTree->GetSelection();
   if (sel != 0)
   {
   int sel = m_editorResourceTree->GetSelection();
   if (sel != 0)
   {

-    wxItemResource *res = (wxItemResource *)m_editorResourceTree->GetItemData(sel);
+    wxResourceTreeData *data = (wxResourceTreeData *)m_editorResourceTree->GetItemData(sel);
+    wxItemResource *res = data->GetResource();

     return Edit(res);
   }
   return FALSE;
     return Edit(res);
   }
   return FALSE;


@@ -921,8 +925,8 @@ bool wxResourceManager::Edit(wxItemResource *res)
   }
   else
   {
   }
   else
   {

-        long style = res->GetStyle();
-        res->SetStyle(style|wxRAISED_BORDER);
+//        long style = res->GetStyle();
+//        res->SetStyle(style|wxRAISED_BORDER);

         panel = new wxPanel;
         wxResourceEditorDialogHandler *handler = new wxResourceEditorDialogHandler(panel, res, panel->GetEventHandler(),
            this);
         panel = new wxPanel;
         wxResourceEditorDialogHandler *handler = new wxResourceEditorDialogHandler(panel, res, panel->GetEventHandler(),
            this);


@@ -931,7 +935,7 @@ bool wxResourceManager::Edit(wxItemResource *res)
 
         panel->PushEventHandler(handler);
 
 
         panel->PushEventHandler(handler);
 

-        res->SetStyle(style);
+//        res->SetStyle(style);

                handler->AddChildHandlers(); // Add event handlers for all controls
         AssociateResource(res, panel);
 
                handler->AddChildHandlers(); // Add event handlers for all controls
         AssociateResource(res, panel);
 


@@ -957,6 +961,8 @@ bool wxResourceManager::CreateNewPanel()
   resource->SetType("wxDialog");
   resource->SetName(buf);
   resource->SetTitle(buf);
   resource->SetType("wxDialog");
   resource->SetName(buf);
   resource->SetTitle(buf);

+  resource->SetResourceStyle(wxRESOURCE_USE_DEFAULTS);
+  resource->SetResourceStyle(wxRESOURCE_DIALOG_UNITS);

 
   wxString newIdName;
   int id = GenerateWindowId("ID_DIALOG", newIdName);
 
   wxString newIdName;
   int id = GenerateWindowId("ID_DIALOG", newIdName);


@@ -967,13 +973,19 @@ bool wxResourceManager::CreateNewPanel()
 
   m_resourceTable.AddResource(resource);
 
 
   m_resourceTable.AddResource(resource);
 

+  wxSize size(400, 300);
+

   wxPanel *panel = new wxPanel(m_editorPanel, -1,
      wxPoint(m_editorPanel->GetMarginX(), m_editorPanel->GetMarginY()),
   wxPanel *panel = new wxPanel(m_editorPanel, -1,
      wxPoint(m_editorPanel->GetMarginX(), m_editorPanel->GetMarginY()),

-     wxSize(400, 300), wxRAISED_BORDER|wxDEFAULT_DIALOG_STYLE, buf);
+     size, wxRAISED_BORDER|wxDEFAULT_DIALOG_STYLE, buf);

   m_editorPanel->m_childWindow = panel;
 
   resource->SetStyle(panel->GetWindowStyleFlag());
   m_editorPanel->m_childWindow = panel;
 
   resource->SetStyle(panel->GetWindowStyleFlag());

-  resource->SetSize(10, 10, 400, 300);
+
+  // Store dialog units in resource
+  size = panel->ConvertPixelsToDialog(size);
+
+  resource->SetSize(10, 10, size.x, size.y);

 
   // For editing in situ we will need to use the hash table to ensure
   // we don't dereference invalid pointers.
 
   // For editing in situ we will need to use the hash table to ensure
   // we don't dereference invalid pointers.


@@ -1007,7 +1019,14 @@ bool wxResourceManager::CreatePanelItem(wxItemResource *panelResource, wxPanel *
   
   wxItemResource *res = new wxItemResource;
   wxControl *newItem = NULL;
   
   wxItemResource *res = new wxItemResource;
   wxControl *newItem = NULL;

-  res->SetSize(x, y, -1, -1);
+
+  if ((panelResource->GetResourceStyle() & wxRESOURCE_DIALOG_UNITS) != 0)
+  {
+    wxPoint pt = panel->ConvertPixelsToDialog(wxPoint(x, y));
+    res->SetSize(pt.x, pt.y, -1, -1);
+  }
+  else res->SetSize(x, y, -1, -1);
+

   res->SetType(iType);
 
   wxString prefix;
   res->SetType(iType);
 
   wxString prefix;


@@ -1020,7 +1039,7 @@ bool wxResourceManager::CreatePanelItem(wxItemResource *panelResource, wxPanel *
       MakeUniqueName("button", buf);
       res->SetName(buf);
       if (isBitmap)
       MakeUniqueName("button", buf);
       res->SetName(buf);
       if (isBitmap)

-        newItem = new wxBitmapButton(panel, -1, m_bitmapImage, wxPoint(x, y), wxSize(-1, -1), 0, wxDefaultValidator, buf);
+        newItem = new wxBitmapButton(panel, -1, * m_bitmapImage, wxPoint(x, y), wxSize(-1, -1), 0, wxDefaultValidator, buf);

       else
         newItem = new wxButton(panel, -1, "Button", wxPoint(x, y), wxSize(-1, -1), 0, wxDefaultValidator, buf);
     }
       else
         newItem = new wxButton(panel, -1, "Button", wxPoint(x, y), wxSize(-1, -1), 0, wxDefaultValidator, buf);
     }


@@ -1029,7 +1048,7 @@ bool wxResourceManager::CreatePanelItem(wxItemResource *panelResource, wxPanel *
       prefix = "ID_BITMAPBUTTON";
       MakeUniqueName("button", buf);
       res->SetName(buf);
       prefix = "ID_BITMAPBUTTON";
       MakeUniqueName("button", buf);
       res->SetName(buf);

-      newItem = new wxBitmapButton(panel, -1, m_bitmapImage, wxPoint(x, y), wxSize(-1, -1), 0, wxDefaultValidator, buf);
+      newItem = new wxBitmapButton(panel, -1, * m_bitmapImage, wxPoint(x, y), wxSize(-1, -1), 0, wxDefaultValidator, buf);

     }
   else if (itemType == "wxMessage" || itemType == "wxStaticText")
     {
     }
   else if (itemType == "wxMessage" || itemType == "wxStaticText")
     {


@@ -1037,7 +1056,7 @@ bool wxResourceManager::CreatePanelItem(wxItemResource *panelResource, wxPanel *
       MakeUniqueName("statictext", buf);
       res->SetName(buf);
       if (isBitmap)
       MakeUniqueName("statictext", buf);
       res->SetName(buf);
       if (isBitmap)

-        newItem = new wxStaticBitmap(panel, -1, m_bitmapImage, wxPoint(x, y), wxSize(0, 0), 0, buf);
+        newItem = new wxStaticBitmap(panel, -1, * m_bitmapImage, wxPoint(x, y), wxSize(0, 0), 0, buf);

       else
         newItem = new wxStaticText(panel, -1, "Static", wxPoint(x, y), wxSize(-1, -1), 0, buf);
     }
       else
         newItem = new wxStaticText(panel, -1, "Static", wxPoint(x, y), wxSize(-1, -1), 0, buf);
     }


@@ -1046,7 +1065,7 @@ bool wxResourceManager::CreatePanelItem(wxItemResource *panelResource, wxPanel *
       prefix = "ID_STATICBITMAP";
       MakeUniqueName("static", buf);
       res->SetName(buf);
       prefix = "ID_STATICBITMAP";
       MakeUniqueName("static", buf);
       res->SetName(buf);

-      newItem = new wxStaticBitmap(panel, -1, m_bitmapImage, wxPoint(x, y), wxSize(-1, -1), 0, buf);
+      newItem = new wxStaticBitmap(panel, -1, * m_bitmapImage, wxPoint(x, y), wxSize(-1, -1), 0, buf);

     }
   else if (itemType == "wxCheckBox")
     {
     }
   else if (itemType == "wxCheckBox")
     {


@@ -1070,7 +1089,7 @@ bool wxResourceManager::CreatePanelItem(wxItemResource *panelResource, wxPanel *
       wxString names[] = { "One", "Two" };
       newItem = new wxRadioBox(panel, -1, "Radiobox", wxPoint(x, y), wxSize(-1, -1), 2, names, 2,
             wxHORIZONTAL, wxDefaultValidator, buf);
       wxString names[] = { "One", "Two" };
       newItem = new wxRadioBox(panel, -1, "Radiobox", wxPoint(x, y), wxSize(-1, -1), 2, names, 2,
             wxHORIZONTAL, wxDefaultValidator, buf);

-      res->SetStringValues(new wxStringList("One", "Two", NULL));
+      res->SetStringValues(wxStringList("One", "Two", NULL));

     }
   else if (itemType == "wxRadioButton")
     {
     }
   else if (itemType == "wxRadioButton")
     {


@@ -1142,6 +1161,16 @@ bool wxResourceManager::CreatePanelItem(wxItemResource *panelResource, wxPanel *
   if (!newItem)
     return FALSE;
 
   if (!newItem)
     return FALSE;
 

+  int actualW, actualH;
+  newItem->GetSize(&actualW, &actualH);
+  wxSize actualSize(actualW, actualH);
+
+  if ((panelResource->GetResourceStyle() & wxRESOURCE_DIALOG_UNITS) != 0)
+  {
+    actualSize = panel->ConvertPixelsToDialog(actualSize);
+  }
+  res->SetSize(res->GetX(), res->GetY(), actualSize.x, actualSize.y);
+

   wxString newIdName;
   int id = GenerateWindowId(prefix, newIdName);
   res->SetId(id);
   wxString newIdName;
   int id = GenerateWindowId(prefix, newIdName);
   res->SetId(id);


@@ -1184,16 +1213,13 @@ bool wxResourceManager::TestCurrentDialog(wxWindow* parent)
     InstantiateResourceFromWindow(item, m_editorPanel->m_childWindow, TRUE);
 
     wxDialog* dialog = new wxDialog;
     InstantiateResourceFromWindow(item, m_editorPanel->m_childWindow, TRUE);
 
     wxDialog* dialog = new wxDialog;

-    long oldStyle = item->GetStyle();

     bool success = FALSE;
     bool success = FALSE;

-//    item->SetStyle(wxDEFAULT_DIALOG_STYLE);

     if (dialog->LoadFromResource(parent, item->GetName(), & m_resourceTable))
     {
         dialog->Centre();
         dialog->ShowModal();
         success = TRUE;
     }
     if (dialog->LoadFromResource(parent, item->GetName(), & m_resourceTable))
     {
         dialog->Centre();
         dialog->ShowModal();
         success = TRUE;
     }

-//    item->SetStyle(oldStyle);

     return success;
   }
   return FALSE;
     return success;
   }
   return FALSE;


@@ -1205,13 +1231,13 @@ wxWindow *wxResourceManager::FindParentOfSelection()
 {
   m_resourceTable.BeginFind();
   wxNode *node;
 {
   m_resourceTable.BeginFind();
   wxNode *node;

-  while (node = m_resourceTable.Next())
+  while ((node = m_resourceTable.Next()))

   {
     wxItemResource *res = (wxItemResource *)node->Data();
     wxWindow *win = FindWindowForResource(res);
     if (win)
     {
   {
     wxItemResource *res = (wxItemResource *)node->Data();
     wxWindow *win = FindWindowForResource(res);
     if (win)
     {

-      wxNode *node1 = win->GetChildren()->First();
+      wxNode *node1 = win->GetChildren().First();

       while (node1)
       {
         wxControl *item = (wxControl *)node1->Data();
       while (node1)
       {
         wxControl *item = (wxControl *)node1->Data();


@@ -1247,7 +1273,7 @@ void wxResourceManager::AlignItems(int flag)
   int centreX = (int)(firstX + (firstW / 2));
   int centreY = (int)(firstY + (firstH / 2));
 
   int centreX = (int)(firstX + (firstW / 2));
   int centreY = (int)(firstY + (firstH / 2));
 

-  while (node = node->Next())
+  while ((node = node->Next()))

   {
     wxControl *item = (wxControl *)node->Data();
     if (item->GetParent() == win)
   {
     wxControl *item = (wxControl *)node->Data();
     if (item->GetParent() == win)


@@ -1301,7 +1327,22 @@ void wxResourceManager::AlignItems(int flag)
           break;
       }
 
           break;
       }
 

+      wxItemResource* resource = wxResourceManager::GetCurrentResourceManager()->FindResourceForWindow(item);
+      wxItemResource* parentResource = wxResourceManager::GetCurrentResourceManager()->FindResourceForWindow(item->GetParent());
+

       item->SetSize(newX, newY, w, h);
       item->SetSize(newX, newY, w, h);

+
+         // Also update the associated resource
+         // We need to convert to dialog units if this is not a dialog or panel, but
+         // the parent resource specifies dialog units.
+         if (parentResource->GetResourceStyle() & wxRESOURCE_DIALOG_UNITS)
+      {
+        wxPoint pt = item->GetParent()->ConvertPixelsToDialog(wxPoint(newX, newY));
+        newX = pt.x; newY = pt.y;
+        wxSize sz = item->GetParent()->ConvertPixelsToDialog(wxSize(w, h));
+        w = sz.x; h = sz.y;
+      }
+      resource->SetSize(newX, newY, w, h);

     }
   }
   win->Refresh();
     }
   }
   win->Refresh();


@@ -1326,14 +1367,28 @@ void wxResourceManager::CopySize()
   int firstW, firstH;
   firstSelection->GetPosition(&firstX, &firstY);
   firstSelection->GetSize(&firstW, &firstH);
   int firstW, firstH;
   firstSelection->GetPosition(&firstX, &firstY);
   firstSelection->GetSize(&firstW, &firstH);

-  int centreX = (int)(firstX + (firstW / 2));
-  int centreY = (int)(firstY + (firstH / 2));

 
 

-  while (node = node->Next())
+  while ((node = node->Next()))

   {
     wxControl *item = (wxControl *)node->Data();
     if (item->GetParent() == win)
   {
     wxControl *item = (wxControl *)node->Data();
     if (item->GetParent() == win)

+       {

       item->SetSize(-1, -1, firstW, firstH);
       item->SetSize(-1, -1, firstW, firstH);

+
+      wxItemResource* resource = wxResourceManager::GetCurrentResourceManager()->FindResourceForWindow(item);
+      wxItemResource* parentResource = wxResourceManager::GetCurrentResourceManager()->FindResourceForWindow(item->GetParent());
+
+         // Also update the associated resource
+         // We need to convert to dialog units if this is not a dialog or panel, but
+         // the parent resource specifies dialog units.
+         if (parentResource->GetResourceStyle() & wxRESOURCE_DIALOG_UNITS)
+      {
+        wxSize sz = item->GetParent()->ConvertPixelsToDialog(wxSize(firstW, firstH));
+        firstW = sz.x; firstH = sz.y;
+      }
+      resource->SetSize(resource->GetX(), resource->GetY(), firstW, firstH);
+
+       }

   }
   win->Refresh();
 }
   }
   win->Refresh();
 }


@@ -1352,18 +1407,18 @@ void wxResourceManager::ToBackOrFront(bool toBack)
     wxItemResource *itemResource = FindResourceForWindow(item);
     if (item->GetParent() == win)
     {
     wxItemResource *itemResource = FindResourceForWindow(item);
     if (item->GetParent() == win)
     {

-      win->GetChildren()->DeleteObject(item);
+      win->GetChildren().DeleteObject(item);

       if (winResource)
         winResource->GetChildren().DeleteObject(itemResource);
       if (toBack)
       {
       if (winResource)
         winResource->GetChildren().DeleteObject(itemResource);
       if (toBack)
       {

-        win->GetChildren()->Insert(item);
+        win->GetChildren().Insert(item);

         if (winResource)
           winResource->GetChildren().Insert(itemResource);
       }
       else
       {
         if (winResource)
           winResource->GetChildren().Insert(itemResource);
       }
       else
       {

-        win->GetChildren()->Append(item);
+        win->GetChildren().Append(item);

         if (winResource)
           winResource->GetChildren().Append(itemResource);
       }
         if (winResource)
           winResource->GetChildren().Append(itemResource);
       }


@@ -1390,7 +1445,7 @@ bool wxResourceManager::RemoveResourceFromParent(wxItemResource *res)
 {
   m_resourceTable.BeginFind();
   wxNode *node;
 {
   m_resourceTable.BeginFind();
   wxNode *node;

-  while (node = m_resourceTable.Next())
+  while ((node = m_resourceTable.Next()))

   {
     wxItemResource *thisRes = (wxItemResource *)node->Data();
     if (thisRes->GetChildren().Member(res))
   {
     wxItemResource *thisRes = (wxItemResource *)node->Data();
     if (thisRes->GetChildren().Member(res))


@@ -1421,11 +1476,13 @@ bool wxResourceManager::DeleteResource(wxItemResource *res)
   // If this is a button or message resource, delete the
   // associate bitmap resource if not being used.
   wxString resType(res->GetType());
   // If this is a button or message resource, delete the
   // associate bitmap resource if not being used.
   wxString resType(res->GetType());

-  
+
+/* shouldn't have to do this now bitmaps are ref-counted

   if ((resType == "wxMessage" || resType == "wxStaticBitmap" || resType == "wxButton" || resType == "wxBitmapButton") && res->GetValue4())
   {
     PossiblyDeleteBitmapResource(res->GetValue4());
   }
   if ((resType == "wxMessage" || resType == "wxStaticBitmap" || resType == "wxButton" || resType == "wxBitmapButton") && res->GetValue4())
   {
     PossiblyDeleteBitmapResource(res->GetValue4());
   }

+*/

 
   // Remove symbol from table if appropriate
   if (!IsSymbolUsed(res, res->GetId()))
 
   // Remove symbol from table if appropriate
   if (!IsSymbolUsed(res, res->GetId()))


@@ -1451,7 +1508,9 @@ bool wxResourceManager::DeleteResource(wxWindow *win)
     {
       RemoveSelection(item);
       childHandler->SelectItem(FALSE);
     {
       RemoveSelection(item);
       childHandler->SelectItem(FALSE);

+#ifndef __WXGTK__

       item->GetParent()->Refresh();
       item->GetParent()->Refresh();

+#endif

     }
   }
   
     }
   }
   


@@ -1466,7 +1525,7 @@ bool wxResourceManager::DeleteResource(wxWindow *win)
 
 // Will eventually have bitmap type information, for different
 // kinds of bitmap.
 
 // Will eventually have bitmap type information, for different
 // kinds of bitmap.

-char *wxResourceManager::AddBitmapResource(char *filename)
+wxString wxResourceManager::AddBitmapResource(const wxString& filename)

 {
   wxItemResource *resource = FindBitmapResourceByFilename(filename);
   if (!resource)
 {
   wxItemResource *resource = FindBitmapResourceByFilename(filename);
   if (!resource)


@@ -1495,11 +1554,11 @@ char *wxResourceManager::AddBitmapResource(char *filename)
   if (resource)
     return resource->GetName();
   else
   if (resource)
     return resource->GetName();
   else

-    return NULL;
+    return wxEmptyString;

 }
 
  // Delete the bitmap resource if it isn't being used by another resource.
 }
 
  // Delete the bitmap resource if it isn't being used by another resource.

-void wxResourceManager::PossiblyDeleteBitmapResource(char *resourceName)
+void wxResourceManager::PossiblyDeleteBitmapResource(const wxString& resourceName)

 {
   if (!IsBitmapResourceUsed(resourceName))
   {
 {
   if (!IsBitmapResourceUsed(resourceName))
   {


@@ -1509,11 +1568,11 @@ void wxResourceManager::PossiblyDeleteBitmapResource(char *resourceName)
   }
 }
 
   }
 }
 

-bool wxResourceManager::IsBitmapResourceUsed(char *resourceName)
+bool wxResourceManager::IsBitmapResourceUsed(const wxString& resourceName)

 {
   m_resourceTable.BeginFind();
   wxNode *node;
 {
   m_resourceTable.BeginFind();
   wxNode *node;

-  while (node = m_resourceTable.Next())
+  while ((node = m_resourceTable.Next()))

   {
     wxItemResource *res = (wxItemResource *)node->Data();
     wxString resType(res->GetType());
   {
     wxItemResource *res = (wxItemResource *)node->Data();
     wxString resType(res->GetType());


@@ -1537,13 +1596,13 @@ bool wxResourceManager::IsBitmapResourceUsed(char *resourceName)
 }
 
 // Given a wxButton or wxMessage, find the corresponding bitmap filename.
 }
 
 // Given a wxButton or wxMessage, find the corresponding bitmap filename.

-char *wxResourceManager::FindBitmapFilenameForResource(wxItemResource *resource)
+wxString wxResourceManager::FindBitmapFilenameForResource(wxItemResource *resource)

 {
 {

-  if (!resource || !resource->GetValue4())
-    return NULL;
+  if (!resource || (resource->GetValue4() == ""))
+    return wxEmptyString;

   wxItemResource *bitmapResource = m_resourceTable.FindResource(resource->GetValue4());
   if (!bitmapResource)
   wxItemResource *bitmapResource = m_resourceTable.FindResource(resource->GetValue4());
   if (!bitmapResource)

-    return NULL;
+    return wxEmptyString;

 
   wxNode *node = bitmapResource->GetChildren().First();
   while (node)
 
   wxNode *node = bitmapResource->GetChildren().First();
   while (node)


@@ -1556,14 +1615,14 @@ char *wxResourceManager::FindBitmapFilenameForResource(wxItemResource *resource)
     
     node = node->Next();
   }
     
     node = node->Next();
   }

-  return NULL;
+  return wxEmptyString;

 }
 
 }
 

-wxItemResource *wxResourceManager::FindBitmapResourceByFilename(char *filename)
+wxItemResource *wxResourceManager::FindBitmapResourceByFilename(const wxString& filename)

 {
   m_resourceTable.BeginFind();
   wxNode *node;
 {
   m_resourceTable.BeginFind();
   wxNode *node;

-  while (node = m_resourceTable.Next())
+  while ((node = m_resourceTable.Next()))

   {
     wxItemResource *res = (wxItemResource *)node->Data();
     wxString resType(res->GetType());
   {
     wxItemResource *res = (wxItemResource *)node->Data();
     wxString resType(res->GetType());


@@ -1590,7 +1649,7 @@ bool wxResourceManager::IsSymbolUsed(wxItemResource* thisResource, wxWindowID id
 {
   m_resourceTable.BeginFind();
   wxNode *node;
 {
   m_resourceTable.BeginFind();
   wxNode *node;

-  while (node = m_resourceTable.Next())
+  while ((node = m_resourceTable.Next()))

   {
     wxItemResource *res = (wxItemResource *)node->Data();
 
   {
     wxItemResource *res = (wxItemResource *)node->Data();
 


@@ -1633,7 +1692,7 @@ void wxResourceManager::ChangeIds(int oldId, int newId)
 {
   m_resourceTable.BeginFind();
   wxNode *node;
 {
   m_resourceTable.BeginFind();
   wxNode *node;

-  while (node = m_resourceTable.Next())
+  while ((node = m_resourceTable.Next()))

   {
     wxItemResource *res = (wxItemResource *)node->Data();
 
   {
     wxItemResource *res = (wxItemResource *)node->Data();
 


@@ -1665,7 +1724,7 @@ bool wxResourceManager::RepairResourceIds()
 
   m_resourceTable.BeginFind();
   wxNode *node;
 
   m_resourceTable.BeginFind();
   wxNode *node;

-  while (node = m_resourceTable.Next())
+  while ((node = m_resourceTable.Next()))

   {
     wxItemResource *res = (wxItemResource *)node->Data();
     wxString resType(res->GetType());
   {
     wxItemResource *res = (wxItemResource *)node->Data();
     wxString resType(res->GetType());


@@ -1741,6 +1800,9 @@ wxWindow *wxResourceManager::RecreateWindowFromResource(wxWindow *win, wxWindowP
 
   wxWindow *newWin = NULL;
   wxWindow *parent = win->GetParent();
 
   wxWindow *newWin = NULL;
   wxWindow *parent = win->GetParent();

+  wxItemResource* parentResource = NULL;
+  if (parent)
+    parentResource = FindResourceForWindow(parent);

   
   if (win->IsKindOf(CLASSINFO(wxPanel)))
   {
   
   if (win->IsKindOf(CLASSINFO(wxPanel)))
   {


@@ -1754,7 +1816,7 @@ wxWindow *wxResourceManager::RecreateWindowFromResource(wxWindow *win, wxWindowP
         win->PopEventHandler(TRUE);
 
     DeleteWindow(win);
         win->PopEventHandler(TRUE);
 
     DeleteWindow(win);

-    newWin = m_resourceTable.CreateItem((wxPanel *)parent, resource);
+    newWin = m_resourceTable.CreateItem((wxPanel *)parent, resource, parentResource);

     newWin->PushEventHandler(new wxResourceEditorControlHandler((wxControl*) newWin, (wxControl*) newWin));
     AssociateResource(resource, newWin);
     UpdateResourceList();
     newWin->PushEventHandler(new wxResourceEditorControlHandler((wxControl*) newWin, (wxControl*) newWin));
     AssociateResource(resource, newWin);
     UpdateResourceList();


@@ -1775,7 +1837,8 @@ bool wxResourceManager::DeleteSelection()
   int sel = m_editorResourceTree->GetSelection();
   if (sel != 0)
   {
   int sel = m_editorResourceTree->GetSelection();
   if (sel != 0)
   {

-    wxItemResource *res = (wxItemResource *)m_editorResourceTree->GetItemData(sel);
+    wxResourceTreeData *data = (wxResourceTreeData *)m_editorResourceTree->GetItemData(sel);
+    wxItemResource *res = data->GetResource();

     wxWindow *win = FindWindowForResource(res);
     if (win)
     {
     wxWindow *win = FindWindowForResource(res);
     if (win)
     {


@@ -1810,7 +1873,7 @@ bool wxResourceManager::RecreateSelection()
   return TRUE;
 }
 
   return TRUE;
 }
 

-bool wxResourceManager::EditDialog(wxDialog *dialog, wxWindow *parent)
+bool wxResourceManager::EditDialog(wxDialog *WXUNUSED(dialog), wxWindow *WXUNUSED(parent))

 {
   return FALSE;
 }
 {
   return FALSE;
 }


@@ -1821,7 +1884,7 @@ bool wxResourceManager::InstantiateAllResourcesFromWindows()
 {
   m_resourceTable.BeginFind();
   wxNode *node;
 {
   m_resourceTable.BeginFind();
   wxNode *node;

-  while (node = m_resourceTable.Next())
+  while ((node = m_resourceTable.Next()))

   {
     wxItemResource *res = (wxItemResource *)node->Data();
     wxString resType(res->GetType());
   {
     wxItemResource *res = (wxItemResource *)node->Data();
     wxString resType(res->GetType());


@@ -1860,7 +1923,7 @@ bool wxResourceManager::InstantiateResourceFromWindow(wxItemResource *resource,
       if (!childWindow)
       {
         char buf[200];
       if (!childWindow)
       {
         char buf[200];

-        sprintf(buf, "Could not find window %s", child->GetName());
+        sprintf(buf, "Could not find window %s", (const char*) child->GetName());

         wxMessageBox(buf, "Dialog Editor problem", wxOK);
       }
       else
         wxMessageBox(buf, "Dialog Editor problem", wxOK);
       }
       else


@@ -1908,30 +1971,30 @@ wxWindowPropertyInfo *wxResourceManager::CreatePropertyInfoForWindow(wxWindow *w
         {
           info = new wxRadioButtonPropertyInfo(win);
         }
         {
           info = new wxRadioButtonPropertyInfo(win);
         }

-  else if (win->IsKindOf(CLASSINFO(wxChoice)))
-        {
-          info = new wxChoicePropertyInfo(win);
-        }

   else if (win->IsKindOf(CLASSINFO(wxComboBox)))
         {
           info = new wxComboBoxPropertyInfo(win);
         }
   else if (win->IsKindOf(CLASSINFO(wxComboBox)))
         {
           info = new wxComboBoxPropertyInfo(win);
         }

-  else if (win->IsKindOf(CLASSINFO(wxButton)))
+  else if (win->IsKindOf(CLASSINFO(wxChoice)))

         {
         {

-          info = new wxButtonPropertyInfo(win);
+          info = new wxChoicePropertyInfo(win);

         }
   else if (win->IsKindOf(CLASSINFO(wxBitmapButton)))
         {
           info = new wxBitmapButtonPropertyInfo(win);
         }
         }
   else if (win->IsKindOf(CLASSINFO(wxBitmapButton)))
         {
           info = new wxBitmapButtonPropertyInfo(win);
         }

-  else if (win->IsKindOf(CLASSINFO(wxStaticText)))
+  else if (win->IsKindOf(CLASSINFO(wxButton)))

         {
         {

-          info = new wxStaticTextPropertyInfo(win);
+          info = new wxButtonPropertyInfo(win);

         }
   else if (win->IsKindOf(CLASSINFO(wxStaticBitmap)))
         {
           info = new wxStaticBitmapPropertyInfo(win);
         }
         }
   else if (win->IsKindOf(CLASSINFO(wxStaticBitmap)))
         {
           info = new wxStaticBitmapPropertyInfo(win);
         }

+  else if (win->IsKindOf(CLASSINFO(wxStaticText)))
+        {
+          info = new wxStaticTextPropertyInfo(win);
+        }

   else if (win->IsKindOf(CLASSINFO(wxTextCtrl)))
         {
           info = new wxTextPropertyInfo(win);
   else if (win->IsKindOf(CLASSINFO(wxTextCtrl)))
         {
           info = new wxTextPropertyInfo(win);


@@ -2008,6 +2071,7 @@ BEGIN_EVENT_TABLE(wxResourceEditorFrame, wxFrame)
     EVT_MENU(RESED_DELETE, wxResourceEditorFrame::OnDeleteSelection)
     EVT_MENU(RESED_RECREATE, wxResourceEditorFrame::OnRecreateSelection)
     EVT_MENU(RESED_TEST, wxResourceEditorFrame::OnTest)
     EVT_MENU(RESED_DELETE, wxResourceEditorFrame::OnDeleteSelection)
     EVT_MENU(RESED_RECREATE, wxResourceEditorFrame::OnRecreateSelection)
     EVT_MENU(RESED_TEST, wxResourceEditorFrame::OnTest)

+    EVT_CLOSE(wxResourceEditorFrame::OnCloseWindow)

 END_EVENT_TABLE()
 
 wxResourceEditorFrame::wxResourceEditorFrame(wxResourceManager *resMan, wxFrame *parent, const wxString& title,
 END_EVENT_TABLE()
 
 wxResourceEditorFrame::wxResourceEditorFrame(wxResourceManager *resMan, wxFrame *parent, const wxString& title,


@@ -2053,8 +2117,8 @@ void wxResourceEditorFrame::OnSaveAs(wxCommandEvent& WXUNUSED(event))
 
 void wxResourceEditorFrame::OnExit(wxCommandEvent& WXUNUSED(event))
 {
 
 void wxResourceEditorFrame::OnExit(wxCommandEvent& WXUNUSED(event))
 {

-         manager->Clear(TRUE, FALSE) ;
-      this->Close();
+      manager->Clear(TRUE, FALSE) ;
+      this->Destroy();

 }
 
 void wxResourceEditorFrame::OnAbout(wxCommandEvent& WXUNUSED(event))
 }
 
 void wxResourceEditorFrame::OnAbout(wxCommandEvent& WXUNUSED(event))


@@ -2089,12 +2153,15 @@ void wxResourceEditorFrame::OnRecreateSelection(wxCommandEvent& WXUNUSED(event))
       manager->RecreateSelection();
 }
 
       manager->RecreateSelection();
 }
 

-bool wxResourceEditorFrame::OnClose()
+void wxResourceEditorFrame::OnCloseWindow(wxCloseEvent& event)

 {
   if (manager->Modified())
   {
 {
   if (manager->Modified())
   {

-  if (!manager->Clear(TRUE, FALSE))
-     return FALSE;
+     if (!manager->Clear(TRUE, FALSE))
+     {
+       event.Veto();
+       return;
+    }

   }
     
   if (!Iconized())
   }
     
   if (!Iconized())


@@ -2113,7 +2180,7 @@ bool wxResourceEditorFrame::OnClose()
   manager->SetEditorFrame(NULL);
   manager->SetEditorToolBar(NULL);
 
   manager->SetEditorFrame(NULL);
   manager->SetEditorToolBar(NULL);
 

-  return TRUE;
+  this->Destroy();

 }
 
 /*
 }
 
 /*


@@ -2132,6 +2199,8 @@ wxResourceEditorScrolledWindow::wxResourceEditorScrolledWindow(wxWindow *parent,
     m_marginX = 10;
     m_marginY = 40;
     m_childWindow = NULL;
     m_marginX = 10;
     m_marginY = 40;
     m_childWindow = NULL;

+
+    SetBackgroundColour(* wxWHITE);

 }
 
 wxResourceEditorScrolledWindow::~wxResourceEditorScrolledWindow()
 }
 
 wxResourceEditorScrolledWindow::~wxResourceEditorScrolledWindow()


@@ -2211,7 +2280,7 @@ void ObjectMenuProc(wxMenu& menu, wxCommandEvent& event)
  *
  */
 
  *
  */
 

-#ifdef __WXGTK__   // I don't dare to delete it...
+#if defined(__WXGTK__) || defined(__WXMOTIF__)   // I don't dare to delete it...

  
 BEGIN_EVENT_TABLE(EditorToolBar, wxToolBar)
 END_EVENT_TABLE()
  
 BEGIN_EVENT_TABLE(EditorToolBar, wxToolBar)
 END_EVENT_TABLE()


@@ -2230,7 +2299,7 @@ EditorToolBar::EditorToolBar(wxFrame *frame, const wxPoint& pos, const wxSize& s
 {
 }
 
 {
 }
 

-bool EditorToolBar::OnLeftClick(int toolIndex, bool toggled)
+bool EditorToolBar::OnLeftClick(int toolIndex, bool WXUNUSED(toggled))

 {
   wxResourceManager *manager = wxResourceManager::GetCurrentResourceManager();
 
 {
   wxResourceManager *manager = wxResourceManager::GetCurrentResourceManager();