summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorCarl Hetherington <cth@carlh.net>2020-11-21 23:00:55 +0100
committerCarl Hetherington <cth@carlh.net>2020-11-21 23:02:49 +0100
commit44da1840ca17d0945cb1886bb17218990850894c (patch)
treeb6fab2f9fa7202aef68f948f2028a36ca476d5ea /src
parentce058471aa9995adac968cecc205f5a4319f449e (diff)
Remove the "simple" UI (#1868).
It just seemed to cause more problems than it solved; mainly people enabling it by mistake and then being told to click buttons that they could not see.
Diffstat (limited to 'src')
-rw-r--r--src/lib/config.cc15
-rw-r--r--src/lib/config.h17
-rw-r--r--src/tools/dcpomatic.cc8
-rw-r--r--src/wx/audio_panel.cc43
-rw-r--r--src/wx/content_sub_panel.cc15
-rw-r--r--src/wx/content_sub_panel.h5
-rw-r--r--src/wx/dcp_panel.cc138
-rw-r--r--src/wx/full_config_dialog.cc28
-rw-r--r--src/wx/initial_setup_dialog.cc84
-rw-r--r--src/wx/initial_setup_dialog.h36
-rw-r--r--src/wx/text_panel.cc17
-rw-r--r--src/wx/timing_panel.cc28
-rw-r--r--src/wx/video_panel.cc73
-rw-r--r--src/wx/wscript1
14 files changed, 94 insertions, 414 deletions
diff --git a/src/lib/config.cc b/src/lib/config.cc
index 80c0891a0..8b849dcc1 100644
--- a/src/lib/config.cc
+++ b/src/lib/config.cc
@@ -165,7 +165,6 @@ Config::set_defaults ()
_christie_password = optional<string>();
_gdc_username = optional<string>();
_gdc_password = optional<string>();
- _interface_complexity = INTERFACE_SIMPLE;
_player_mode = PLAYER_MODE_WINDOW;
_image_display = 0;
_video_view_type = VIDEO_VIEW_SIMPLE;
@@ -571,10 +570,6 @@ try
_gdc_username = f.optional_string_child("GDCUsername");
_gdc_password = f.optional_string_child("GDCPassword");
- optional<string> ic = f.optional_string_child("InterfaceComplexity");
- if (ic && *ic == "full") {
- _interface_complexity = INTERFACE_FULL;
- }
optional<string> pm = f.optional_string_child("PlayerMode");
if (pm && *pm == "window") {
_player_mode = PLAYER_MODE_WINDOW;
@@ -1008,16 +1003,6 @@ Config::write_config () const
root->add_child("GDCPassword")->add_child_text(*_gdc_password);
}
- /* [XML] InterfaceComplexity <code>simple</code> for the reduced interface or <code>full</code> for the full interface. */
- switch (_interface_complexity) {
- case INTERFACE_SIMPLE:
- root->add_child("InterfaceComplexity")->add_child_text("simple");
- break;
- case INTERFACE_FULL:
- root->add_child("InterfaceComplexity")->add_child_text("full");
- break;
- }
-
/* [XML] PlayerMode <code>window</code> for a single window, <code>full</code> for full-screen and <code>dual</code> for full screen playback
with controls on another monitor.
*/
diff --git a/src/lib/config.h b/src/lib/config.h
index 74a44bc2f..11ca36b85 100644
--- a/src/lib/config.h
+++ b/src/lib/config.h
@@ -80,7 +80,6 @@ public:
DKDM_RECIPIENTS,
SOUND,
SOUND_OUTPUT,
- INTERFACE_COMPLEXITY,
PLAYER_CONTENT_DIRECTORY,
PLAYER_PLAYLIST_DIRECTORY,
PLAYER_DEBUG_LOG,
@@ -401,8 +400,6 @@ public:
NAG_ENCRYPTED_METADATA,
NAG_ALTER_DECRYPTION_CHAIN,
NAG_BAD_SIGNER_CHAIN,
- /* Not really a nag but it's the same idea */
- NAG_INITIAL_SETUP,
NAG_IMPORT_DECRYPTION_CHAIN,
NAG_DELETE_DKDM,
NAG_32_ON_64,
@@ -494,15 +491,6 @@ public:
return _gdc_password;
}
- enum Interface {
- INTERFACE_SIMPLE,
- INTERFACE_FULL
- };
-
- Interface interface_complexity () const {
- return _interface_complexity;
- }
-
enum PlayerMode {
PLAYER_MODE_WINDOW, ///< one window containing image and controls
PLAYER_MODE_FULL, ///< just the image filling the screen
@@ -1014,10 +1002,6 @@ public:
maybe_set (_gdc_password, boost::optional<std::string>());
}
- void set_interface_complexity (Interface i) {
- maybe_set (_interface_complexity, i, INTERFACE_COMPLEXITY);
- }
-
void set_player_mode (PlayerMode m) {
maybe_set (_player_mode, m);
}
@@ -1344,7 +1328,6 @@ private:
boost::optional<std::string> _christie_password;
boost::optional<std::string> _gdc_username;
boost::optional<std::string> _gdc_password;
- Interface _interface_complexity;
PlayerMode _player_mode;
int _image_display;
VideoViewType _video_view_type;
diff --git a/src/tools/dcpomatic.cc b/src/tools/dcpomatic.cc
index a56386359..3ddf2256e 100644
--- a/src/tools/dcpomatic.cc
+++ b/src/tools/dcpomatic.cc
@@ -50,7 +50,6 @@
#include "wx/paste_dialog.h"
#include "wx/focus_manager.h"
#include "wx/html_dialog.h"
-#include "wx/initial_setup_dialog.h"
#include "wx/send_i18n_dialog.h"
#include "wx/i18n_hook.h"
#include "lib/film.h"
@@ -1620,13 +1619,6 @@ private:
_frame->Maximize ();
close_splash ();
- if (!Config::instance()->nagged(Config::NAG_INITIAL_SETUP)) {
- InitialSetupDialog* d = new InitialSetupDialog ();
- d->ShowModal ();
- d->Destroy ();
- Config::instance()->set_nagged(Config::NAG_INITIAL_SETUP, true);
- }
-
if (running_32_on_64 ()) {
NagDialog::maybe_nag (
_frame, Config::NAG_32_ON_64,
diff --git a/src/wx/audio_panel.cc b/src/wx/audio_panel.cc
index 76dcbad67..2809ce44a 100644
--- a/src/wx/audio_panel.cc
+++ b/src/wx/audio_panel.cc
@@ -121,20 +121,13 @@ AudioPanel::AudioPanel (ContentPanel* p)
void
AudioPanel::add_to_grid ()
{
- bool const full = Config::instance()->interface_complexity() == Config::INTERFACE_FULL;
-
int r = 0;
- _reference->Show (full);
- _reference_note->Show (full);
-
- if (full) {
- wxBoxSizer* reference_sizer = new wxBoxSizer (wxVERTICAL);
- reference_sizer->Add (_reference, 0);
- reference_sizer->Add (_reference_note, 0);
- _grid->Add (reference_sizer, wxGBPosition(r, 0), wxGBSpan(1, 4));
- ++r;
- }
+ wxBoxSizer* reference_sizer = new wxBoxSizer (wxVERTICAL);
+ reference_sizer->Add (_reference, 0);
+ reference_sizer->Add (_reference_note, 0);
+ _grid->Add (reference_sizer, wxGBPosition(r, 0), wxGBSpan(1, 4));
+ ++r;
_grid->Add (_show, wxGBPosition (r, 0), wxGBSpan (1, 2));
_grid->Add (_peak, wxGBPosition (r, 2), wxGBSpan (1, 2), wxALIGN_CENTER_VERTICAL);
@@ -148,25 +141,15 @@ AudioPanel::add_to_grid ()
_grid->Add (s, wxGBPosition(r, 1));
}
- _gain_calculate_button->Show (full);
-
- if (full) {
- _grid->Add (_gain_calculate_button, wxGBPosition(r, 2), wxDefaultSpan, wxALIGN_CENTER_VERTICAL);
- ++r;
- }
-
- _delay_label->Show (full);
- _delay->show (full);
- _delay_ms_label->Show (full);
+ _grid->Add (_gain_calculate_button, wxGBPosition(r, 2), wxDefaultSpan, wxALIGN_CENTER_VERTICAL);
+ ++r;
- if (full) {
- add_label_to_sizer (_grid, _delay_label, true, wxGBPosition(r, 0));
- wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
- s->Add (_delay->wrapped(), 1, wxALIGN_CENTER_VERTICAL | wxTOP | wxBOTTOM | wxRIGHT, 6);
- s->Add (_delay_ms_label, 0, wxALIGN_CENTER_VERTICAL);
- _grid->Add (s, wxGBPosition(r, 1));
- ++r;
- }
+ add_label_to_sizer (_grid, _delay_label, true, wxGBPosition(r, 0));
+ wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
+ s->Add (_delay->wrapped(), 1, wxALIGN_CENTER_VERTICAL | wxTOP | wxBOTTOM | wxRIGHT, 6);
+ s->Add (_delay_ms_label, 0, wxALIGN_CENTER_VERTICAL);
+ _grid->Add (s, wxGBPosition(r, 1));
+ ++r;
}
AudioPanel::~AudioPanel ()
diff --git a/src/wx/content_sub_panel.cc b/src/wx/content_sub_panel.cc
index 8cf04b93c..1a92c0dcd 100644
--- a/src/wx/content_sub_panel.cc
+++ b/src/wx/content_sub_panel.cc
@@ -45,19 +45,6 @@ ContentSubPanel::ContentSubPanel (ContentPanel* p, wxString name)
_grid = new wxGridBagSizer (DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
_sizer->Add (_grid, 0, wxALL, 8);
-
- _config_connection = Config::instance()->Changed.connect (boost::bind (&ContentSubPanel::config_changed, this, _1));
-}
-
-void
-ContentSubPanel::config_changed (Config::Property p)
-{
- if (p == Config::INTERFACE_COMPLEXITY) {
- _grid->Clear ();
- add_to_grid ();
- _sizer->Layout ();
- _grid->Layout ();
- }
}
void
@@ -73,7 +60,7 @@ ContentSubPanel::setup_refer_button (wxCheckBox* button, wxStaticText* note, sha
note->Wrap (400);
- if (cannot.IsEmpty() || Config::instance()->interface_complexity() == Config::INTERFACE_SIMPLE) {
+ if (cannot.IsEmpty()) {
note->Hide ();
} else {
note->Show ();
diff --git a/src/wx/content_sub_panel.h b/src/wx/content_sub_panel.h
index 128a3862d..cb59f38dd 100644
--- a/src/wx/content_sub_panel.h
+++ b/src/wx/content_sub_panel.h
@@ -22,7 +22,6 @@
#define DCPOMATIC_CONTENT_SUB_PANEL_H
#include "lib/film.h"
-#include "lib/config.h"
#include "lib/warnings.h"
DCPOMATIC_DISABLE_WARNINGS
#include <wx/wx.h>
@@ -58,10 +57,6 @@ protected:
wxSizer* _sizer;
wxGridBagSizer* _grid;
wxString _name;
-
-private:
- void config_changed (Config::Property);
- boost::signals2::scoped_connection _config_connection;
};
#endif
diff --git a/src/wx/dcp_panel.cc b/src/wx/dcp_panel.cc
index 4e023db32..9322e68cf 100644
--- a/src/wx/dcp_panel.cc
+++ b/src/wx/dcp_panel.cc
@@ -159,8 +159,6 @@ DCPPanel::DCPPanel (wxNotebook* n, shared_ptr<Film> film, weak_ptr<FilmViewer> v
void
DCPPanel::add_to_grid ()
{
- Config::Interface interface = Config::instance()->interface_complexity ();
-
int r = 0;
add_label_to_sizer (_grid, _name_label, true, wxGBPosition (r, 0));
@@ -172,22 +170,14 @@ DCPPanel::add_to_grid ()
flags |= wxALIGN_RIGHT;
#endif
- bool const full = interface == Config::INTERFACE_FULL;
-
- _use_isdcf_name->Show (full);
- _edit_isdcf_button->Show (full);
- _copy_isdcf_name_button->Show (full);
-
- if (full) {
- _grid->Add (_use_isdcf_name, wxGBPosition (r, 0), wxDefaultSpan, flags);
- {
- wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
- s->Add (_edit_isdcf_button, 1, wxEXPAND | wxRIGHT, DCPOMATIC_SIZER_X_GAP);
- s->Add (_copy_isdcf_name_button, 1, wxEXPAND | wxLEFT, DCPOMATIC_SIZER_X_GAP);
- _grid->Add (s, wxGBPosition (r, 1), wxDefaultSpan, wxEXPAND);
- }
- ++r;
+ _grid->Add (_use_isdcf_name, wxGBPosition (r, 0), wxDefaultSpan, flags);
+ {
+ wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
+ s->Add (_edit_isdcf_button, 1, wxEXPAND | wxRIGHT, DCPOMATIC_SIZER_X_GAP);
+ s->Add (_copy_isdcf_name_button, 1, wxEXPAND | wxLEFT, DCPOMATIC_SIZER_X_GAP);
+ _grid->Add (s, wxGBPosition (r, 1), wxDefaultSpan, wxEXPAND);
}
+ ++r;
_grid->Add (_dcp_name, wxGBPosition(r, 0), wxGBSpan (1, 2), wxALIGN_CENTER_VERTICAL | wxEXPAND);
++r;
@@ -199,43 +189,28 @@ DCPPanel::add_to_grid ()
_grid->Add (_encrypted, wxGBPosition (r, 0), wxGBSpan (1, 2));
++r;
+ add_label_to_sizer (_grid, _reels_label, true, wxGBPosition (r, 0));
+ _grid->Add (_reel_type, wxGBPosition (r, 1), wxDefaultSpan, wxALIGN_CENTER_VERTICAL);
+ ++r;
- _reels_label->Show (full);
- _reel_type->Show (full);
- _reel_length_label->Show (full);
- _reel_length->Show (full);
- _reel_length_gb_label->Show (full);
- _standard_label->Show (full);
- _standard->Show (full);
- _markers->Show (full);
- _metadata->Show (full);
- _reencode_j2k->Show (full);
- _encrypted->Show (full);
-
- if (full) {
- add_label_to_sizer (_grid, _reels_label, true, wxGBPosition (r, 0));
- _grid->Add (_reel_type, wxGBPosition (r, 1), wxDefaultSpan, wxALIGN_CENTER_VERTICAL);
- ++r;
-
- add_label_to_sizer (_grid, _reel_length_label, true, wxGBPosition (r, 0));
- {
- wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
- s->Add (_reel_length);
- add_label_to_sizer (s, _reel_length_gb_label, false);
- _grid->Add (s, wxGBPosition (r, 1));
- }
- ++r;
+ add_label_to_sizer (_grid, _reel_length_label, true, wxGBPosition (r, 0));
+ {
+ wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
+ s->Add (_reel_length);
+ add_label_to_sizer (s, _reel_length_gb_label, false);
+ _grid->Add (s, wxGBPosition (r, 1));
+ }
+ ++r;
- add_label_to_sizer (_grid, _standard_label, true, wxGBPosition (r, 0));
- _grid->Add (_standard, wxGBPosition (r, 1), wxDefaultSpan, wxALIGN_CENTER_VERTICAL);
- ++r;
+ add_label_to_sizer (_grid, _standard_label, true, wxGBPosition (r, 0));
+ _grid->Add (_standard, wxGBPosition (r, 1), wxDefaultSpan, wxALIGN_CENTER_VERTICAL);
+ ++r;
- wxBoxSizer* extra = new wxBoxSizer (wxHORIZONTAL);
- extra->Add (_markers, 1, wxRIGHT, DCPOMATIC_SIZER_X_GAP);
- extra->Add (_metadata, 1, wxRIGHT, DCPOMATIC_SIZER_X_GAP);
- _grid->Add (extra, wxGBPosition(r, 0), wxGBSpan(1, 2));
- ++r;
- }
+ wxBoxSizer* extra = new wxBoxSizer (wxHORIZONTAL);
+ extra->Add (_markers, 1, wxRIGHT, DCPOMATIC_SIZER_X_GAP);
+ extra->Add (_metadata, 1, wxRIGHT, DCPOMATIC_SIZER_X_GAP);
+ _grid->Add (extra, wxGBPosition(r, 0), wxGBSpan(1, 2));
+ ++r;
}
void
@@ -708,20 +683,7 @@ DCPPanel::config_changed (Config::Property p)
_j2k_bandwidth->SetRange (1, Config::instance()->maximum_j2k_bandwidth() / 1000000);
setup_frame_rate_widget ();
- if (p == Config::INTERFACE_COMPLEXITY) {
- _grid->Clear ();
- add_to_grid ();
- _sizer->Layout ();
- _grid->Layout ();
-
- _video_grid->Clear ();
- add_video_panel_to_grid ();
- _video_grid->Layout ();
-
- _audio_grid->Clear ();
- add_audio_panel_to_grid ();
- _audio_grid->Layout ();
- } else if (p == Config::SHOW_EXPERIMENTAL_AUDIO_PROCESSORS) {
+ if (p == Config::SHOW_EXPERIMENTAL_AUDIO_PROCESSORS) {
_audio_processor->Clear ();
add_audio_processors ();
if (_film) {
@@ -805,8 +767,6 @@ DCPPanel::make_video_panel ()
void
DCPPanel::add_video_panel_to_grid ()
{
- bool const full = Config::instance()->interface_complexity() == Config::INTERFACE_FULL;
-
int r = 0;
add_label_to_sizer (_video_grid, _container_label, true, wxGBPosition (r, 0));
@@ -835,19 +795,13 @@ DCPPanel::add_video_panel_to_grid ()
_video_grid->Add (_three_d, wxGBPosition (r, 0), wxGBSpan (1, 2));
++r;
- _j2k_bandwidth_label->Show (full);
- _j2k_bandwidth->Show (full);
- _mbits_label->Show (full);
-
- if (full) {
- add_label_to_sizer (_video_grid, _j2k_bandwidth_label, true, wxGBPosition (r, 0));
- wxSizer* s = new wxBoxSizer (wxHORIZONTAL);
- s->Add (_j2k_bandwidth, 1);
- add_label_to_sizer (s, _mbits_label, false);
- _video_grid->Add (s, wxGBPosition (r, 1));
- ++r;
- _video_grid->Add (_reencode_j2k, wxGBPosition(r, 0), wxGBSpan(1, 2));
- }
+ add_label_to_sizer (_video_grid, _j2k_bandwidth_label, true, wxGBPosition (r, 0));
+ wxSizer* s = new wxBoxSizer (wxHORIZONTAL);
+ s->Add (_j2k_bandwidth, 1);
+ add_label_to_sizer (s, _mbits_label, false);
+ _video_grid->Add (s, wxGBPosition (r, 1));
+ ++r;
+ _video_grid->Add (_reencode_j2k, wxGBPosition(r, 0), wxGBSpan(1, 2));
}
int
@@ -896,27 +850,15 @@ DCPPanel::make_audio_panel ()
void
DCPPanel::add_audio_panel_to_grid ()
{
- bool const full = Config::instance()->interface_complexity() == Config::INTERFACE_FULL;
-
int r = 0;
- _channels_label->Show (full);
- _audio_channels->Show (full);
-
- if (full) {
- add_label_to_sizer (_audio_grid, _channels_label, true, wxGBPosition (r, 0));
- _audio_grid->Add (_audio_channels, wxGBPosition (r, 1));
- ++r;
- }
-
- _processor_label->Show (full);
- _audio_processor->Show (full);
+ add_label_to_sizer (_audio_grid, _channels_label, true, wxGBPosition (r, 0));
+ _audio_grid->Add (_audio_channels, wxGBPosition (r, 1));
+ ++r;
- if (full) {
- add_label_to_sizer (_audio_grid, _processor_label, true, wxGBPosition (r, 0));
- _audio_grid->Add (_audio_processor, wxGBPosition (r, 1));
- ++r;
- }
+ add_label_to_sizer (_audio_grid, _processor_label, true, wxGBPosition (r, 0));
+ _audio_grid->Add (_audio_processor, wxGBPosition (r, 1));
+ ++r;
_audio_grid->Add (_show_audio, wxGBPosition (r, 0), wxGBSpan (1, 2));
++r;
diff --git a/src/wx/full_config_dialog.cc b/src/wx/full_config_dialog.cc
index 2d4ac3fdf..c41236096 100644
--- a/src/wx/full_config_dialog.cc
+++ b/src/wx/full_config_dialog.cc
@@ -92,11 +92,6 @@ private:
int r = 0;
add_language_controls (table, r);
- add_label_to_sizer (table, _panel, _("Interface complexity"), true, wxGBPosition(r, 0));
- _interface_complexity = new wxChoice (_panel, wxID_ANY);
- table->Add (_interface_complexity, wxGBPosition (r, 1));
- ++r;
-
add_label_to_sizer (table, _panel, _("Number of threads DCP-o-matic should use"), true, wxGBPosition (r, 0));
_master_encoding_threads = new wxSpinCtrl (_panel);
table->Add (_master_encoding_threads, wxGBPosition (r, 1));
@@ -134,10 +129,6 @@ private:
_config_file->Bind (wxEVT_FILEPICKER_CHANGED, boost::bind (&FullGeneralPage::config_file_changed, this));
_cinemas_file->Bind (wxEVT_FILEPICKER_CHANGED, boost::bind (&FullGeneralPage::cinemas_file_changed, this));
- _interface_complexity->Append (_("Simple"));
- _interface_complexity->Append (_("Full"));
- _interface_complexity->Bind (wxEVT_CHOICE, boost::bind(&FullGeneralPage::interface_complexity_changed, this));
-
_master_encoding_threads->SetRange (1, 128);
_master_encoding_threads->Bind (wxEVT_SPINCTRL, boost::bind (&FullGeneralPage::master_encoding_threads_changed, this));
_server_encoding_threads->SetRange (1, 128);
@@ -154,14 +145,6 @@ private:
{
Config* config = Config::instance ();
- switch (config->interface_complexity()) {
- case Config::INTERFACE_SIMPLE:
- checked_set (_interface_complexity, 0);
- break;
- case Config::INTERFACE_FULL:
- checked_set (_interface_complexity, 1);
- break;
- }
checked_set (_master_encoding_threads, config->master_encoding_threads ());
checked_set (_server_encoding_threads, config->server_encoding_threads ());
#ifdef DCPOMATIC_HAVE_EBUR128_PATCHED_FFMPEG
@@ -187,16 +170,6 @@ private:
d->Destroy ();
}
- void interface_complexity_changed ()
- {
- if (_interface_complexity->GetSelection() == 0) {
- Config::instance()->set_interface_complexity (Config::INTERFACE_SIMPLE);
- } else {
- Config::instance()->set_interface_complexity (Config::INTERFACE_FULL);
- }
- }
-
-
#ifdef DCPOMATIC_HAVE_EBUR128_PATCHED_FFMPEG
void analyse_ebur128_changed ()
{
@@ -250,7 +223,6 @@ private:
Config::instance()->set_cinemas_file (wx_to_std (_cinemas_file->GetPath ()));
}
- wxChoice* _interface_complexity;
wxSpinCtrl* _master_encoding_threads;
wxSpinCtrl* _server_encoding_threads;
FilePickerCtrl* _config_file;
diff --git a/src/wx/initial_setup_dialog.cc b/src/wx/initial_setup_dialog.cc
deleted file mode 100644
index 65b005207..000000000
--- a/src/wx/initial_setup_dialog.cc
+++ /dev/null
@@ -1,84 +0,0 @@
-/*
- Copyright (C) 2018 Carl Hetherington <cth@carlh.net>
-
- This file is part of DCP-o-matic.
-
- DCP-o-matic is free software; you can redistribute it and/or modify
- it under the terms of the GNU General Public License as published by
- the Free Software Foundation; either version 2 of the License, or
- (at your option) any later version.
-
- DCP-o-matic is distributed in the hope that it will be useful,
- but WITHOUT ANY WARRANTY; without even the implied warranty of
- MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- GNU General Public License for more details.
-
- You should have received a copy of the GNU General Public License
- along with DCP-o-matic. If not, see <http://www.gnu.org/licenses/>.
-
-*/
-
-#include "initial_setup_dialog.h"
-#include "static_text.h"
-#include "lib/config.h"
-#include <boost/bind.hpp>
-
-InitialSetupDialog::InitialSetupDialog ()
- : wxDialog (0, wxID_ANY, _("DCP-o-matic setup"))
-{
- wxBoxSizer* sizer = new wxBoxSizer (wxVERTICAL);
- wxStaticText* text1 = new StaticText (this, wxEmptyString, wxDefaultPosition, wxSize(600, -1));
- sizer->Add (text1, 1, wxEXPAND | wxALL, 12);
-
- text1->SetLabelMarkup (
- _(
- "<span weight=\"bold\" size=\"larger\">Welcome to DCP-o-matic!</span>\n\n"
- "DCP-o-matic can work in two modes: '<i>simple</i>' or '<i>full</i>'.\n\n"
- "<i>Simple mode</i> is ideal for producing straightforward DCPs without too many confusing "
- "options.\n\n"
- "<i>Full mode</i> gives you the most control over the DCPs you make.\n\n"
- "Please choose which mode you would like to start DCP-o-matic in:"
- )
- );
-
- wxBoxSizer* mode_sizer = new wxBoxSizer (wxVERTICAL);
-
- _simple = new wxRadioButton (this, wxID_ANY, _("Simple mode"), wxDefaultPosition, wxDefaultSize, wxRB_GROUP);
- mode_sizer->Add (_simple, 0, wxTOP, 12);
- _full = new wxRadioButton (this, wxID_ANY, _("Full mode"));
- mode_sizer->Add (_full, 0, wxTOP, 8);
-
- sizer->Add (mode_sizer, 0, wxLEFT, 24);
-
- if (Config::instance()->interface_complexity() == Config::INTERFACE_SIMPLE) {
- _simple->SetValue (true);
- } else {
- _full->SetValue (true);
- }
-
- wxStaticText* text2 = new StaticText (this, wxEmptyString, wxDefaultPosition, wxSize(400, -1));
- sizer->Add (text2, 0, wxEXPAND | wxALL, 12);
-
- text2->SetLabel (_("\nYou can change the mode at any time from the General page of Preferences."));
-
- _simple->Bind (wxEVT_RADIOBUTTON, boost::bind(&InitialSetupDialog::interface_complexity_changed, this));
- _full->Bind (wxEVT_RADIOBUTTON, boost::bind(&InitialSetupDialog::interface_complexity_changed, this));
-
- wxSizer* buttons = CreateSeparatedButtonSizer (wxOK);
- if (buttons) {
- sizer->Add(buttons, wxSizerFlags().Expand().DoubleBorder());
- }
-
- sizer->Layout ();
- SetSizerAndFit (sizer);
-}
-
-void
-InitialSetupDialog::interface_complexity_changed ()
-{
- if (_simple->GetValue()) {
- Config::instance()->set_interface_complexity (Config::INTERFACE_SIMPLE);
- } else {
- Config::instance()->set_interface_complexity (Config::INTERFACE_FULL);
- }
-}
diff --git a/src/wx/initial_setup_dialog.h b/src/wx/initial_setup_dialog.h
deleted file mode 100644
index 2198fb426..000000000
--- a/src/wx/initial_setup_dialog.h
+++ /dev/null
@@ -1,36 +0,0 @@
-/*
- Copyright (C) 2018 Carl Hetherington <cth@carlh.net>
-
- This file is part of DCP-o-matic.
-
- DCP-o-matic is free software; you can redistribute it and/or modify
- it under the terms of the GNU General Public License as published by
- the Free Software Foundation; either version 2 of the License, or
- (at your option) any later version.
-
- DCP-o-matic is distributed in the hope that it will be useful,
- but WITHOUT ANY WARRANTY; without even the implied warranty of
- MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- GNU General Public License for more details.
-
- You should have received a copy of the GNU General Public License
- along with DCP-o-matic. If not, see <http://www.gnu.org/licenses/>.
-
-*/
-
-#include "lib/warnings.h"
-DCPOMATIC_DISABLE_WARNINGS
-#include <wx/wx.h>
-DCPOMATIC_ENABLE_WARNINGS
-
-class InitialSetupDialog : public wxDialog
-{
-public:
- InitialSetupDialog ();
-
-private:
- void interface_complexity_changed ();
-
- wxRadioButton* _simple;
- wxRadioButton* _full;
-};
diff --git a/src/wx/text_panel.cc b/src/wx/text_panel.cc
index f29aea6ff..30c2010d0 100644
--- a/src/wx/text_panel.cc
+++ b/src/wx/text_panel.cc
@@ -190,20 +190,13 @@ TextPanel::setup_visibility ()
void
TextPanel::add_to_grid ()
{
- Config::Interface const interface = Config::instance()->interface_complexity();
-
int r = 0;
- _reference->Show (interface == Config::INTERFACE_FULL);
- _reference_note->Show (interface == Config::INTERFACE_FULL);
-
- if (interface == Config::INTERFACE_FULL) {
- wxBoxSizer* reference_sizer = new wxBoxSizer (wxVERTICAL);
- reference_sizer->Add (_reference, 0);
- reference_sizer->Add (_reference_note, 0);
- _grid->Add (reference_sizer, wxGBPosition(r, 0), wxGBSpan(1, 4));
- ++r;
- }
+ wxBoxSizer* reference_sizer = new wxBoxSizer (wxVERTICAL);
+ reference_sizer->Add (_reference, 0);
+ reference_sizer->Add (_reference_note, 0);
+ _grid->Add (reference_sizer, wxGBPosition(r, 0), wxGBSpan(1, 4));
+ ++r;
wxBoxSizer* use = new wxBoxSizer (wxHORIZONTAL);
use->Add (_use, 0, wxEXPAND | wxRIGHT, DCPOMATIC_SIZER_GAP);
diff --git a/src/wx/timing_panel.cc b/src/wx/timing_panel.cc
index c643d2080..195887a29 100644
--- a/src/wx/timing_panel.cc
+++ b/src/wx/timing_panel.cc
@@ -129,8 +129,6 @@ TimingPanel::TimingPanel (ContentPanel* p, weak_ptr<FilmViewer> viewer)
void
TimingPanel::add_to_grid ()
{
- bool const full = Config::instance()->interface_complexity() == Config::INTERFACE_FULL;
-
int r = 0;
wxSizer* labels = new wxBoxSizer (wxHORIZONTAL);
@@ -148,20 +146,12 @@ TimingPanel::add_to_grid ()
_grid->Add (_position, wxGBPosition(r, 1));
++r;
- _move_to_start_of_reel->Show (full);
- _full_length_label->Show (full);
- _full_length->Show (full);
- _play_length_label->Show (full);
- _play_length->Show (full);
-
- if (full) {
- _grid->Add (_move_to_start_of_reel, wxGBPosition(r, 1));
- ++r;
+ _grid->Add (_move_to_start_of_reel, wxGBPosition(r, 1));
+ ++r;
- add_label_to_sizer (_grid, _full_length_label, true, wxGBPosition(r, 0));
- _grid->Add (_full_length, wxGBPosition(r, 1));
- ++r;
- }
+ add_label_to_sizer (_grid, _full_length_label, true, wxGBPosition(r, 0));
+ _grid->Add (_full_length, wxGBPosition(r, 1));
+ ++r;
add_label_to_sizer (_grid, _trim_start_label, true, wxGBPosition(r, 0));
_grid->Add (_trim_start, wxGBPosition(r, 1));
@@ -177,11 +167,9 @@ TimingPanel::add_to_grid ()
_grid->Add (_trim_end_to_playhead, wxGBPosition(r, 1));
++r;
- if (full) {
- add_label_to_sizer (_grid, _play_length_label, true, wxGBPosition(r, 0));
- _grid->Add (_play_length, wxGBPosition(r, 1));
- ++r;
- }
+ add_label_to_sizer (_grid, _play_length_label, true, wxGBPosition(r, 0));
+ _grid->Add (_play_length, wxGBPosition(r, 1));
+ ++r;
/* Completely speculative fix for #891 */
_grid->Layout ();
diff --git a/src/wx/video_panel.cc b/src/wx/video_panel.cc
index e7d02fe30..e58c8e4f5 100644
--- a/src/wx/video_panel.cc
+++ b/src/wx/video_panel.cc
@@ -217,20 +217,13 @@ VideoPanel::VideoPanel (ContentPanel* p)
void
VideoPanel::add_to_grid ()
{
- bool const full = Config::instance()->interface_complexity() == Config::INTERFACE_FULL;
-
int r = 0;
- _reference->Show (full);
- _reference_note->Show (full && !_reference_note->GetLabel().IsEmpty());
-
- if (full) {
- wxBoxSizer* reference_sizer = new wxBoxSizer (wxVERTICAL);
- reference_sizer->Add (_reference, 0);
- reference_sizer->Add (_reference_note, 0);
- _grid->Add (reference_sizer, wxGBPosition(r, 0), wxGBSpan(1, 3));
- ++r;
- }
+ wxBoxSizer* reference_sizer = new wxBoxSizer (wxVERTICAL);
+ reference_sizer->Add (_reference, 0);
+ reference_sizer->Add (_reference_note, 0);
+ _grid->Add (reference_sizer, wxGBPosition(r, 0), wxGBSpan(1, 3));
+ ++r;
add_label_to_sizer (_grid, _type_label, true, wxGBPosition(r, 0));
_frame_type->add (_grid, wxGBPosition(r, 1), wxGBSpan(1, 2));
@@ -267,16 +260,6 @@ VideoPanel::add_to_grid ()
_grid->Add (crop, wxGBPosition(r, 1));
++r;
- _scale_label->Show (full);
- _scale_fit->Show (full);
- _scale_custom->Show (full);
- _scale_custom_edit->Show (full);
- _colour_conversion_label->Show (full);
- _colour_conversion->Show (full);
- _edit_colour_conversion_button->Show (full);
- _range_label->Show (full);
- _range->Show (full);
-
add_label_to_sizer (_grid, _fade_in_label, true, wxGBPosition (r, 0));
_grid->Add (_fade_in, wxGBPosition (r, 1), wxGBSpan (1, 3));
++r;
@@ -285,32 +268,30 @@ VideoPanel::add_to_grid ()
_grid->Add (_fade_out, wxGBPosition (r, 1), wxGBSpan (1, 3));
++r;
- if (full) {
- add_label_to_sizer (_grid, _scale_label, true, wxGBPosition (r, 0));
- {
- wxSizer* v = new wxBoxSizer (wxVERTICAL);
- v->Add (_scale_fit, 0, wxBOTTOM, 4);
- wxSizer* h = new wxBoxSizer (wxHORIZONTAL);
- h->Add (_scale_custom, 1, wxRIGHT | wxALIGN_CENTER_VERTICAL, 6);
- h->Add (_scale_custom_edit, 0, wxALIGN_CENTER_VERTICAL);
- v->Add (h, 0);
- _grid->Add (v, wxGBPosition(r, 1));
- }
- ++r;
-
- add_label_to_sizer (_grid, _colour_conversion_label, true, wxGBPosition(r, 0));
- {
- wxSizer* s = new wxBoxSizer (wxHORIZONTAL);
- s->Add (_colour_conversion, 1, wxALIGN_CENTER_VERTICAL | wxTOP | wxBOTTOM | wxRIGHT, 6);
- s->Add (_edit_colour_conversion_button, 0, wxALIGN_CENTER_VERTICAL);
- _grid->Add (s, wxGBPosition (r, 1), wxDefaultSpan, wxALIGN_CENTER_VERTICAL);
- }
- ++r;
+ add_label_to_sizer (_grid, _scale_label, true, wxGBPosition (r, 0));
+ {
+ wxSizer* v = new wxBoxSizer (wxVERTICAL);
+ v->Add (_scale_fit, 0, wxBOTTOM, 4);
+ wxSizer* h = new wxBoxSizer (wxHORIZONTAL);
+ h->Add (_scale_custom, 1, wxRIGHT | wxALIGN_CENTER_VERTICAL, 6);
+ h->Add (_scale_custom_edit, 0, wxALIGN_CENTER_VERTICAL);
+ v->Add (h, 0);
+ _grid->Add (v, wxGBPosition(r, 1));
+ }
+ ++r;
- add_label_to_sizer (_grid, _range_label, true, wxGBPosition(r, 0));
- _grid->Add (_range, wxGBPosition(r, 1), wxGBSpan(1, 2), wxALIGN_CENTER_VERTICAL);
- ++r;
+ add_label_to_sizer (_grid, _colour_conversion_label, true, wxGBPosition(r, 0));
+ {
+ wxSizer* s = new wxBoxSizer (wxHORIZONTAL);
+ s->Add (_colour_conversion, 1, wxALIGN_CENTER_VERTICAL | wxTOP | wxBOTTOM | wxRIGHT, 6);
+ s->Add (_edit_colour_conversion_button, 0, wxALIGN_CENTER_VERTICAL);
+ _grid->Add (s, wxGBPosition (r, 1), wxDefaultSpan, wxALIGN_CENTER_VERTICAL);
}
+ ++r;
+
+ add_label_to_sizer (_grid, _range_label, true, wxGBPosition(r, 0));
+ _grid->Add (_range, wxGBPosition(r, 1), wxGBSpan(1, 2), wxALIGN_CENTER_VERTICAL);
+ ++r;
_grid->Add (_description, wxGBPosition (r, 0), wxGBSpan (1, 4), wxEXPAND | wxALIGN_CENTER_VERTICAL, 6);
++r;
diff --git a/src/wx/wscript b/src/wx/wscript
index a5597a87b..dee54e038 100644
--- a/src/wx/wscript
+++ b/src/wx/wscript
@@ -81,7 +81,6 @@ sources = """
gl_video_view.cc
hints_dialog.cc
html_dialog.cc
- initial_setup_dialog.cc
instant_i18n_dialog.cc
interop_metadata_dialog.cc
i18n_hook.cc