chiark / gitweb /
Merge branch 'master' into SteamEngine
authordaid303 <daid303@gmail.com>
Mon, 22 Apr 2013 12:55:30 +0000 (14:55 +0200)
committerdaid303 <daid303@gmail.com>
Mon, 22 Apr 2013 12:55:30 +0000 (14:55 +0200)
Conflicts:
package.sh

1  2 
Cura/gui/mainWindow.py
Cura/gui/preferencesDialog.py

diff --combined Cura/gui/mainWindow.py
index 91adfa299040a53b9eefc2102a2c2bc3848e385d,1c2aefbaa4083e74b63908f9a022928a25bcb58f..bd78aa4ac0b77bed77c2cfab6694f6350d2513e0
@@@ -6,23 -6,29 +6,23 @@@ import webbrowse
  
  from Cura.gui import configBase
  from Cura.gui import expertConfig
 -from Cura.gui import preview3d
 -from Cura.gui import sliceProgressPanel
  from Cura.gui import alterationPanel
  from Cura.gui import pluginPanel
  from Cura.gui import preferencesDialog
  from Cura.gui import configWizard
  from Cura.gui import firmwareInstall
 -from Cura.gui import printWindow
  from Cura.gui import simpleMode
 -from Cura.gui import projectPlanner
 +from Cura.gui import sceneView
  from Cura.gui.tools import batchRun
 -from Cura.gui import flatSlicerWindow
  from Cura.gui.util import dropTarget
  from Cura.gui.tools import minecraftImport
 -from Cura.util import validators
  from Cura.util import profile
  from Cura.util import version
 -from Cura.util import sliceRun
  from Cura.util import meshLoader
  
  class mainWindow(wx.Frame):
        def __init__(self):
 -              super(mainWindow, self).__init__(None, title='Cura - ' + version.getVersion())
 +              super(mainWindow, self).__init__(None, title='Cura Steam Engine BETA - ' + version.getVersion())
  
                self.extruderCount = int(profile.getPreference('extruder_amount'))
  
                self.menubar = wx.MenuBar()
                self.fileMenu = wx.Menu()
                i = self.fileMenu.Append(-1, 'Load model file...\tCTRL+L')
 -              self.Bind(wx.EVT_MENU, lambda e: self._showModelLoadDialog(1), i)
 -              i = self.fileMenu.Append(-1, 'Prepare print...\tCTRL+R')
 -              self.Bind(wx.EVT_MENU, self.OnSlice, i)
 +              self.Bind(wx.EVT_MENU, lambda e: self.scene.ShowLoadModel(1), i)
                i = self.fileMenu.Append(-1, 'Print...\tCTRL+P')
 -              self.Bind(wx.EVT_MENU, self.OnPrint, i)
 +              self.Bind(wx.EVT_MENU, lambda e: self.scene.ShowPrintWindow(), i)
  
                self.fileMenu.AppendSeparator()
                i = self.fileMenu.Append(-1, 'Open Profile...')
                self.switchToNormalMenuItem = i
                self.Bind(wx.EVT_MENU, self.OnNormalSwitch, i)
                toolsMenu.AppendSeparator()
 -              i = toolsMenu.Append(-1, 'Project planner...')
 -              self.Bind(wx.EVT_MENU, self.OnProjectPlanner, i)
 -              self.normalModeOnlyItems.append(i)
 -              i = toolsMenu.Append(-1, 'Batch run...')
 -              self.Bind(wx.EVT_MENU, self.OnBatchRun, i)
 -              self.normalModeOnlyItems.append(i)
 -              #               i = toolsMenu.Append(-1, 'Open SVG (2D) slicer...')
 -              #               self.Bind(wx.EVT_MENU, self.OnSVGSlicerOpen, i)
 +              #i = toolsMenu.Append(-1, 'Batch run...')
 +              #self.Bind(wx.EVT_MENU, self.OnBatchRun, i)
 +              #self.normalModeOnlyItems.append(i)
                if minecraftImport.hasMinecraft():
                        i = toolsMenu.Append(-1, 'Minecraft import...')
                        self.Bind(wx.EVT_MENU, self.OnMinecraftImport, i)
  
                if profile.getPreference('lastFile') != '':
                        self.filelist = profile.getPreference('lastFile').split(';')
 -                      self.SetTitle('Cura - %s - %s' % (version.getVersion(), self.filelist[-1]))
                else:
                        self.filelist = []
 -              self.progressPanelList = []
  
                self.splitter = wx.SplitterWindow(self, style = wx.SP_3D | wx.SP_LIVE_UPDATE)
                self.leftPane = wx.Panel(self.splitter, style=wx.BORDER_NONE)
  
                ##Gui components##
                self.simpleSettingsPanel = simpleMode.simpleModePanel(self.leftPane)
 -              self.normalSettingsPanel = normalSettingsPanel(self.leftPane)
 +              self.normalSettingsPanel = normalSettingsPanel(self.leftPane, lambda : self.scene.sceneUpdated())
  
                self.leftSizer = wx.BoxSizer(wx.VERTICAL)
                self.leftSizer.Add(self.simpleSettingsPanel)
                self.leftPane.SetSizer(self.leftSizer)
                
                #Preview window
 -              self.preview3d = preview3d.previewPanel(self.rightPane)
 -
 -              #Also bind double clicking the 3D preview to load an STL file.
 -              #self.preview3d.glCanvas.Bind(wx.EVT_LEFT_DCLICK, lambda e: self._showModelLoadDialog(1), self.preview3d.glCanvas)
 +              self.scene = sceneView.SceneView(self.rightPane)
  
                #Main sizer, to position the preview window, buttons and tab control
                sizer = wx.BoxSizer()
                self.rightPane.SetSizer(sizer)
 -              sizer.Add(self.preview3d, 1, flag=wx.EXPAND)
 +              sizer.Add(self.scene, 1, flag=wx.EXPAND)
  
                # Main window sizer
                sizer = wx.BoxSizer(wx.VERTICAL)
                self.sizer = sizer
  
                if len(self.filelist) > 0:
 -                      self.preview3d.loadModelFiles(self.filelist)
 +                      self.scene.loadScene(self.filelist)
  
                        # Update the Model MRU
                        for idx in xrange(0, len(self.filelist)):
                        self.splitter.SetSashPosition(self.normalSashPos, True)
                        # Enabled sash
                        self.splitter.SetSashSize(4)
 +              self.scene.updateProfileToControls()
                                                                
        def OnPreferences(self, e):
                prefDialog = preferencesDialog.preferencesDialog(self)
                prefDialog.Centre()
