]> git.saurik.com Git - wxWidgets.git/blobdiff - wxPython/wx/tools/XRCed/params.py
Use wx.FileHistory for the recent files menu
[wxWidgets.git] / wxPython / wx / tools / XRCed / params.py
index 442c9c22570c52be8c2acb02dc46c754347aeb7f..7142449f847963328d686208c44793040386c9bb 100644 (file)
@@ -8,7 +8,6 @@ import string
 import os.path
 from globals import *
 from types import *
-from wxPython.xrc import *
 
 genericStyles = [
     'wxSIMPLE_BORDER', 'wxSUNKEN_BORDER', 'wxDOUBLE_BORDER',
@@ -26,21 +25,25 @@ genericExStyles = [
     'wxWS_EX_PROCESS_UI_UPDATES'
     ]
 
-buttonSize = (35,-1)    # in dialog units, transformed to pixels in panel ctor
+# Global var initialized in Panel.__init__ for button size in screen pixels
+buttonSize = None
+# Button size in dialog units
+buttonSizeD = (35,-1)
 
 # Class that can properly disable children
-class PPanel(wxPanel):
+class PPanel(wx.Panel):
     def __init__(self, parent, name):
-        wxPanel.__init__(self, parent, -1, name=name)
+        wx.Panel.__init__(self, parent, -1, name=name)
         self.modified = self.freeze = False
     def Enable(self, value):
+        self.enabled = value
         # Something strange is going on with enable so we make sure...
         for w in self.GetChildren():
             w.Enable(value)
-        #wxPanel.Enable(self, value)
-    def SetModified(self):
-        self.modified = True
-        g.panel.SetModified(True)
+        #wx.Panel.Enable(self, value)
+    def SetModified(self, state=True):
+        self.modified = state
+        if state: g.panel.SetModified(True)
     # Common method to set modified state
     def OnChange(self, evt):
         if self.freeze: return
@@ -50,18 +53,16 @@ class PPanel(wxPanel):
 class ParamBinaryOr(PPanel):
     def __init__(self, parent, name):
         PPanel.__init__(self, parent, name)
-        self.ID_TEXT_CTRL = wxNewId()
-        self.ID_BUTTON_CHOICES = wxNewId()
-        sizer = wxBoxSizer()
-        self.text = wxTextCtrl(self, self.ID_TEXT_CTRL, size=wxSize(200,-1))
-        sizer.Add(self.text, 0, wxRIGHT | wxALIGN_CENTER_VERTICAL, 5)
-        self.button = wxButton(self, self.ID_BUTTON_CHOICES, 'Edit...', size=buttonSize)
-        sizer.Add(self.button, 0, wxALIGN_CENTER_VERTICAL)
-        self.SetAutoLayout(True)
+        self.ID_TEXT_CTRL = wx.NewId()
+        self.ID_BUTTON_CHOICES = wx.NewId()
+        sizer = wx.BoxSizer()
+        self.text = wx.TextCtrl(self, self.ID_TEXT_CTRL, size=wx.Size(200,-1))
+        sizer.Add(self.text, 0, wx.RIGHT | wx.ALIGN_CENTER_VERTICAL, 5)
+        self.button = wx.Button(self, self.ID_BUTTON_CHOICES, 'Edit...', size=buttonSize)
+        sizer.Add(self.button, 0, wx.ALIGN_CENTER_VERTICAL)
         self.SetSizer(sizer)
-        sizer.Fit(self)
-        EVT_BUTTON(self, self.ID_BUTTON_CHOICES, self.OnButtonChoices)
-        EVT_TEXT(self, self.ID_TEXT_CTRL, self.OnChange)
+        wx.EVT_BUTTON(self, self.ID_BUTTON_CHOICES, self.OnButtonChoices)
+        wx.EVT_TEXT(self, self.ID_TEXT_CTRL, self.OnChange)
     def GetValue(self):
         return self.text.GetValue()
     def SetValue(self, value):
@@ -73,7 +74,7 @@ class ParamBinaryOr(PPanel):
         if self.GetName() == 'flag':  dlg.SetTitle('Sizer item flags')
         elif self.GetName() == 'style':  dlg.SetTitle('Window styles')
         elif self.GetName() == 'exstyle':  dlg.SetTitle('Extended window styles')
-        listBox = XRCCTRL(dlg, 'CHECKLIST')
+        listBox = xrc.XRCCTRL(dlg, 'CHECKLIST')
         listBox.InsertItems(self.values, 0)
         value = map(string.strip, self.text.GetValue().split('|'))
         if value == ['']: value = []
@@ -88,17 +89,14 @@ class ParamBinaryOr(PPanel):
                 else:
                     print 'WARNING: unknown flag: %s: ignored.' % i
                     ignored.append(i)
-        if dlg.ShowModal() == wxID_OK:
+        if dlg.ShowModal() == wx.ID_OK:
             value = []
             for i in range(listBox.GetCount()):
                 if listBox.IsChecked(i):
                     value.append(self.values[i])
             # Add ignored flags
             value.extend(ignored)
-            if value:
-                self.SetValue(reduce(lambda a,b: a+'|'+b, value))
-            else:
-                self.SetValue('')
+            self.SetValue('|'.join(value))
             self.SetModified()
         dlg.Destroy()
 
@@ -132,14 +130,14 @@ class ParamStyle(ParamBinaryOr):
             # Remove duplicates
             self.valuesGeneric = [s for s in genericStyles
                                   if s not in self.valuesSpecific]
-            EVT_BUTTON(self, self.ID_BUTTON_CHOICES, self.OnButtonChoicesBoth)
+            wx.EVT_BUTTON(self, self.ID_BUTTON_CHOICES, self.OnButtonChoicesBoth)
         else:
             self.values = genericStyles
     def OnButtonChoicesBoth(self, evt):
         dlg = g.frame.res.LoadDialog(self, 'DIALOG_STYLES')
-        listBoxSpecific = XRCCTRL(dlg, 'CHECKLIST_SPECIFIC')
+        listBoxSpecific = xrc.XRCCTRL(dlg, 'CHECKLIST_SPECIFIC')
         listBoxSpecific.InsertItems(self.valuesSpecific, 0)
-        listBoxGeneric = XRCCTRL(dlg, 'CHECKLIST_GENERIC')
+        listBoxGeneric = xrc.XRCCTRL(dlg, 'CHECKLIST_GENERIC')
         listBoxGeneric.InsertItems(self.valuesGeneric, 0)
         value = map(string.strip, self.text.GetValue().split('|'))
         if value == ['']: value = []
@@ -166,17 +164,14 @@ class ParamStyle(ParamBinaryOr):
                 else:
                     print 'WARNING: unknown flag: %s: ignored.' % i
                     ignored.append(i)
-        if dlg.ShowModal() == wxID_OK:
+        if dlg.ShowModal() == wx.ID_OK:
             value = [self.valuesSpecific[i]
                      for i in range(listBoxSpecific.GetCount())
                      if listBoxSpecific.IsChecked(i)] + \
                      [self.valuesGeneric[i]
                       for i in range(listBoxGeneric.GetCount())
                       if listBoxGeneric.IsChecked(i)] + ignored
-            if value:
-                self.SetValue(reduce(lambda a,b: a+'|'+b, value))
-            else:
-                self.SetValue('')
+            self.SetValue('|'.join(value))
             self.SetModified()
         dlg.Destroy()
 
@@ -191,20 +186,18 @@ class ParamExStyle(ParamBinaryOr):
 class ParamColour(PPanel):
     def __init__(self, parent, name):
         PPanel.__init__(self, parent, name)
-        self.ID_TEXT_CTRL = wxNewId()
-        self.ID_BUTTON = wxNewId()
-        sizer = wxBoxSizer()
-        self.text = wxTextCtrl(self, self.ID_TEXT_CTRL, size=(80,-1))
-        sizer.Add(self.text, 0, wxALIGN_CENTER_VERTICAL | wxTOP | wxBOTTOM, 2)
-        self.button = wxPanel(self, self.ID_BUTTON, wxDefaultPosition, wxSize(20, 20))
-        sizer.Add(self.button, 0, wxALIGN_CENTER_VERTICAL | wxLEFT, 5)
-        self.SetAutoLayout(True)
+        self.ID_TEXT_CTRL = wx.NewId()
+        self.ID_BUTTON = wx.NewId()
+        sizer = wx.BoxSizer()
+        self.text = wx.TextCtrl(self, self.ID_TEXT_CTRL, size=(80,-1))
+        sizer.Add(self.text, 0, wx.ALIGN_CENTER_VERTICAL | wx.TOP | wx.BOTTOM, 2)
+        self.button = wx.Panel(self, self.ID_BUTTON, wx.DefaultPosition, wx.Size(20, 20))
+        sizer.Add(self.button, 0, wx.ALIGN_CENTER_VERTICAL | wx.LEFT, 5)
         self.SetSizer(sizer)
-        sizer.Fit(self)
         self.textModified = False
-        EVT_PAINT(self.button, self.OnPaintButton)
-        EVT_TEXT(self, self.ID_TEXT_CTRL, self.OnChange)
-        EVT_LEFT_DOWN(self.button, self.OnLeftDown)
+        wx.EVT_PAINT(self.button, self.OnPaintButton)
+        wx.EVT_TEXT(self, self.ID_TEXT_CTRL, self.OnChange)
+        wx.EVT_LEFT_DOWN(self.button, self.OnLeftDown)
     def GetValue(self):
         return self.text.GetValue()
     def SetValue(self, value):
@@ -212,24 +205,24 @@ class ParamColour(PPanel):
         if not value: value = '#FFFFFF'
         self.text.SetValue(str(value))  # update text ctrl
         try:
-            colour = wxColour(int(value[1:3], 16), int(value[3:5], 16), int(value[5:7], 16))
+            colour = wx.Colour(int(value[1:3], 16), int(value[3:5], 16), int(value[5:7], 16))
             self.button.SetBackgroundColour(colour)
         except:                         # ignore errors
             pass
         self.button.Refresh()
         self.freeze = False
     def OnPaintButton(self, evt):
-        dc = wxPaintDC(self.button)
-        dc.SetBrush(wxTRANSPARENT_BRUSH)
-        if self.IsEnabled(): dc.SetPen(wxBLACK_PEN)
-        else: dc.SetPen(wxGREY_PEN)
+        dc = wx.PaintDC(self.button)
+        dc.SetBrush(wx.TRANSPARENT_BRUSH)
+        if self.IsEnabled(): dc.SetPen(wx.BLACK_PEN)
+        else: dc.SetPen(wx.GREY_PEN)
         size = self.button.GetSize()
         dc.DrawRectangle(0, 0, size.width, size.height)
     def OnLeftDown(self, evt):
-        data = wxColourData()
+        data = wx.ColourData()
         data.SetColour(self.GetValue())
-        dlg = wxColourDialog(self, data)
-        if dlg.ShowModal() == wxID_OK:
+        dlg = wx.ColourDialog(self, data)
+        if dlg.ShowModal() == wx.ID_OK:
             self.SetValue('#%02X%02X%02X' % dlg.GetColourData().GetColour().Get())
             self.SetModified()
         dlg.Destroy()
@@ -237,11 +230,11 @@ class ParamColour(PPanel):
 ################################################################################
 
 # Mapping from wx constants to XML strings
-fontFamiliesWx2Xml = {wxDEFAULT: 'default', wxDECORATIVE: 'decorative',
-                wxROMAN: 'roman', wxSCRIPT: 'script', wxSWISS: 'swiss',
-                wxMODERN: 'modern'}
-fontStylesWx2Xml = {wxNORMAL: 'normal', wxSLANT: 'slant', wxITALIC: 'italic'}
-fontWeightsWx2Xml = {wxNORMAL: 'normal', wxLIGHT: 'light', wxBOLD: 'bold'}
+fontFamiliesWx2Xml = {wx.DEFAULT: 'default', wx.DECORATIVE: 'decorative',
+                wx.ROMAN: 'roman', wx.SCRIPT: 'script', wx.SWISS: 'swiss',
+                wx.MODERN: 'modern'}
+fontStylesWx2Xml = {wx.NORMAL: 'normal', wx.SLANT: 'slant', wx.ITALIC: 'italic'}
+fontWeightsWx2Xml = {wx.NORMAL: 'normal', wx.LIGHT: 'light', wx.BOLD: 'bold'}
 def ReverseMap(m):
     rm = {}
     for k,v in m.items(): rm[v] = k
@@ -253,19 +246,17 @@ fontWeightsXml2wx = ReverseMap(fontWeightsWx2Xml)
 class ParamFont(PPanel):
     def __init__(self, parent, name):
         PPanel.__init__(self, parent, name)
-        self.ID_TEXT_CTRL = wxNewId()
-        self.ID_BUTTON_SELECT = wxNewId()
-        sizer = wxBoxSizer()
-        self.text = wxTextCtrl(self, self.ID_TEXT_CTRL, size=(200,-1))
-        sizer.Add(self.text, 0, wxRIGHT | wxALIGN_CENTER_VERTICAL, 5)
-        self.button = wxButton(self, self.ID_BUTTON_SELECT, 'Select...', size=buttonSize)
-        sizer.Add(self.button, 0, wxALIGN_CENTER_VERTICAL)
-        self.SetAutoLayout(True)
+        self.ID_TEXT_CTRL = wx.NewId()
+        self.ID_BUTTON_SELECT = wx.NewId()
+        sizer = wx.BoxSizer()
+        self.text = wx.TextCtrl(self, self.ID_TEXT_CTRL, size=(200,-1))
+        sizer.Add(self.text, 0, wx.RIGHT | wx.ALIGN_CENTER_VERTICAL, 5)
+        self.button = wx.Button(self, self.ID_BUTTON_SELECT, 'Select...', size=buttonSize)
+        sizer.Add(self.button, 0, wx.ALIGN_CENTER_VERTICAL)
         self.SetSizer(sizer)
-        sizer.Fit(self)
         self.textModified = False
-        EVT_BUTTON(self, self.ID_BUTTON_SELECT, self.OnButtonSelect)
-        EVT_TEXT(self, self.ID_TEXT_CTRL, self.OnChange)
+        wx.EVT_BUTTON(self, self.ID_BUTTON_SELECT, self.OnButtonSelect)
+        wx.EVT_TEXT(self, self.ID_TEXT_CTRL, self.OnChange)
     def OnChange(self, evt):
         PPanel.OnChange(self, evt)
         self.textModified = True
@@ -276,7 +267,7 @@ class ParamFont(PPanel):
             try:
                 return eval(self.text.GetValue())
             except SyntaxError:
-                wxLogError('Syntax error in parameter value: ' + self.GetName())
+                wx.LogError('Syntax error in parameter value: ' + self.GetName())
                 return self._defaultValue()
         return self.value
     def SetValue(self, value):
@@ -290,48 +281,47 @@ class ParamFont(PPanel):
             try:
                 self.value = eval(self.text.GetValue())
             except SyntaxError:
-                wxLogError('Syntax error in parameter value: ' + self.GetName())
+                wx.LogError('Syntax error in parameter value: ' + self.GetName())
                 self.value = self._defaultValue()
         # Make initial font
         # Default values
         size = g._sysFont.GetPointSize()
-        family = wxDEFAULT
-        style = weight = wxNORMAL
+        family = wx.DEFAULT
+        style = weight = wx.NORMAL
         underlined = 0
         face = ''
-        enc = wxFONTENCODING_DEFAULT
+        enc = wx.FONTENCODING_DEFAULT
         # Fall back to default if exceptions
         error = False
         try:
             try: size = int(self.value[0])
-            except ValueError: error = True; wxLogError('Invalid size specification')
+            except ValueError: error = True; wx.LogError('Invalid size specification')
             try: family = fontFamiliesXml2wx[self.value[1]]
-            except KeyError: error = True; wxLogError('Invalid family specification')
+            except KeyError: error = True; wx.LogError('Invalid family specification')
             try: style = fontStylesXml2wx[self.value[2]]
-            except KeyError: error = True; wxLogError('Invalid style specification')
+            except KeyError: error = True; wx.LogError('Invalid style specification')
             try: weight = fontWeightsXml2wx[self.value[3]]
-            except KeyError: error = True; wxLogError('Invalid weight specification')
+            except KeyError: error = True; wx.LogError('Invalid weight specification')
             try: underlined = bool(self.value[4])
-            except ValueError: error = True; wxLogError('Invalid underlined flag specification')
+            except ValueError: error = True; wx.LogError('Invalid underlined flag specification')
             face = self.value[5]
         except IndexError:
             error = True
-        mapper = wxFontMapper()
+        mapper = wx.FontMapper()
         if not self.value[6]: enc = mapper.CharsetToEncoding(self.value[6])
             
-        if error: wxLogError('Invalid font specification')
-        if enc == wxFONTENCODING_DEFAULT: enc = wxFONTENCODING_SYSTEM
-        font = wxFont(size, family, style, weight, underlined, face, enc)
-        data = wxFontData()
+        if error: wx.LogError('Invalid font specification')
+        if enc == wx.FONTENCODING_DEFAULT: enc = wx.FONTENCODING_SYSTEM
+        font = wx.Font(size, family, style, weight, underlined, face, enc)
+        data = wx.FontData()
         data.SetInitialFont(font)
-        dlg = wxFontDialog(self, data)
-        if dlg.ShowModal() == wxID_OK:
+        dlg = wx.FontDialog(self, data)
+        if dlg.ShowModal() == wx.ID_OK:
             font = dlg.GetFontData().GetChosenFont()
-            print font.GetEncoding()
-            if font.GetEncoding() == wxFONTENCODING_SYSTEM:
+            if font.GetEncoding() == wx.FONTENCODING_SYSTEM:
                 encName = ''
             else:
-                encName = wxFontMapper_GetEncodingName(font.GetEncoding()).encode()
+                encName = wx.FontMapper.GetEncodingName(font.GetEncoding()).encode()
             value = [str(font.GetPointSize()),
                      fontFamiliesWx2Xml.get(font.GetFamily(), "default"),
                      fontStylesWx2Xml.get(font.GetStyle(), "normal"),
@@ -350,15 +340,13 @@ class ParamFont(PPanel):
 class ParamInt(PPanel):
     def __init__(self, parent, name):
         PPanel.__init__(self, parent, name)
-        self.ID_SPIN_CTRL = wxNewId()
-        sizer = wxBoxSizer()
-        self.spin = wxSpinCtrl(self, self.ID_SPIN_CTRL, size=(60,-1))
+        self.ID_SPIN_CTRL = wx.NewId()
+        sizer = wx.BoxSizer()
+        self.spin = wx.SpinCtrl(self, self.ID_SPIN_CTRL, size=(60,-1))
         self.spin.SetRange(-2147483648, 2147483647) # min/max integers
         sizer.Add(self.spin)
-        self.SetAutoLayout(True)
         self.SetSizer(sizer)
-        sizer.Fit(self)
-        EVT_SPINCTRL(self, self.ID_SPIN_CTRL, self.OnChange)
+        wx.EVT_SPINCTRL(self, self.ID_SPIN_CTRL, self.OnChange)
     def GetValue(self):
         return str(self.spin.GetValue())
     def SetValue(self, value):
@@ -371,15 +359,13 @@ class ParamInt(PPanel):
 class ParamIntNN(PPanel):
     def __init__(self, parent, name):
         PPanel.__init__(self, parent, name)
-        self.ID_SPIN_CTRL = wxNewId()
-        sizer = wxBoxSizer()
-        self.spin = wxSpinCtrl(self, self.ID_SPIN_CTRL, size=(60,-1))
+        self.ID_SPIN_CTRL = wx.NewId()
+        sizer = wx.BoxSizer()
+        self.spin = wx.SpinCtrl(self, self.ID_SPIN_CTRL, size=(60,-1))
         self.spin.SetRange(0, 10000) # min/max integers
         sizer.Add(self.spin)
-        self.SetAutoLayout(True)
         self.SetSizer(sizer)
-        sizer.Fit(self)
-        EVT_SPINCTRL(self, self.ID_SPIN_CTRL, self.OnChange)
+        wx.EVT_SPINCTRL(self, self.ID_SPIN_CTRL, self.OnChange)
     def GetValue(self):
         return str(self.spin.GetValue())
     def SetValue(self, value):
@@ -392,29 +378,26 @@ class ParamIntNN(PPanel):
 class ParamUnit(PPanel):
     def __init__(self, parent, name):
         PPanel.__init__(self, parent, name)
-        self.ID_TEXT_CTRL = wxNewId()
-        self.ID_SPIN_BUTTON = wxNewId()
-        sizer = wxBoxSizer(wxHORIZONTAL)
-        self.spin = wxSpinButton(self, self.ID_SPIN_BUTTON, style = wxSP_VERTICAL, size=(-1,1))
+        self.ID_TEXT_CTRL = wx.NewId()
+        self.ID_SPIN_BUTTON = wx.NewId()
+        sizer = wx.BoxSizer(wx.HORIZONTAL)
+        self.spin = wx.SpinButton(self, self.ID_SPIN_BUTTON, style = wx.SP_VERTICAL, size=(-1,0))
         textW = 60 - self.spin.GetSize()[0]
-        self.text = wxTextCtrl(self, self.ID_TEXT_CTRL, size=(textW,-1))
+        self.text = wx.TextCtrl(self, self.ID_TEXT_CTRL, size=(textW,-1))
         self.spin.SetRange(-10000, 10000)
-        sizer.Add(self.text, 0, wxEXPAND)
-        sizer.Add(self.spin, 0, wxEXPAND)
-        #sizer.SetMinSize((50,-1))
-        self.SetAutoLayout(True)
+        sizer.Add(self.text, 0, wx.EXPAND)
+        sizer.Add(self.spin, 0, wx.EXPAND)
         self.SetSizer(sizer)
-        sizer.Fit(self)
-        EVT_SPIN_UP(self, self.ID_SPIN_BUTTON, self.OnSpinUp)
-        EVT_SPIN_DOWN(self, self.ID_SPIN_BUTTON, self.OnSpinDown)
+        self.spin.Bind(wx.EVT_SPIN_UP, self.OnSpinUp)
+        self.spin.Bind(wx.EVT_SPIN_DOWN, self.OnSpinDown)
     def GetValue(self):
         return self.text.GetValue()
     def SetValue(self, value):
-        self.freeze = True
-        if not value: value = '0'
+        if not value: value = '0'        
         self.text.SetValue(value)
-        self.freeze = False
+        self.Change(0)
     def Change(self, x):
+        self.freeze = True
         # Check if we are working with dialog units
         value = self.text.GetValue()
         units = ''
@@ -424,31 +407,34 @@ class ParamUnit(PPanel):
         try:
             intValue = int(value) + x
             self.spin.SetValue(intValue)
-            self.text.SetValue(str(intValue) + units)
-            self.SetModified()
+            if x:                       # 0 can be passed to update spin value only
+                self.text.SetValue(str(intValue) + units)
+                self.SetModified()
         except:
-            # !!! Strange, if I use wxLogWarning, event is re-generated
-            print 'incorrect unit format'
+            # !!! Strange, if I use wx.LogWarning, event is re-generated
+            print 'ERROR: incorrect unit format'
+        self.freeze = False
     def OnSpinUp(self, evt):
+        self.freeze = True
         self.Change(1)
     def OnSpinDown(self, evt):
+        if self.freeze: return
+        self.freeze = True
         self.Change(-1)
 
 class ParamMultilineText(PPanel):
     def __init__(self, parent, name, textWidth=-1):
         PPanel.__init__(self, parent, name)
-        self.ID_TEXT_CTRL = wxNewId()
-        self.ID_BUTTON_EDIT = wxNewId()
-        sizer = wxBoxSizer()
-        self.text = wxTextCtrl(self, self.ID_TEXT_CTRL, size=wxSize(200,-1))
-        sizer.Add(self.text, 0, wxRIGHT | wxALIGN_CENTER_VERTICAL, 5)
-        self.button = wxButton(self, self.ID_BUTTON_EDIT, 'Edit...', size=buttonSize)
-        sizer.Add(self.button, 0, wxALIGN_CENTER_VERTICAL)
-        self.SetAutoLayout(True)
-        self.SetSizer(sizer)
-        sizer.Fit(self)
-        EVT_BUTTON(self, self.ID_BUTTON_EDIT, self.OnButtonEdit)
-        EVT_TEXT(self, self.ID_TEXT_CTRL, self.OnChange)
+        self.ID_TEXT_CTRL = wx.NewId()
+        self.ID_BUTTON_EDIT = wx.NewId()
+        sizer = wx.BoxSizer()
+        self.text = wx.TextCtrl(self, self.ID_TEXT_CTRL, size=wx.Size(200,-1))
+        sizer.Add(self.text, 0, wx.RIGHT | wx.ALIGN_CENTER_VERTICAL, 5)
+        self.button = wx.Button(self, self.ID_BUTTON_EDIT, 'Edit...', size=buttonSize)
+        sizer.Add(self.button, 0, wx.ALIGN_CENTER_VERTICAL)
+        self.SetSizerAndFit(sizer)
+        wx.EVT_BUTTON(self, self.ID_BUTTON_EDIT, self.OnButtonEdit)
+        wx.EVT_TEXT(self, self.ID_TEXT_CTRL, self.OnChange)
     def GetValue(self):
         return self.text.GetValue()
     def SetValue(self, value):
@@ -457,27 +443,25 @@ class ParamMultilineText(PPanel):
         self.freeze = False             # disable other handlers
     def OnButtonEdit(self, evt):
         dlg = g.frame.res.LoadDialog(self, 'DIALOG_TEXT')
-        textCtrl = XRCCTRL(dlg, 'TEXT')
+        textCtrl = xrc.XRCCTRL(dlg, 'TEXT')
         textCtrl.SetValue(self.text.GetValue())
-        if dlg.ShowModal() == wxID_OK:
+        if dlg.ShowModal() == wx.ID_OK:
             self.text.SetValue(textCtrl.GetValue())
             self.SetModified()
         dlg.Destroy()
 
 class ParamText(PPanel):
-    def __init__(self, parent, name, textWidth=-1):
+    def __init__(self, parent, name, textWidth=-1, style=0):
         PPanel.__init__(self, parent, name)
-        self.ID_TEXT_CTRL = wxNewId()
+        self.ID_TEXT_CTRL = wx.NewId()
         # We use sizer even here to have the same size of text control
-        sizer = wxBoxSizer()
-        self.text = wxTextCtrl(self, self.ID_TEXT_CTRL, size=wxSize(textWidth,-1))
+        sizer = wx.BoxSizer()
+        self.text = wx.TextCtrl(self, self.ID_TEXT_CTRL, size=wx.Size(textWidth,-1), style=style)
         if textWidth == -1: option = 1
         else: option = 0
-        sizer.Add(self.text, option, wxALIGN_CENTER_VERTICAL | wxTOP | wxBOTTOM, 2)
-        self.SetAutoLayout(True)
+        sizer.Add(self.text, option, wx.ALIGN_CENTER_VERTICAL | wx.TOP | wx.BOTTOM, 2)
         self.SetSizer(sizer)
-        sizer.Fit(self)
-        EVT_TEXT(self, self.ID_TEXT_CTRL, self.OnChange)
+        wx.EVT_TEXT(self, self.ID_TEXT_CTRL, self.OnChange)
     def GetValue(self):
         return self.text.GetValue()
     def SetValue(self, value):
@@ -501,31 +485,35 @@ class ParamEncoding(ParamText):
     def __init__(self, parent, name):
         ParamText.__init__(self, parent, name, 100)
 
-class ContentDialog(wxDialog):
+class ParamComment(ParamText):
+    def __init__(self, parent, name):
+        ParamText.__init__(self, parent, name, 330 + buttonSize[0],
+                           style=wx.TE_PROCESS_ENTER)
+
+class ContentDialog(wx.Dialog):
     def __init__(self, parent, value):
         # Load from resource
-        pre = wxPreDialog()
+        pre = wx.PreDialog()
         g.frame.res.LoadOnDialog(pre, parent, 'DIALOG_CONTENT')
-        self.this = pre.this
-        self._setOORInfo(self)
-        self.list = XRCCTRL(self, 'LIST')
+        self.PostCreate(pre)
+        self.list = xrc.XRCCTRL(self, 'LIST')
         # Set list items
         for v in value:
             self.list.Append(v)
         self.SetAutoLayout(True)
         self.GetSizer().Fit(self)
         # Callbacks
-        self.ID_BUTTON_APPEND = XRCID('BUTTON_APPEND')
-        self.ID_BUTTON_REMOVE = XRCID('BUTTON_REMOVE')
-        self.ID_BUTTON_UP = XRCID('BUTTON_UP')
-        self.ID_BUTTON_DOWN = XRCID('BUTTON_DOWN')
-        EVT_BUTTON(self, self.ID_BUTTON_UP, self.OnButtonUp)
-        EVT_BUTTON(self, self.ID_BUTTON_DOWN, self.OnButtonDown)
-        EVT_BUTTON(self, self.ID_BUTTON_APPEND, self.OnButtonAppend)
-        EVT_BUTTON(self, self.ID_BUTTON_REMOVE, self.OnButtonRemove)
-        EVT_UPDATE_UI(self, self.ID_BUTTON_UP, self.OnUpdateUI)
-        EVT_UPDATE_UI(self, self.ID_BUTTON_DOWN, self.OnUpdateUI)
-        EVT_UPDATE_UI(self, self.ID_BUTTON_REMOVE, self.OnUpdateUI)
+        self.ID_BUTTON_APPEND = xrc.XRCID('BUTTON_APPEND')
+        self.ID_BUTTON_REMOVE = xrc.XRCID('BUTTON_REMOVE')
+        self.ID_BUTTON_UP = xrc.XRCID('BUTTON_UP')
+        self.ID_BUTTON_DOWN = xrc.XRCID('BUTTON_DOWN')
+        wx.EVT_BUTTON(self, self.ID_BUTTON_UP, self.OnButtonUp)
+        wx.EVT_BUTTON(self, self.ID_BUTTON_DOWN, self.OnButtonDown)
+        wx.EVT_BUTTON(self, self.ID_BUTTON_APPEND, self.OnButtonAppend)
+        wx.EVT_BUTTON(self, self.ID_BUTTON_REMOVE, self.OnButtonRemove)
+        wx.EVT_UPDATE_UI(self, self.ID_BUTTON_UP, self.OnUpdateUI)
+        wx.EVT_UPDATE_UI(self, self.ID_BUTTON_DOWN, self.OnUpdateUI)
+        wx.EVT_UPDATE_UI(self, self.ID_BUTTON_REMOVE, self.OnUpdateUI)
     def OnButtonUp(self, evt):
         i = self.list.GetSelection()
         str = self.list.GetString(i)
@@ -539,7 +527,7 @@ class ContentDialog(wxDialog):
         self.list.InsertItems([str], i+1)
         self.list.SetSelection(i+1)
     def OnButtonAppend(self, evt):
-        str = wxGetTextFromUser('Enter new item:', 'Append', '', self)
+        str = wx.GetTextFromUser('Enter new item:', 'Append', '', self)
         self.list.Append(str)
     def OnButtonRemove(self, evt):
         self.list.Delete(self.list.GetSelection())
@@ -552,13 +540,12 @@ class ContentDialog(wxDialog):
             evt.Enable(self.list.GetSelection() != -1 and \
                        self.list.GetSelection() < self.list.GetCount() - 1)
 
-class ContentCheckListDialog(wxDialog):
+class ContentCheckListDialog(wx.Dialog):
     def __init__(self, parent, value):
-        pre = wxPreDialog()
+        pre = wx.PreDialog()
         g.frame.res.LoadOnDialog(pre, parent, 'DIALOG_CONTENT_CHECKLIST')
-        self.this = pre.this
-        self._setOORInfo(self)
-        self.list = XRCCTRL(self, 'CHECKLIST')
+        self.PostCreate(pre)
+        self.list = xrc.XRCCTRL(self, 'CHECK_LIST')
         # Set list items
         i = 0
         for v,ch in value:
@@ -568,18 +555,18 @@ class ContentCheckListDialog(wxDialog):
         self.SetAutoLayout(True)
         self.GetSizer().Fit(self)
         # Callbacks
-        self.ID_BUTTON_APPEND = XRCID('BUTTON_APPEND')
-        self.ID_BUTTON_REMOVE = XRCID('BUTTON_REMOVE')
-        self.ID_BUTTON_UP = XRCID('BUTTON_UP')
-        self.ID_BUTTON_DOWN = XRCID('BUTTON_DOWN')
-        EVT_CHECKLISTBOX(self, self.list.GetId(), self.OnCheck)
-        EVT_BUTTON(self, self.ID_BUTTON_UP, self.OnButtonUp)
-        EVT_BUTTON(self, self.ID_BUTTON_DOWN, self.OnButtonDown)
-        EVT_BUTTON(self, self.ID_BUTTON_APPEND, self.OnButtonAppend)
-        EVT_BUTTON(self, self.ID_BUTTON_REMOVE, self.OnButtonRemove)
-        EVT_UPDATE_UI(self, self.ID_BUTTON_UP, self.OnUpdateUI)
-        EVT_UPDATE_UI(self, self.ID_BUTTON_DOWN, self.OnUpdateUI)
-        EVT_UPDATE_UI(self, self.ID_BUTTON_REMOVE, self.OnUpdateUI)
+        self.ID_BUTTON_APPEND = xrc.XRCID('BUTTON_APPEND')
+        self.ID_BUTTON_REMOVE = xrc.XRCID('BUTTON_REMOVE')
+        self.ID_BUTTON_UP = xrc.XRCID('BUTTON_UP')
+        self.ID_BUTTON_DOWN = xrc.XRCID('BUTTON_DOWN')
+        wx.EVT_CHECKLISTBOX(self, self.list.GetId(), self.OnCheck)
+        wx.EVT_BUTTON(self, self.ID_BUTTON_UP, self.OnButtonUp)
+        wx.EVT_BUTTON(self, self.ID_BUTTON_DOWN, self.OnButtonDown)
+        wx.EVT_BUTTON(self, self.ID_BUTTON_APPEND, self.OnButtonAppend)
+        wx.EVT_BUTTON(self, self.ID_BUTTON_REMOVE, self.OnButtonRemove)
+        wx.EVT_UPDATE_UI(self, self.ID_BUTTON_UP, self.OnUpdateUI)
+        wx.EVT_UPDATE_UI(self, self.ID_BUTTON_DOWN, self.OnUpdateUI)
+        wx.EVT_UPDATE_UI(self, self.ID_BUTTON_REMOVE, self.OnUpdateUI)
     def OnCheck(self, evt):
         # !!! Wrong wxGTK (wxMSW?) behavior: toggling selection if checking
         self.list.Deselect(evt.GetSelection())
@@ -598,7 +585,7 @@ class ContentCheckListDialog(wxDialog):
         self.list.Check(i+1, ch)
         self.list.SetSelection(i+1)
     def OnButtonAppend(self, evt):
-        str = wxGetTextFromUser('Enter new item:', 'Append', '', self)
+        str = wx.GetTextFromUser('Enter new item:', 'Append', '', self)
         self.list.Append(str)
     def OnButtonRemove(self, evt):
         self.list.Delete(self.list.GetSelection())
@@ -614,45 +601,39 @@ class ContentCheckListDialog(wxDialog):
 class ParamContent(PPanel):
     def __init__(self, parent, name):
         PPanel.__init__(self, parent, name)
-        self.ID_TEXT_CTRL = wxNewId()
-        self.ID_BUTTON_EDIT = wxNewId()
-        sizer = wxBoxSizer()
-        self.text = wxTextCtrl(self, self.ID_TEXT_CTRL, size=wxSize(200,-1))
-        sizer.Add(self.text, 0, wxRIGHT | wxALIGN_CENTER_VERTICAL, 5)
-        self.button = wxButton(self, self.ID_BUTTON_EDIT, 'Edit...', size=buttonSize)
-        sizer.Add(self.button, 0, wxALIGN_CENTER_VERTICAL)
-        self.SetAutoLayout(True)
+        self.ID_TEXT_CTRL = wx.NewId()
+        self.ID_BUTTON_EDIT = wx.NewId()
+        sizer = wx.BoxSizer()
+        self.text = wx.TextCtrl(self, self.ID_TEXT_CTRL, size=wx.Size(200,-1))
+        sizer.Add(self.text, 0, wx.RIGHT | wx.ALIGN_CENTER_VERTICAL, 5)
+        self.button = wx.Button(self, self.ID_BUTTON_EDIT, 'Edit...', size=buttonSize)
+        sizer.Add(self.button, 0, wx.ALIGN_CENTER_VERTICAL)
         self.SetSizer(sizer)
-        sizer.Fit(self)
         self.textModified = False
-        EVT_BUTTON(self, self.ID_BUTTON_EDIT, self.OnButtonEdit)
-        EVT_TEXT(self, self.ID_TEXT_CTRL, self.OnChange)
+        wx.EVT_BUTTON(self, self.ID_BUTTON_EDIT, self.OnButtonEdit)
+        wx.EVT_TEXT(self, self.ID_TEXT_CTRL, self.OnChange)
     def OnChange(self, evt):
         PPanel.OnChange(self, evt)
         self.textModified = True
     def GetValue(self):
         if self.textModified:           # text has newer value
             try:
-                return eval(self.text.GetValue())
-            except SyntaxError:
-                wxLogError('Syntax error in parameter value: ' + self.GetName())
+                return self.text.GetValue().split('|')
+            except ValueError:
                 return []
         return self.value
     def SetValue(self, value):
         self.freeze = True
         if not value: value = []
         self.value = value
-        self.text.SetValue(str(value))  # update text ctrl
+        repr_ = '|'.join(map(str, value))
+        self.text.SetValue(repr_)  # update text ctrl
         self.freeze = False
     def OnButtonEdit(self, evt):
         if self.textModified:           # text has newer value
-            try:
-                self.value = eval(self.text.GetValue())
-            except SyntaxError:
-                wxLogError('Syntax error in parameter value: ' + self.GetName())
-                self.value = []
+            self.value = self.GetValue()
         dlg = ContentDialog(self, self.value)
-        if dlg.ShowModal() == wxID_OK:
+        if dlg.ShowModal() == wx.ID_OK:
             value = []
             for i in range(dlg.list.GetCount()):
                 value.append(dlg.list.GetString(i))
@@ -660,6 +641,9 @@ class ParamContent(PPanel):
             self.SetModified()
             self.textModified = False
         dlg.Destroy()
+    def SetModified(self, state=True):
+        PPanel.SetModified(self, state)
+        self.textModified = False
 
 # CheckList content
 class ParamContentCheckList(ParamContent):
@@ -667,13 +651,9 @@ class ParamContentCheckList(ParamContent):
         ParamContent.__init__(self, parent, name)
     def OnButtonEdit(self, evt):
         if self.textModified:           # text has newer value
-            try:
-                self.value = eval(self.text.GetValue())
-            except SyntaxError:
-                wxLogError('Syntax error in parameter value: ' + self.GetName())
-                self.value = []
+            self.value = self.GetValue()
         dlg = ContentCheckListDialog(self, self.value)
-        if dlg.ShowModal() == wxID_OK:
+        if dlg.ShowModal() == wx.ID_OK:
             value = []
             for i in range(dlg.list.GetCount()):
                 value.append((dlg.list.GetString(i), int(dlg.list.IsChecked(i))))
@@ -681,30 +661,36 @@ class ParamContentCheckList(ParamContent):
             self.SetModified()
             self.textModified = False
         dlg.Destroy()
+    def SetValue(self, value):
+        self.freeze = True
+        if not value: value = []
+        self.value = value
+        repr_ = '|'.join(map(str,value))
+        self.text.SetValue(repr_)  # update text ctrl
+        self.freeze = False        
 
-class IntListDialog(wxDialog):
+class IntListDialog(wx.Dialog):
     def __init__(self, parent, value):
-        pre = wxPreDialog()
+        pre = wx.PreDialog()
         g.frame.res.LoadOnDialog(pre, parent, 'DIALOG_INTLIST')
-        self.this = pre.this
-        self._setOORInfo(self)
-        self.list = XRCCTRL(self, 'LIST')
+        self.PostCreate(pre)
+        self.list = xrc.XRCCTRL(self, 'LIST')
         # Set list items
         value.sort()
         for v in value:
             if type(v) != IntType:
-                wxLogError('Invalid item type')
+                wx.LogError('Invalid item type')
             else:
                 self.list.Append(str(v))
         self.SetAutoLayout(True)
         self.GetSizer().Fit(self)
         # Callbacks
-        self.spinCtrl = XRCCTRL(self, 'SPIN')
-        EVT_BUTTON(self, XRCID('BUTTON_ADD'), self.OnButtonAdd)
-        self.ID_BUTTON_REMOVE = XRCID('BUTTON_REMOVE')
-        EVT_BUTTON(self, self.ID_BUTTON_REMOVE, self.OnButtonRemove)
-        EVT_BUTTON(self, XRCID('BUTTON_CLEAR'), self.OnButtonClear)
-        EVT_UPDATE_UI(self, self.ID_BUTTON_REMOVE, self.OnUpdateUI)
+        self.spinCtrl = xrc.XRCCTRL(self, 'SPIN')
+        wx.EVT_BUTTON(self, xrc.XRCID('BUTTON_ADD'), self.OnButtonAdd)
+        self.ID_BUTTON_REMOVE = xrc.XRCID('BUTTON_REMOVE')
+        wx.EVT_BUTTON(self, self.ID_BUTTON_REMOVE, self.OnButtonRemove)
+        wx.EVT_BUTTON(self, xrc.XRCID('BUTTON_CLEAR'), self.OnButtonClear)
+        wx.EVT_UPDATE_UI(self, self.ID_BUTTON_REMOVE, self.OnUpdateUI)
     def OnButtonAdd(self, evt):
         # Check that it's unique
         try:
@@ -721,7 +707,7 @@ class IntListDialog(wxDialog):
                 if found: self.list.InsertItems([s], i)
                 else: self.list.Append(s)
         except ValueError:
-            wxLogError('List item is not an int!')
+            wx.LogError('List item is not an int!')
     def OnButtonRemove(self, evt):
         self.list.Delete(self.list.GetSelection())
     def OnButtonClear(self, evt):
@@ -737,12 +723,11 @@ class ParamIntList(ParamContent):
     def OnButtonEdit(self, evt):
         if self.textModified:           # text has newer value
             try:
-                self.value = eval(self.text.GetValue())
-            except SyntaxError:
-                wxLogError('Syntax error in parameter value: ' + self.GetName())
+                self.value = map(int, self.text.GetValue().split('|'))
+            except ValueError:
                 self.value = []
         dlg = IntListDialog(self, self.value)
-        if dlg.ShowModal() == wxID_OK:
+        if dlg.ShowModal() == wx.ID_OK:
             value = []
             for i in range(dlg.list.GetCount()):
                 value.append(int(dlg.list.GetString(i)))
@@ -754,17 +739,15 @@ class ParamIntList(ParamContent):
 # Boxless radiobox
 class RadioBox(PPanel):
     def __init__(self, parent, id, choices,
-                 pos=wxDefaultPosition, name='radiobox'):
+                 pos=wx.DefaultPosition, name='radiobox'):
         PPanel.__init__(self, parent, name)
         self.choices = choices
-        topSizer = wxBoxSizer()
+        topSizer = wx.BoxSizer()
         for i in choices:
-            button = wxRadioButton(self, -1, i, size=(-1,buttonSize[1]), name=i)
-            topSizer.Add(button, 0, wxRIGHT, 5)
-            EVT_RADIOBUTTON(self, button.GetId(), self.OnRadioChoice)
-        self.SetAutoLayout(True)
+            button = wx.RadioButton(self, -1, i, size=(-1,buttonSize[1]), name=i)
+            topSizer.Add(button, 0, wx.RIGHT, 5)
+            wx.EVT_RADIOBUTTON(self, button.GetId(), self.OnRadioChoice)
         self.SetSizer(topSizer)
-        topSizer.Fit(self)
     def SetStringSelection(self, value):
         self.freeze = True
         for i in self.choices:
@@ -815,19 +798,17 @@ class ParamOrientation(RadioBox):
 class ParamFile(PPanel):
     def __init__(self, parent, name):
         PPanel.__init__(self, parent, name)
-        self.ID_TEXT_CTRL = wxNewId()
-        self.ID_BUTTON_BROWSE = wxNewId()
-        sizer = wxBoxSizer()
-        self.text = wxTextCtrl(self, self.ID_TEXT_CTRL, size=wxSize(200,-1))
-        sizer.Add(self.text, 0, wxRIGHT | wxALIGN_CENTER_VERTICAL, 5)
-        self.button = wxButton(self, self.ID_BUTTON_BROWSE, 'Browse...',size=buttonSize)
-        sizer.Add(self.button, 0, wxALIGN_CENTER_VERTICAL)
-        self.SetAutoLayout(True)
+        self.ID_TEXT_CTRL = wx.NewId()
+        self.ID_BUTTON_BROWSE = wx.NewId()
+        sizer = wx.BoxSizer()
+        self.text = wx.TextCtrl(self, self.ID_TEXT_CTRL, size=wx.Size(200,-1))
+        sizer.Add(self.text, 0, wx.RIGHT | wx.ALIGN_CENTER_VERTICAL, 5)
+        self.button = wx.Button(self, self.ID_BUTTON_BROWSE, 'Browse...',size=buttonSize)
+        sizer.Add(self.button, 0, wx.ALIGN_CENTER_VERTICAL)
         self.SetSizer(sizer)
-        sizer.Fit(self)
         self.textModified = False
-        EVT_BUTTON(self, self.ID_BUTTON_BROWSE, self.OnButtonBrowse)
-        EVT_TEXT(self, self.ID_TEXT_CTRL, self.OnChange)
+        wx.EVT_BUTTON(self, self.ID_BUTTON_BROWSE, self.OnButtonBrowse)
+        wx.EVT_TEXT(self, self.ID_TEXT_CTRL, self.OnChange)
     def OnChange(self, evt):
         PPanel.OnChange(self, evt)
         self.textModified = True
@@ -843,10 +824,10 @@ class ParamFile(PPanel):
     def OnButtonBrowse(self, evt):
         if self.textModified:           # text has newer value
             self.value = self.text.GetValue()
-        dlg = wxFileDialog(self,
+        dlg = wx.FileDialog(self,
                            defaultDir = os.path.abspath(os.path.dirname(self.value)),
                            defaultFile = os.path.basename(self.value))
-        if dlg.ShowModal() == wxID_OK:
+        if dlg.ShowModal() == wx.ID_OK:
             # Get common part of selected path and current
             if g.frame.dataFile:
                 curpath = os.path.abspath(g.frame.dataFile)
@@ -860,26 +841,25 @@ class ParamFile(PPanel):
 
 class ParamBitmap(PPanel):
     def __init__(self, parent, name):
-        pre = wxPrePanel()
+        pre = wx.PrePanel()
         g.frame.res.LoadOnPanel(pre, parent, 'PANEL_BITMAP')
-        self.this = pre.this
-        self._setOORInfo(self)
+        self.PostCreate(pre)
         self.modified = self.freeze = False
-        self.radio_std = XRCCTRL(self, 'RADIO_STD')
-        self.radio_file = XRCCTRL(self, 'RADIO_FILE')
-        self.combo = XRCCTRL(self, 'COMBO_STD')
-        self.text = XRCCTRL(self, 'TEXT_FILE')
-        self.button = XRCCTRL(self, 'BUTTON_BROWSE')
+        self.radio_std = xrc.XRCCTRL(self, 'RADIO_STD')
+        self.radio_file = xrc.XRCCTRL(self, 'RADIO_FILE')
+        self.combo = xrc.XRCCTRL(self, 'COMBO_STD')
+        self.text = xrc.XRCCTRL(self, 'TEXT_FILE')
+        self.button = xrc.XRCCTRL(self, 'BUTTON_BROWSE')
         self.textModified = False
         self.SetAutoLayout(True)
         self.GetSizer().SetMinSize((260, -1))
         self.GetSizer().Fit(self)
-        EVT_RADIOBUTTON(self, XRCID('RADIO_STD'), self.OnRadioStd)
-        EVT_RADIOBUTTON(self, XRCID('RADIO_FILE'), self.OnRadioFile)
-        EVT_BUTTON(self, XRCID('BUTTON_BROWSE'), self.OnButtonBrowse)
-        EVT_COMBOBOX(self, XRCID('COMBO_STD'), self.OnCombo)
-        EVT_TEXT(self, XRCID('COMBO_STD'), self.OnChange)
-        EVT_TEXT(self, XRCID('TEXT_FILE'), self.OnChange)
+        wx.EVT_RADIOBUTTON(self, xrc.XRCID('RADIO_STD'), self.OnRadioStd)
+        wx.EVT_RADIOBUTTON(self, xrc.XRCID('RADIO_FILE'), self.OnRadioFile)
+        wx.EVT_BUTTON(self, xrc.XRCID('BUTTON_BROWSE'), self.OnButtonBrowse)
+        wx.EVT_COMBOBOX(self, xrc.XRCID('COMBO_STD'), self.OnCombo)
+        wx.EVT_TEXT(self, xrc.XRCID('COMBO_STD'), self.OnChange)
+        wx.EVT_TEXT(self, xrc.XRCID('TEXT_FILE'), self.OnChange)
     def OnRadioStd(self, evt):
         self.SetModified()
         self.SetValue(['wxART_MISSING_IMAGE',''])
@@ -922,10 +902,10 @@ class ParamBitmap(PPanel):
     def OnButtonBrowse(self, evt):
         if self.textModified:           # text has newer value
             self.value[1] = self.text.GetValue()
-        dlg = wxFileDialog(self,
+        dlg = wx.FileDialog(self,
                            defaultDir = os.path.abspath(os.path.dirname(self.value[1])),
                            defaultFile = os.path.basename(self.value[1]))
-        if dlg.ShowModal() == wxID_OK:
+        if dlg.ShowModal() == wx.ID_OK:
             # Get common part of selected path and current
             if g.frame.dataFile:
                 curpath = os.path.abspath(g.frame.dataFile)
@@ -952,5 +932,6 @@ paramDict = {
     'fg': ParamColour, 'bg': ParamColour, 'font': ParamFont,
     'enabled': ParamBool, 'focused': ParamBool, 'hidden': ParamBool,
     'tooltip': ParamText, 'bitmap': ParamBitmap, 'icon': ParamBitmap,
-    'encoding': ParamEncoding, 'borders': ParamUnit
+    'encoding': ParamEncoding, 'borders': ParamUnit,
+    'comment': ParamComment
     }