Merge branch 'master' of https://github.com/prusa3d/PrusaSlicer

deps/CMakeLists.txt

@@ -152,8 +152,10 @@ include(MPFR/MPFR.cmake)
 include(CGAL/CGAL.cmake)
 include(wxWidgets/wxWidgets.cmake)
 
-add_dependencies(dep_blosc ${ZLIB_PKG})
-add_dependencies(dep_openexr ${ZLIB_PKG})
+if (ZLIB_PKG)
+    add_dependencies(dep_blosc ${ZLIB_PKG})
+    add_dependencies(dep_openexr ${ZLIB_PKG})
+endif ()
 
 if (MSVC)
 

src/slic3r/GUI/GUI_App.cpp

@@ -591,11 +591,10 @@ float GUI_App::toolbar_icon_scale(const bool is_limited/* = false*/) const
     return 0.01f * int_val * icon_sc;
 }
 
-void GUI_App::recreate_GUI()
+void GUI_App::recreate_GUI(const wxString& msg_name)
 {
     mainframe->shutdown();
 
-    const auto msg_name = _(L("Changing of an application language")) + dots;
     wxProgressDialog dlg(msg_name, msg_name);
     dlg.Pulse();
     dlg.Update(10, _(L("Recreating")) + dots);
@@ -720,7 +719,7 @@ void GUI_App::import_model(wxWindow *parent, wxArrayString& input_files) const
 bool GUI_App::switch_language()
 {
     if (select_language()) {
-        recreate_GUI();
+        recreate_GUI(_L("Changing of an application language") + dots);
         return true;
     } else {
         return false;
@@ -1024,8 +1023,17 @@ void GUI_App::add_config_menu(wxMenuBar *menu)
             break;
         case ConfigMenuPreferences:
         {
-            PreferencesDialog dlg(mainframe);
-            dlg.ShowModal();
+            bool recreate_app = false;
+            {
+                // the dialog needs to be destroyed before the call to recreate_GUI()
+                // or sometimes the application crashes into wxDialogBase() destructor
+                // so we put it into an inner scope
+                PreferencesDialog dlg(mainframe);
+                dlg.ShowModal();
+                recreate_app = dlg.settings_layout_changed();
+            }
+            if (recreate_app)
+                recreate_GUI(_L("Changing of the settings layout") + dots);
             break;
         }
         case ConfigMenuLanguage:

src/slic3r/GUI/GUI_App.hpp

@@ -136,7 +136,7 @@ class GUI_App : public wxApp
     int             em_unit() const         { return m_em_unit; }
     float           toolbar_icon_scale(const bool is_limited = false) const;
 
-    void            recreate_GUI();
+    void            recreate_GUI(const wxString& message);
     void            system_info();
     void            keyboard_shortcuts();
     void            load_project(wxWindow *parent, wxString& input_file) const;

src/slic3r/GUI/MainFrame.cpp

@@ -1267,9 +1267,8 @@ void MainFrame::select_tab(size_t tab/* = size_t(-1)*/)
         }
         // Show/Activate Settings Dialog
         if (m_settings_dialog->IsShown())
-            m_settings_dialog->SetFocus();
-        else
-            m_settings_dialog->Show();
+            m_settings_dialog->Hide();
+        m_settings_dialog->Show();
     }
     else if (m_layout == slNew) {
         m_plater->Show(tab == 0);

src/slic3r/GUI/OptionsGroup.hpp

@@ -178,7 +178,7 @@ class OptionsGroup {
         if (staticbox) {
             stb = new wxStaticBox(_parent, wxID_ANY, _(title));
             if (!wxOSX) stb->SetBackgroundStyle(wxBG_STYLE_PAINT);
-            stb->SetFont(wxGetApp().bold_font());
+            stb->SetFont(wxOSX ? wxGetApp().normal_font() : wxGetApp().bold_font());
         } else
         	stb = nullptr;
         sizer = (staticbox ? new wxStaticBoxSizer(stb, wxVERTICAL) : new wxBoxSizer(wxVERTICAL));

src/slic3r/GUI/Preferences.cpp

@@ -117,13 +117,6 @@ void PreferencesDialog::build()
 	m_optgroup_general->append_single_option_line(option);
 #endif
 
-	def.label = L("Show the button for the collapse sidebar");
-	def.type = coBool;
-	def.tooltip = L("If enabled, the button for the collapse sidebar will be appeared in top right corner of the 3D Scene");
-	def.set_default_value(new ConfigOptionBool{ app_config->get("show_collapse_button") == "1" });
-	option = Option(def, "show_collapse_button");
-	m_optgroup_general->append_single_option_line(option);
-
 	m_optgroup_camera = std::make_shared<ConfigOptionsGroup>(this, _(L("Camera")));
 	m_optgroup_camera->label_width = 40;
 	m_optgroup_camera->m_on_change = [this](t_config_option_key opt_key, boost::any value) {
@@ -154,6 +147,13 @@ void PreferencesDialog::build()
 		}
 	};
 
+	def.label = L("Show the button for the collapse sidebar");
+	def.type = coBool;
+	def.tooltip = L("If enabled, the button for the collapse sidebar will be appeared in top right corner of the 3D Scene");
+	def.set_default_value(new ConfigOptionBool{ app_config->get("show_collapse_button") == "1" });
+	option = Option(def, "show_collapse_button");
+	m_optgroup_gui->append_single_option_line(option);
+
 	def.label = L("Use custom size for toolbar icons");
 	def.type = coBool;
 	def.tooltip = L("If enabled, you can change size of toolbar icons manually.");
@@ -190,11 +190,11 @@ void PreferencesDialog::accept()
 
     auto app_config = get_app_config();
 
-	bool settings_layout_changed =	m_values.find("old_settings_layout_mode") != m_values.end() ||
-		                            m_values.find("new_settings_layout_mode") != m_values.end() ||
-		                            m_values.find("dlg_settings_layout_mode") != m_values.end();
+	m_settings_layout_changed =	m_values.find("old_settings_layout_mode") != m_values.end() ||
+		                        m_values.find("new_settings_layout_mode") != m_values.end() ||
+		                        m_values.find("dlg_settings_layout_mode") != m_values.end();
 
-	if (settings_layout_changed) {
+	if (m_settings_layout_changed) {
 		// the dialog needs to be destroyed before the call to recreate_gui()
 		// or sometimes the application crashes into wxDialogBase() destructor
 		// so we put it into an inner scope
@@ -222,9 +222,8 @@ void PreferencesDialog::accept()
 	app_config->save();
 	EndModal(wxID_OK);
 
-	if (settings_layout_changed)
-		// recreate application, if settings layout was changed
-		wxGetApp().recreate_GUI();
+	if (m_settings_layout_changed)
+		;// application will be recreated after Preference dialog will be destroyed
 	else
 	    // Nothify the UI to update itself from the ini file.
         wxGetApp().update_ui_from_settings();

src/slic3r/GUI/Preferences.hpp

@@ -23,10 +23,13 @@ class PreferencesDialog : public DPIDialog
 	wxSizer*                            m_icon_size_sizer;
 	wxRadioBox*							m_layout_mode_box;
     bool                                isOSX {false};
+	bool								m_settings_layout_changed {false};
 public:
 	PreferencesDialog(wxWindow* parent);
 	~PreferencesDialog() {}
 
+	bool settings_layout_changed() { return m_settings_layout_changed; }
+
 	void	build();
 	void	accept();