-               prefDialog.Show(True)
+               prefDialog.Show()
  
 -      def _showOpenDialog(self, title, wildcard = meshLoader.wildcardFilter()):
 -              dlg=wx.FileDialog(self, title, os.path.split(profile.getPreference('lastFile'))[0], style=wx.FD_OPEN|wx.FD_FILE_MUST_EXIST)
 -              dlg.SetWildcard(wildcard)
 -              if dlg.ShowModal() == wx.ID_OK:
 -                      filename = dlg.GetPath()
 -                      dlg.Destroy()
 -                      if not(os.path.exists(filename)):
 -                              return False
 -                      profile.putPreference('lastFile', filename)
 -                      return filename
 -              dlg.Destroy()
 -              return False
 -
 -      def _showModelLoadDialog(self, amount):
 -              filelist = []
 -              for i in xrange(0, amount):
 -                      filelist.append(self._showOpenDialog("Open file to print"))
 -                      if filelist[-1] == False:
 -                              return
 -              self._loadModels(filelist)
 -
 -      def _loadModels(self, filelist):
 -              self.filelist = filelist
 -              self.SetTitle('Cura - %s - %s' % (version.getVersion(), filelist[-1]))
 -              profile.putPreference('lastFile', ';'.join(self.filelist))
 -              self.preview3d.loadModelFiles(self.filelist, True)
 -              self.preview3d.setViewMode("Normal")
 -              
 -              # Update the Model MRU
 -              for idx in xrange(0, len(self.filelist)):
 -                      self.addToModelMRU(self.filelist[idx])
 -
        def OnDropFiles(self, files):
 -              profile.putProfileSetting('model_matrix', '1,0,0,0,1,0,0,0,1')
                profile.setPluginConfig([])
                self.updateProfileToControls()
 -              self._loadModels(files)
 -
 -      def OnLoadModel(self, e):
 -              self._showModelLoadDialog(1)
 -
 -      def OnLoadModel2(self, e):
 -              self._showModelLoadDialog(2)
 -
 -      def OnLoadModel3(self, e):
 -              self._showModelLoadDialog(3)
 -
 -      def OnLoadModel4(self, e):
 -              self._showModelLoadDialog(4)
 -
 -      def OnSlice(self, e):
 -              if len(self.filelist) < 1:
 -                      wx.MessageBox('You need to load a file before you can prepare it.', 'Print error', wx.OK | wx.ICON_INFORMATION)
 -                      return
 -              isSimple = profile.getPreference('startMode') == 'Simple'
 -              if isSimple:
 -                      #save the current profile so we can put it back latter
 -                      oldProfile = profile.getGlobalProfileString()
 -                      self.simpleSettingsPanel.setupSlice()
 -              #Create a progress panel and add it to the window. The progress panel will start the Skein operation.
 -              spp = sliceProgressPanel.sliceProgressPanel(self, self, self.filelist)
 -              self.sizer.Add(spp, 0, flag=wx.EXPAND)
 -              self.sizer.Layout()
 -              newSize = self.GetSize()
 -              newSize.IncBy(0, spp.GetSize().GetHeight())
 -              if newSize.GetWidth() < wx.GetDisplaySize()[0]:
 -                      self.SetSize(newSize)
 -              self.progressPanelList.append(spp)
 -              if isSimple:
 -                      profile.loadGlobalProfileFromString(oldProfile)
 -
 -      def OnPrint(self, e):
 -              if len(self.filelist) < 1:
 -                      wx.MessageBox('You need to load a file and prepare it before you can print.', 'Print error', wx.OK | wx.ICON_INFORMATION)
 -                      return
 -              if not os.path.exists(sliceRun.getExportFilename(self.filelist[0])):
 -                      wx.MessageBox('You need to prepare a print before you can run the actual print.', 'Print error', wx.OK | wx.ICON_INFORMATION)
 -                      return
 -              printWindow.printFile(sliceRun.getExportFilename(self.filelist[0]))
 +              self.scene.loadScene(files)
  
        def OnModelMRU(self, e):
                fileNum = e.GetId() - self.ID_MRU_MODEL1
                self.config.Flush()
                # Load Model
                filelist = [ path ]
 -              self._loadModels(filelist)
 +              self.scene.loadScene(filelist)
  
        def addToModelMRU(self, file):
                self.modelFileHistory.AddFileToHistory(file)
                self.profileFileHistory.Save(self.config)
                self.config.Flush()
                # Load Profile  
 -              profile.loadGlobalProfile(path)
 +              profile.loadProfile(path)
                self.updateProfileToControls()
  
        def addToProfileMRU(self, file):
                self.profileFileHistory.Save(self.config)
                self.config.Flush()                     
  
 -      def removeSliceProgress(self, spp):
 -              self.progressPanelList.remove(spp)
 -              newSize = self.GetSize()
 -              newSize.IncBy(0, -spp.GetSize().GetHeight())
 -              if newSize.GetWidth() < wx.GetDisplaySize()[0]:
 -                      self.SetSize(newSize)
 -              spp.Show(False)
 -              self.sizer.Detach(spp)
 -              self.sizer.Layout()
 -
        def updateProfileToControls(self):
 -              self.preview3d.updateProfileToControls()
 +              self.scene.updateProfileToControls()
                self.normalSettingsPanel.updateProfileToControls()
                self.simpleSettingsPanel.updateProfileToControls()
  
                dlg.SetWildcard("ini files (*.ini)|*.ini")
                if dlg.ShowModal() == wx.ID_OK:
                        profileFile = dlg.GetPath()
 -                      profile.loadGlobalProfile(profileFile)
 +                      profile.loadProfile(profileFile)
                        self.updateProfileToControls()
  
                        # Update the Profile MRU
                        hasProfile = False
                        for line in f:
                                if line.startswith(';CURA_PROFILE_STRING:'):
 -                                      profile.loadGlobalProfileFromString(line[line.find(':')+1:].strip())
 +                                      profile.loadProfileFromString(line[line.find(':')+1:].strip())
                                        hasProfile = True
                        if hasProfile:
                                self.updateProfileToControls()
                dlg.SetWildcard("ini files (*.ini)|*.ini")
                if dlg.ShowModal() == wx.ID_OK:
                        profileFile = dlg.GetPath()
 -                      profile.saveGlobalProfile(profileFile)
 +                      profile.saveProfile(profileFile)
                dlg.Destroy()
  
        def OnResetProfile(self, e):
                result = dlg.ShowModal() == wx.ID_YES
                dlg.Destroy()
                if result:
 -                      profile.resetGlobalProfile()
 +                      profile.resetProfile()
                        self.updateProfileToControls()
  
        def OnBatchRun(self, e):
  
        def OnCustomFirmware(self, e):
                if profile.getPreference('machine_type') == 'ultimaker':
 -                      wx.MessageBox('Warning: Installing a custom firmware does not garantee that you machine will function correctly, and could damage your machine.', 'Firmware update', wx.OK | wx.ICON_EXCLAMATION)
 +                      wx.MessageBox('Warning: Installing a custom firmware does not guarantee that you machine will function correctly, and could damage your machine.', 'Firmware update', wx.OK | wx.ICON_EXCLAMATION)
                dlg=wx.FileDialog(self, "Open firmware to upload", os.path.split(profile.getPreference('lastFile'))[0], style=wx.FD_OPEN|wx.FD_FILE_MUST_EXIST)
                dlg.SetWildcard("HEX file (*.hex)|*.hex;*.HEX")
                if dlg.ShowModal() == wx.ID_OK:
        def OnExpertOpen(self, e):
                ecw = expertConfig.expertConfigWindow()
                ecw.Centre()
 -              ecw.Show(True)
 -
 -      def OnProjectPlanner(self, e):
 -              pp = projectPlanner.projectPlanner()
 -              pp.Centre()
 -              pp.Show(True)
 +              ecw.ShowModal()
 +              self.scene.sceneUpdated()
  
        def OnMinecraftImport(self, e):
                mi = minecraftImport.minecraftImportWindow(self)
                mi.Centre()
                mi.Show(True)
  
 -      def OnSVGSlicerOpen(self, e):
 -              svgSlicer = flatSlicerWindow.flatSlicerWindow()
 -              svgSlicer.Centre()
 -              svgSlicer.Show(True)
 -
        def OnCheckForUpdate(self, e):
                newVersion = version.checkForNewerVersion()
                if newVersion is not None:
                        wx.MessageBox('You are running the latest version of Cura!', 'Awesome!', wx.ICON_INFORMATION)
  
        def OnClose(self, e):
 -              profile.saveGlobalProfile(profile.getDefaultProfilePath())
 +              profile.saveProfile(profile.getDefaultProfilePath())
  
                # Save the window position, size & state from the preferences file
                profile.putPreference('window_maximized', self.IsMaximized())
                        profile.putPreference('window_normal_sash', self.normalSashPos)
  
                #HACK: Set the paint function of the glCanvas to nothing so it won't keep refreshing. Which keeps wxWidgets from quiting.
 -              self.preview3d.glCanvas.OnPaint = lambda e : e
 +              print "Closing down"
 +              self.scene.OnPaint = lambda e : e
 +              self.scene._slicer.cleanup()
                self.Destroy()
  
        def OnQuit(self, e):
  
  class normalSettingsPanel(configBase.configPanelBase):
        "Main user interface window"
 -      def __init__(self, parent):
 -              super(normalSettingsPanel, self).__init__(parent)
 +      def __init__(self, parent, callback = None):
 +              super(normalSettingsPanel, self).__init__(parent, callback)
  
                #Main tabs
                self.nb = wx.Notebook(self)
                self.GetSizer().Add(self.nb, 1, wx.EXPAND)
  
                (left, right, self.printPanel) = self.CreateDynamicConfigTab(self.nb, 'Basic')
 -
 -              configBase.TitleRow(left, "Quality")
 -              c = configBase.SettingRow(left, "Layer height (mm)", 'layer_height', '0.2', 'Layer height in millimeters.\n0.2 is a good value for quick prints.\n0.1 gives high quality prints.')
 -              validators.validFloat(c, 0.0001)
 -              validators.warningAbove(c, lambda : (float(profile.getProfileSetting('nozzle_size')) * 80.0 / 100.0), "Thicker layers then %.2fmm (80%% nozzle size) usually give bad results and are not recommended.")
 -              c = configBase.SettingRow(left, "Wall thickness (mm)", 'wall_thickness', '0.8', 'Thickness of the walls.\nThis is used in combination with the nozzle size to define the number\nof perimeter lines and the thickness of those perimeter lines.')
 -              validators.validFloat(c, 0.0001)
 -              validators.wallThicknessValidator(c)
 -              c = configBase.SettingRow(left, "Enable retraction", 'retraction_enable', False, 'Retract the filament when the nozzle is moving over a none-printed area. Details about the retraction can be configured in the advanced tab.')
 -
 -              configBase.TitleRow(left, "Fill")
 -              c = configBase.SettingRow(left, "Bottom/Top thickness (mm)", 'solid_layer_thickness', '0.6', 'This controls the thickness of the bottom and top layers, the amount of solid layers put down is calculated by the layer thickness and this value.\nHaving this value a multiply of the layer thickness makes sense. And keep it near your wall thickness to make an evenly strong part.')
 -              validators.validFloat(c, 0.0)
 -              c = configBase.SettingRow(left, "Fill Density (%)", 'fill_density', '20', 'This controls how densily filled the insides of your print will be. For a solid part use 100%, for an empty part use 0%. A value around 20% is usually enough')
 -              validators.validFloat(c, 0.0, 100.0)
 -
 -              configBase.TitleRow(right, "Speed && Temperature")
 -              c = configBase.SettingRow(right, "Print speed (mm/s)", 'print_speed', '50', 'Speed at which printing happens. A well adjusted Ultimaker can reach 150mm/s, but for good quality prints you want to print slower. Printing speed depends on a lot of factors. So you will be experimenting with optimal settings for this.')
 -              validators.validFloat(c, 1.0)
 -              validators.warningAbove(c, 150.0, "It is highly unlikely that your machine can achieve a printing speed above 150mm/s")
 -              validators.printSpeedValidator(c)
 -
 -              #configBase.TitleRow(right, "Temperature")
 -              c = configBase.SettingRow(right, "Printing temperature", 'print_temperature', '0', 'Temperature used for printing. Set at 0 to pre-heat yourself')
 -              validators.validFloat(c, 0.0, 340.0)
 -              validators.warningAbove(c, 260.0, "Temperatures above 260C could damage your machine, be careful!")
 -              if int(profile.getPreference('extruder_amount')) > 1:
 -                      c = configBase.SettingRow(right, "2nd nozzle temperature", 'print_temperature2', '0', 'Temperature used for printing with the 2nd nozzle. Set at 0 to use the same temperature as for nozzle 1')
 -                      validators.validFloat(c, 0.0, 340.0)
 -                      validators.warningAbove(c, 260.0, "Temperatures above 260C could damage your machine, be careful!")
 -              if int(profile.getPreference('extruder_amount')) > 2:
 -                      c = configBase.SettingRow(right, "3th nozzle temperature", 'print_temperature3', '0', 'Temperature used for printing with the 3th nozzle. Set at 0 to use the same temperature as for nozzle 1')
 -                      validators.validFloat(c, 0.0, 340.0)
 -                      validators.warningAbove(c, 260.0, "Temperatures above 260C could damage your machine, be careful!")
 -              if int(profile.getPreference('extruder_amount')) > 3:
 -                      c = configBase.SettingRow(right, "4th nozzle temperature", 'print_temperature4', '0', 'Temperature used for printing with the 4th nozzle. Set at 0 to use the same temperature as for nozzle 1')
 -                      validators.validFloat(c, 0.0, 340.0)
 -                      validators.warningAbove(c, 260.0, "Temperatures above 260C could damage your machine, be careful!")
 -              if profile.getPreference('has_heated_bed') == 'True':
 -                      c = configBase.SettingRow(right, "Bed temperature", 'print_bed_temperature', '0', 'Temperature used for the heated printer bed. Set at 0 to pre-heat yourself')
 -                      validators.validFloat(c, 0.0, 340.0)
 -
 -              configBase.TitleRow(right, "Support structure")
 -              c = configBase.SettingRow(right, "Support type", 'support', ['None', 'Exterior Only', 'Everywhere'], 'Type of support structure build.\n"Exterior only" is the most commonly used support setting.\n\nNone does not do any support.\nExterior only only creates support where the support structure will touch the build platform.\nEverywhere creates support even on the insides of the model.')
 -              c = configBase.SettingRow(right, "Add raft", 'enable_raft', False, 'A raft is a few layers of lines below the bottom of the object. It prevents warping. Full raft settings can be found in the expert settings.\nFor PLA this is usually not required. But if you print with ABS it is almost required.')
 -              if int(profile.getPreference('extruder_amount')) > 1:
 -                      c = configBase.SettingRow(right, "Support dual extrusion", 'support_dual_extrusion', False, 'Print the support material with the 2nd extruder in a dual extrusion setup. The primary extruder will be used for normal material, while the second extruder is used to print support material.')
 -
 -              configBase.TitleRow(right, "Filament")
 -              c = configBase.SettingRow(right, "Diameter (mm)", 'filament_diameter', '2.89', 'Diameter of your filament, as accurately as possible.\nIf you cannot measure this value you will have to calibrate it, a higher number means less extrusion, a smaller number generates more extrusion.')
 -              validators.validFloat(c, 1.0)
 -              validators.warningAbove(c, 3.5, "Are you sure your filament is that thick? Normal filament is around 3mm or 1.75mm.")
 -              if int(profile.getPreference('extruder_amount')) > 1:
 -                      c = configBase.SettingRow(right, "Diameter (mm)", 'filament_diameter2', '2.89', 'Diameter of your filament for the 2nd nozzle, as accurately as possible.\nIf you cannot measure this value you will have to calibrate it, a higher number means less extrusion, a smaller number generates more extrusion. Use 0 to use the same diameter as for nozzle 1.')
 -                      validators.validFloat(c, 0.0)
 -                      validators.warningAbove(c, 3.5, "Are you sure your filament is that thick? Normal filament is around 3mm or 1.75mm.")
 -              if int(profile.getPreference('extruder_amount')) > 2:
 -                      c = configBase.SettingRow(right, "Diameter (mm)", 'filament_diameter3', '2.89', 'Diameter of your filament for the 3th nozzle, as accurately as possible.\nIf you cannot measure this value you will have to calibrate it, a higher number means less extrusion, a smaller number generates more extrusion. Use 0 to use the same diameter as for nozzle 1.')
 -                      validators.validFloat(c, 0.0)
 -                      validators.warningAbove(c, 3.5, "Are you sure your filament is that thick? Normal filament is around 3mm or 1.75mm.")
 -              if int(profile.getPreference('extruder_amount')) > 3:
 -                      c = configBase.SettingRow(right, "Diameter (mm)", 'filament_diameter4', '2.89', 'Diameter of your filament for the 4th nozzle, as accurately as possible.\nIf you cannot measure this value you will have to calibrate it, a higher number means less extrusion, a smaller number generates more extrusion. Use 0 to use the same diameter as for nozzle 1.')
 -                      validators.validFloat(c, 0.0)
 -                      validators.warningAbove(c, 3.5, "Are you sure your filament is that thick? Normal filament is around 3mm or 1.75mm.")
 -              c = configBase.SettingRow(right, "Packing Density", 'filament_density', '1.00', 'Packing density of your filament. This should be 1.00 for PLA and 0.85 for ABS')
 -              validators.validFloat(c, 0.5, 1.5)
 -
 +              self._addSettingsToPanels('basic', left, right)
                self.SizeLabelWidths(left, right)
                
                (left, right, self.advancedPanel) = self.CreateDynamicConfigTab(self.nb, 'Advanced')
 -              
 -              configBase.TitleRow(left, "Machine size")
 -              c = configBase.SettingRow(left, "Nozzle size (mm)", 'nozzle_size', '0.4', 'The nozzle size is very important, this is used to calculate the line width of the infill, and used to calculate the amount of outside wall lines and thickness for the wall thickness you entered in the print settings.')
 -              validators.validFloat(c, 0.1, 10.0)
 -
 -              configBase.TitleRow(left, "Skirt")
 -              c = configBase.SettingRow(left, "Line count", 'skirt_line_count', '1', 'The skirt is a line drawn around the object at the first layer. This helps to prime your extruder, and to see if the object fits on your platform.\nSetting this to 0 will disable the skirt. Multiple skirt lines can help priming your extruder better for small objects.')
 -              validators.validInt(c, 0, 10)
 -              c = configBase.SettingRow(left, "Start distance (mm)", 'skirt_gap', '6.0', 'The distance between the skirt and the first layer.\nThis is the minimal distance, multiple skirt lines will be put outwards from this distance.')
 -              validators.validFloat(c, 0.0)
 -
 -              configBase.TitleRow(left, "Retraction")
 -              c = configBase.SettingRow(left, "Minimum travel (mm)", 'retraction_min_travel', '5.0', 'Minimum amount of travel needed for a retraction to happen at all. To make sure you do not get a lot of retractions in a small area')
 -              validators.validFloat(c, 0.0)
 -              c = configBase.SettingRow(left, "Speed (mm/s)", 'retraction_speed', '40.0', 'Speed at which the filament is retracted, a higher retraction speed works better. But a very high retraction speed can lead to filament grinding.')
 -              validators.validFloat(c, 0.1)
 -              c = configBase.SettingRow(left, "Distance (mm)", 'retraction_amount', '0.0', 'Amount of retraction, set at 0 for no retraction at all. A value of 2.0mm seems to generate good results.')
 -              validators.validFloat(c, 0.0)
 -              c = configBase.SettingRow(left, "Extra length on start (mm)", 'retraction_extra', '0.0', 'Extra extrusion amount when restarting after a retraction, to better "Prime" your extruder after retraction.')
 -              validators.validFloat(c, 0.0)
 -
 -              configBase.TitleRow(right, "Speed")
 -              c = configBase.SettingRow(right, "Travel speed (mm/s)", 'travel_speed', '150', 'Speed at which travel moves are done, a high quality build Ultimaker can reach speeds of 250mm/s. But some machines might miss steps then.')
 -              validators.validFloat(c, 1.0)
 -              validators.warningAbove(c, 300.0, "It is highly unlikely that your machine can achieve a travel speed above 300mm/s")
 -              c = configBase.SettingRow(right, "Max Z speed (mm/s)", 'max_z_speed', '1.0', 'Speed at which Z moves are done. When you Z axis is properly lubercated you can increase this for less Z blob.')
 -              validators.validFloat(c, 0.5)
 -              c = configBase.SettingRow(right, "Bottom layer speed (mm/s)", 'bottom_layer_speed', '25', 'Print speed for the bottom layer, you want to print the first layer slower so it sticks better to the printer bed.')
 -              validators.validFloat(c, 0.0)
 -
 -              configBase.TitleRow(right, "Cool")
 -              c = configBase.SettingRow(right, "Minimal layer time (sec)", 'cool_min_layer_time', '10', 'Minimum time spend in a layer, gives the layer time to cool down before the next layer is put on top. If the layer will be placed down too fast the printer will slow down to make sure it has spend atleast this amount of seconds printing this layer.')
 -              validators.validFloat(c, 0.0)
 -              c = configBase.SettingRow(right, "Enable cooling fan", 'fan_enabled', True, 'Enable the cooling fan during the print. The extra cooling from the cooling fan is essensial during faster prints.')
 -
 -              configBase.TitleRow(right, "Quality")
 -              c = configBase.SettingRow(right, "Initial layer thickness (mm)", 'bottom_thickness', '0.0', 'Layer thickness of the bottom layer. A thicker bottom layer makes sticking to the bed easier. Set to 0.0 to have the bottom layer thickness the same as the other layers.')
 -              validators.validFloat(c, 0.0)
 -              validators.warningAbove(c, lambda : (float(profile.getProfileSetting('nozzle_size')) * 3.0 / 4.0), "A bottom layer of more then %.2fmm (3/4 nozzle size) usually give bad results and is not recommended.")
 -              c = configBase.SettingRow(right, "Cut off object bottom (mm)", 'object_sink', '0.00', 'Sinks the object into the platform, this can be used for objects that do not have a flat bottom and thus create a too small first layer.')
 -              validators.validFloat(c, 0.0)
 -              configBase.settingNotify(c, lambda : self.GetParent().GetParent().GetParent().preview3d.Refresh())
 -              c = configBase.SettingRow(right, "Duplicate outlines", 'enable_skin', False, 'Skin prints the outer lines of the prints twice, each time with half the thickness. This gives the illusion of a higher print quality.')
 -
 +              self._addSettingsToPanels('advanced', left, right)
                self.SizeLabelWidths(left, right)
  
                #Plugin page
                self.pluginPanel = pluginPanel.pluginPanel(self.nb)
 -              if len(self.pluginPanel.pluginList) > 0:
 +              if len(self.pluginPanel.pluginList) > 0 and False:
                        self.nb.AddPage(self.pluginPanel, "Plugins")
                else:
                        self.pluginPanel.Show(False)
                self.UpdateSize(self.printPanel)
                self.UpdateSize(self.advancedPanel)
  
 +      def _addSettingsToPanels(self, category, left, right):
 +              count = len(profile.getSubCategoriesFor(category)) + len(profile.getSettingsForCategory(category))
 +
 +              p = left
 +              n = 0
 +              for title in profile.getSubCategoriesFor(category):
 +                      n += 1 + len(profile.getSettingsForCategory(category, title))
 +                      if n > count / 2:
 +                              p = right
 +                      configBase.TitleRow(p, title)
 +                      for s in profile.getSettingsForCategory(category, title):
 +                              if s.checkConditions():
 +                                      configBase.SettingRow(p, s.getName())
 +
        def SizeLabelWidths(self, left, right):
                leftWidth = self.getLabelColumnWidth(left)
                rightWidth = self.getLabelColumnWidth(right)
