dialogs/LDPowerRailDialog.py
changeset 714 131ea7f237b9
parent 583 3f3f9b25ff9f
--- a/dialogs/LDPowerRailDialog.py	Fri Jun 15 18:03:25 2012 +0200
+++ b/dialogs/LDPowerRailDialog.py	Mon Jun 25 20:03:53 2012 +0200
@@ -25,145 +25,85 @@
 
 from graphics import *
 
-
 #-------------------------------------------------------------------------------
 #                      Edit Ladder Power Rail Properties Dialog
 #-------------------------------------------------------------------------------
 
-
-[ID_LDPOWERRAILDIALOG, ID_LDPOWERRAILDIALOGSPACER, 
- ID_LDPOWERRAILDIALOGTYPE, ID_LDPOWERRAILDIALOGRADIOBUTTON1, 
- ID_LDPOWERRAILDIALOGRADIOBUTTON2, ID_LDPOWERRAILDIALOGPREVIEW,
- ID_LDPOWERRAILDIALOGSTATICTEXT1, ID_LDPOWERRAILDIALOGSTATICTEXT2, 
- ID_LDPOWERRAILDIALOGSTATICTEXT3, ID_LDPOWERRAILDIALOGPINNUMBER,
-] = [wx.NewId() for _init_ctrls in range(10)]
-
 class LDPowerRailDialog(wx.Dialog):
     
-    if wx.VERSION < (2, 6, 0):
-        def Bind(self, event, function, id = None):
-            if id is not None:
-                event(self, id, function)
-            else:
-                event(self, function)
-    
-    def _init_coll_flexGridSizer1_Items(self, parent):
-        parent.AddSizer(self.MainSizer, 0, border=20, flag=wx.GROW|wx.TOP|wx.LEFT|wx.RIGHT)
-        parent.AddSizer(self.ButtonSizer, 0, border=20, flag=wx.ALIGN_RIGHT|wx.BOTTOM|wx.LEFT|wx.RIGHT)
+    def __init__(self, parent, controller, type = LEFTRAIL, number = 1):
+        wx.Dialog.__init__(self, parent, size=wx.Size(350, 260),
+              title=_('Power Rail Properties'))
         
-    def _init_coll_flexGridSizer1_Growables(self, parent):
-        parent.AddGrowableCol(0)
-        parent.AddGrowableRow(0)
-    
-    def _init_coll_MainSizer_Items(self, parent):
-        parent.AddSizer(self.LeftGridSizer, 1, border=5, flag=wx.GROW|wx.RIGHT)
-        parent.AddSizer(self.RightGridSizer, 1, border=5, flag=wx.GROW|wx.LEFT)
-    
-    def _init_coll_LeftGridSizer_Items(self, parent):
-        parent.AddWindow(self.staticText1, 0, border=0, flag=wx.GROW)
-        parent.AddWindow(self.radioButton1, 0, border=0, flag=wx.GROW)
-        parent.AddWindow(self.radioButton2, 0, border=0, flag=wx.GROW)
-        parent.AddWindow(self.staticText2, 0, border=0, flag=wx.GROW)
-        parent.AddWindow(self.PinNumber, 0, border=0, flag=wx.GROW)
-        parent.AddWindow(self.Spacer, 0, border=0, flag=wx.GROW)
-    
-    def _init_coll_LeftGridSizer_Growables(self, parent):
-        parent.AddGrowableCol(0)
-        parent.AddGrowableRow(5)
-            
-    def _init_coll_RightGridSizer_Items(self, parent):
-        parent.AddWindow(self.staticText3, 0, border=0, flag=wx.GROW)
-        parent.AddWindow(self.Preview, 0, border=0, flag=wx.GROW)
+        main_sizer = wx.FlexGridSizer(cols=1, hgap=0, rows=2, vgap=10)
+        main_sizer.AddGrowableCol(0)
+        main_sizer.AddGrowableRow(0)
         
