diff --git a/deps/CMakeLists.txt b/deps/CMakeLists.txt index 06ab0fa6528..74bf6eb53eb 100644 --- a/deps/CMakeLists.txt +++ b/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) diff --git a/src/slic3r/GUI/GUI_App.cpp b/src/slic3r/GUI/GUI_App.cpp index 51cf7597a22..7f3d86a9c32 100644 --- a/src/slic3r/GUI/GUI_App.cpp +++ b/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: diff --git a/src/slic3r/GUI/GUI_App.hpp b/src/slic3r/GUI/GUI_App.hpp index f0caeed0a5b..50c771879fe 100644 --- a/src/slic3r/GUI/GUI_App.hpp +++ b/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; diff --git a/src/slic3r/GUI/MainFrame.cpp b/src/slic3r/GUI/MainFrame.cpp index 62fffc42392..2b16e92d674 100644 --- a/src/slic3r/GUI/MainFrame.cpp +++ b/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); diff --git a/src/slic3r/GUI/OptionsGroup.hpp b/src/slic3r/GUI/OptionsGroup.hpp index 6128eb7ab47..09df9ea6530 100644 --- a/src/slic3r/GUI/OptionsGroup.hpp +++ b/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)); diff --git a/src/slic3r/GUI/Preferences.cpp b/src/slic3r/GUI/Preferences.cpp index 88c643add3b..ff42da61db3 100644 --- a/src/slic3r/GUI/Preferences.cpp +++ b/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(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(); diff --git a/src/slic3r/GUI/Preferences.hpp b/src/slic3r/GUI/Preferences.hpp index 738e805b257..d90f01e2b2d 100644 --- a/src/slic3r/GUI/Preferences.hpp +++ b/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();