index f1e922463b868228b2888100b130f44793891528,1a8f6144b1fa409ef121b7a4581df06c0a5fe602..e2940e4c522d81375e656c83ee41ba96bdf45d41
@@@ -3,13 -3,13 +3,13 @@@ from __future__ import absolute_impor
  import wx
  
  from Cura.gui import configBase
 -from Cura.util import validators
 +from Cura.util import removableStorage
  from Cura.util import machineCom
  from Cura.util import profile
  
- class preferencesDialog(wx.Frame):
+ class preferencesDialog(wx.Dialog):
        def __init__(self, parent):
-               super(preferencesDialog, self).__init__(None, title="Preferences", style=wx.DEFAULT_DIALOG_STYLE)
+               super(preferencesDialog, self).__init__(None, title="Preferences")
                
                wx.EVT_CLOSE(self, self.OnClose)
                
                
                left, right, main = self.panel.CreateConfigPanel(self)
                configBase.TitleRow(left, 'Machine settings')
 -              c = configBase.SettingRow(left, 'Steps per E', 'steps_per_e', '0', 'Amount of steps per mm filament extrusion', type = 'preference')
 -              validators.validFloat(c, 0.1)
 -              c = configBase.SettingRow(left, 'Maximum width (mm)', 'machine_width', '205', 'Size of the machine in mm', type = 'preference')
 -              validators.validFloat(c, 10.0)
 -              c = configBase.SettingRow(left, 'Maximum depth (mm)', 'machine_depth', '205', 'Size of the machine in mm', type = 'preference')
 -              validators.validFloat(c, 10.0)
 -              c = configBase.SettingRow(left, 'Maximum height (mm)', 'machine_height', '200', 'Size of the machine in mm', type = 'preference')
 -              validators.validFloat(c, 10.0)
 -              c = configBase.SettingRow(left, 'Extruder count', 'extruder_amount', ['1', '2', '3', '4'], 'Amount of extruders in your machine.', type = 'preference')
 -              c = configBase.SettingRow(left, 'Heated bed', 'has_heated_bed', False, 'If you have an heated bed, this enabled heated bed settings', type = 'preference')
 +              configBase.SettingRow(left, 'steps_per_e')
 +              configBase.SettingRow(left, 'machine_width')
 +              configBase.SettingRow(left, 'machine_depth')
 +              configBase.SettingRow(left, 'machine_height')
 +              configBase.SettingRow(left, 'extruder_amount')
 +              configBase.SettingRow(left, 'has_heated_bed')
                
                for i in xrange(1, self.oldExtruderAmount):
                        configBase.TitleRow(left, 'Extruder %d' % (i+1))
 -                      c = configBase.SettingRow(left, 'Offset X', 'extruder_offset_x%d' % (i), '0.0', 'The offset of your secondary extruder compared to the primary.', type = 'preference')
 -                      validators.validFloat(c)
 -                      c = configBase.SettingRow(left, 'Offset Y', 'extruder_offset_y%d' % (i), '0.0', 'The offset of your secondary extruder compared to the primary.', type = 'preference')
 -                      validators.validFloat(c)
 +                      configBase.SettingRow(left, 'extruder_offset_x%d' % (i))
 +                      configBase.SettingRow(left, 'extruder_offset_y%d' % (i))
  
                configBase.TitleRow(left, 'Colours')
 -              c = configBase.SettingRow(left, 'Model colour', 'model_colour', wx.Colour(0,0,0), '', type = 'preference')
 +              configBase.SettingRow(left, 'model_colour', wx.Colour)
                for i in xrange(1, self.oldExtruderAmount):
 -                      c = configBase.SettingRow(left, 'Model colour (%d)' % (i+1), 'model_colour%d' % (i+1), wx.Colour(0,0,0), '', type = 'preference')
 +                      configBase.SettingRow(left, 'model_colour%d' % (i+1), wx.Colour)
  
                configBase.TitleRow(right, 'Filament settings')
 -              c = configBase.SettingRow(right, 'Density (kg/m3)', 'filament_density', '1300', 'Weight of the filament per m3. Around 1300 for PLA. And around 1040 for ABS. This value is used to estimate the weight if the filament used for the print.', type = 'preference')
 -              validators.validFloat(c, 500.0, 3000.0)
 -              c = configBase.SettingRow(right, 'Cost (price/kg)', 'filament_cost_kg', '0', 'Cost of your filament per kg, to estimate the cost of the final print.', type = 'preference')
 -              validators.validFloat(c, 0.0)
 -              c = configBase.SettingRow(right, 'Cost (price/m)', 'filament_cost_meter', '0', 'Cost of your filament per meter, to estimate the cost of the final print.', type = 'preference')
 -              validators.validFloat(c, 0.0)
 -              
 +              configBase.SettingRow(right, 'filament_physical_density')
 +              configBase.SettingRow(right, 'filament_cost_kg')
 +              configBase.SettingRow(right, 'filament_cost_meter')
 +
                configBase.TitleRow(right, 'Communication settings')
 -              c = configBase.SettingRow(right, 'Serial port', 'serial_port', ['AUTO'] + machineCom.serialList(), 'Serial port to use for communication with the printer', type = 'preference')
 -              c = configBase.SettingRow(right, 'Baudrate', 'serial_baud', ['AUTO'] + map(str, machineCom.baudrateList()), 'Speed of the serial port communication\nNeeds to match your firmware settings\nCommon values are 250000, 115200, 57600', type = 'preference')
 +              configBase.SettingRow(right, 'serial_port', ['AUTO'] + machineCom.serialList())
 +              configBase.SettingRow(right, 'serial_baud', ['AUTO'] + map(str, machineCom.baudrateList()))
  
 -              configBase.TitleRow(right, 'Slicer settings')
 -              #c = configBase.SettingRow(right, 'Slicer selection', 'slicer', ['Cura (Skeinforge based)', 'Slic3r'], 'Which slicer to use to slice objects. Usually the Cura engine produces the best results. But Slic3r is developing fast and is faster with slicing.', type = 'preference')
 -              c = configBase.SettingRow(right, 'Save profile on slice', 'save_profile', False, 'When slicing save the profile as [stl_file]_profile.ini next to the model.', type = 'preference')
 +              #configBase.TitleRow(right, 'Slicer settings')
 +              #configBase.SettingRow(right, 'save_profile')
  
 -              configBase.TitleRow(right, 'SD Card settings')
 -              if len(profile.getSDcardDrives()) > 1:
 -                      c = configBase.SettingRow(right, 'SD card drive', 'sdpath', profile.getSDcardDrives(), 'Location of your SD card, when using the copy to SD feature.', type = 'preference')
 -              else:
 -                      c = configBase.SettingRow(right, 'SD card path', 'sdpath', '', 'Location of your SD card, when using the copy to SD feature.', type = 'preference')
 -              c = configBase.SettingRow(right, 'Copy to SD with 8.3 names', 'sdshortnames', False, 'Save the gcode files in short filenames, so they are properly shown on the UltiController', type = 'preference')
 +              #configBase.TitleRow(right, 'SD Card settings')
 +              #configBase.SettingRow(right, 'sdpath', removableStorage.getPossibleSDcardDrives())
  
                configBase.TitleRow(right, 'Cura settings')
 -              c = configBase.SettingRow(right, 'Check for updates', 'check_for_updates', True, 'Check for newer versions of Cura on startup', type = 'preference')
 -              c = configBase.SettingRow(right, 'Send usage statistics', 'submit_slice_information', True, 'Submit anonymous usage information to improve next versions of Cura', type = 'preference')
 +              configBase.SettingRow(right, 'check_for_updates')
 +              configBase.SettingRow(right, 'submit_slice_information')
  
                self.okButton = wx.Button(right, -1, 'Ok')
                right.GetSizer().Add(self.okButton, (right.GetSizer().GetRows(), 0), flag=wx.BOTTOM, border=5)
-               self.okButton.Bind(wx.EVT_BUTTON, self.OnClose)
+               self.okButton.Bind(wx.EVT_BUTTON, lambda e: self.Close())
                
-               self.MakeModal(True)
                main.Fit()
                self.Fit()
  
        def OnClose(self, e):
                if self.oldExtruderAmount != int(profile.getPreference('extruder_amount')):
                        wx.MessageBox('After changing the amount of extruders you need to restart Cura for full effect.', 'Extruder amount warning.', wx.OK | wx.ICON_INFORMATION)
-               self.MakeModal(False)
                self.parent.updateProfileToControls()
                self.Destroy()