-    def _init_coll_RightGridSizer_Growables(self, parent):
-        parent.AddGrowableCol(0)
-        parent.AddGrowableRow(1)
-
-    def _init_sizers(self):
-        self.flexGridSizer1 = wx.FlexGridSizer(cols=1, hgap=0, rows=2, vgap=10)
-        self.MainSizer = wx.BoxSizer(wx.HORIZONTAL)
-        self.LeftGridSizer = wx.FlexGridSizer(cols=1, hgap=0, rows=6, vgap=5)
-        self.RightGridSizer = wx.FlexGridSizer(cols=1, hgap=0, rows=2, vgap=5)
-
-        self._init_coll_flexGridSizer1_Items(self.flexGridSizer1)
-        self._init_coll_flexGridSizer1_Growables(self.flexGridSizer1)
-        self._init_coll_MainSizer_Items(self.MainSizer)
-        self._init_coll_LeftGridSizer_Items(self.LeftGridSizer)
-        self._init_coll_LeftGridSizer_Growables(self.LeftGridSizer)
-        self._init_coll_RightGridSizer_Items(self.RightGridSizer)
-        self._init_coll_RightGridSizer_Growables(self.RightGridSizer)
-
-        self.SetSizer(self.flexGridSizer1)
-
-    def _init_ctrls(self, prnt, ctrler):
-        wx.Dialog.__init__(self, id=ID_LDPOWERRAILDIALOG,
-              name='PowerRailDialog', parent=prnt,
-              size=wx.Size(350, 260), style=wx.DEFAULT_DIALOG_STYLE,
-              title=_('Power Rail Properties'))
-        self.SetClientSize(wx.Size(350, 260))
-
-        self.staticText1 = wx.StaticText(id=ID_LDPOWERRAILDIALOGSTATICTEXT1,
-              label=_('Type:'), name='staticText1', parent=self,
-              pos=wx.Point(0, 0), size=wx.Size(0, 17), style=0)
-
-        self.staticText2 = wx.StaticText(id=ID_LDPOWERRAILDIALOGSTATICTEXT2,
-              label=_('Pin number:'), name='staticText2', parent=self,
-              pos=wx.Point(0, 0), size=wx.Size(0, 17), style=0)
-
-        self.staticText3 = wx.StaticText(id=ID_LDPOWERRAILDIALOGSTATICTEXT3,
-              label=_('Preview:'), name='staticText3', parent=self,
-              pos=wx.Point(0, 0), size=wx.Size(0, 17), style=0)
-
-        self.radioButton1 = wx.RadioButton(id=ID_LDPOWERRAILDIALOGRADIOBUTTON1,
-              label=_('Left PowerRail'), name='radioButton1', parent=self,
-              pos=wx.Point(0, 0), size=wx.Size(0, 24), style=wx.RB_GROUP)
-        self.Bind(wx.EVT_RADIOBUTTON, self.OnTypeChanged, id=ID_LDPOWERRAILDIALOGRADIOBUTTON1)
-        self.radioButton1.SetValue(True)
-
-        self.radioButton2 = wx.RadioButton(id=ID_LDPOWERRAILDIALOGRADIOBUTTON2,
-              label=_('Right PowerRail'), name='radioButton2', parent=self, 
-              pos=wx.Point(0, 0), size=wx.Size(0, 24), style=0)
-        self.Bind(wx.EVT_RADIOBUTTON, self.OnTypeChanged, id=ID_LDPOWERRAILDIALOGRADIOBUTTON2)
-
-        self.PinNumber = wx.SpinCtrl(id=ID_LDPOWERRAILDIALOGPINNUMBER,
-              name='PinNumber', parent=self, pos=wx.Point(0, 0),
-              size=wx.Size(0, 24), style=wx.SP_ARROW_KEYS, min=1, max=50)
-        self.Bind(wx.EVT_SPINCTRL, self.OnPinNumberChanged, id=ID_LDPOWERRAILDIALOGPINNUMBER)
-
-        self.Preview = wx.Panel(id=ID_LDPOWERRAILDIALOGPREVIEW,
-              name='Preview', parent=self, pos=wx.Point(0, 0),
-              size=wx.Size(0, 0), style=wx.TAB_TRAVERSAL|wx.SIMPLE_BORDER)
+        column_sizer = wx.BoxSizer(wx.HORIZONTAL)
+        main_sizer.AddSizer(column_sizer, border=20, 
+              flag=wx.GROW|wx.TOP|wx.LEFT|wx.RIGHT)
+        
+        left_gridsizer = wx.FlexGridSizer(cols=1, hgap=0, rows=5, vgap=5)
+        left_gridsizer.AddGrowableCol(0)
+        column_sizer.AddSizer(left_gridsizer, 1, border=5, 
+              flag=wx.GROW|wx.RIGHT)
+        
+        type_label = wx.StaticText(self, label=_('Type:'))
+        left_gridsizer.AddWindow(type_label, flag=wx.GROW)
+        
+        self.LeftPowerRail = wx.RadioButton(self,
+              label=_('Left PowerRail'), style=wx.RB_GROUP)
+        self.LeftPowerRail.SetValue(True)
+        self.Bind(wx.EVT_RADIOBUTTON, self.OnTypeChanged, self.LeftPowerRail)
+        left_gridsizer.AddWindow(self.LeftPowerRail, flag=wx.GROW)
+        
+        self.RightPowerRail = wx.RadioButton(self, label=_('Right PowerRail'))
+        self.Bind(wx.EVT_RADIOBUTTON, self.OnTypeChanged, self.RightPowerRail)
+        left_gridsizer.AddWindow(self.RightPowerRail, flag=wx.GROW)
+        
+        pin_number_label = wx.StaticText(self, label=_('Pin number:'))
+        left_gridsizer.AddWindow(pin_number_label, flag=wx.GROW)
+        
+        self.PinNumber = wx.SpinCtrl(self, min=1, max=50,
+              style=wx.SP_ARROW_KEYS)
+        self.Bind(wx.EVT_SPINCTRL, self.OnPinNumberChanged, self.PinNumber)
+        left_gridsizer.AddWindow(self.PinNumber, flag=wx.GROW)
+        
+        right_gridsizer = wx.FlexGridSizer(cols=1, hgap=0, rows=2, vgap=5)
+        right_gridsizer.AddGrowableCol(0)
+        right_gridsizer.AddGrowableRow(1)
+        column_sizer.AddSizer(right_gridsizer, 1, border=5, 
+              flag=wx.GROW|wx.LEFT)
+        
+        preview_label = wx.StaticText(self, label=_('Preview:'))
+        right_gridsizer.AddWindow(preview_label, flag=wx.GROW)
+        
+        self.Preview = wx.Panel(self,
+              style=wx.TAB_TRAVERSAL|wx.SIMPLE_BORDER)
         self.Preview.SetBackgroundColour(wx.Colour(255,255,255))
         setattr(self.Preview, "GetDrawingMode", lambda:FREEDRAWING_MODE)
         setattr(self.Preview, "GetScaling", lambda:None)
-        setattr(self.Preview, "IsOfType", ctrler.IsOfType)
-
-        self.Spacer = wx.Panel(id=ID_LDPOWERRAILDIALOGSPACER,
-              name='Spacer', parent=self, pos=wx.Point(0, 0),
-              size=wx.Size(0, 0), style=wx.TAB_TRAVERSAL)
-
-        self.ButtonSizer = self.CreateButtonSizer(wx.OK|wx.CANCEL|wx.CENTRE)
+        setattr(self.Preview, "IsOfType", controller.IsOfType)
+        self.Preview.Bind(wx.EVT_PAINT, self.OnPaint)
+        right_gridsizer.AddWindow(self.Preview, flag=wx.GROW)
         
-        if wx.VERSION >= (2, 5, 0):
-            self.Preview.Bind(wx.EVT_PAINT, self.OnPaint)
-        else:
-            wx.EVT_PAINT(self.Preview, self.OnPaint)
+        button_sizer = self.CreateButtonSizer(wx.OK|wx.CANCEL|wx.CENTRE)
+        main_sizer.AddSizer(button_sizer, border=20, 
+              flag=wx.ALIGN_RIGHT|wx.BOTTOM|wx.LEFT|wx.RIGHT)
         
-        self._init_sizers()
-
-    def __init__(self, parent, controler, type = LEFTRAIL, number = 1):
-        self._init_ctrls(parent, controler)
+        self.SetSizer(main_sizer)
+        
         self.Type = type
         if type == LEFTRAIL:
-            self.radioButton1.SetValue(True)
+            self.LeftPowerRail.SetValue(True)
         elif type == RIGHTRAIL:
-            self.radioButton2.SetValue(True)
+            self.RightPowerRail.SetValue(True)
         self.PinNumber.SetValue(number)
         
         self.PowerRailMinSize = (0, 0)
         self.PowerRail = None
         
-        self.radioButton1.SetFocus()
+        self.LeftPowerRail.SetFocus()
 
     def SetPreviewFont(self, font):
         self.Preview.SetFont(font)
@@ -180,9 +120,9 @@
         return values
 
     def OnTypeChanged(self, event):
-        if self.radioButton1.GetValue():
+        if self.LeftPowerRail.GetValue():
             self.Type = LEFTRAIL
-        elif self.radioButton2.GetValue():
+        elif self.RightPowerRail.GetValue():
             self.Type = RIGHTRAIL
         self.RefreshPreview()
         event.Skip()