projects
/
dcpomatic.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Update dialog layout fixes.
[dcpomatic.git]
/
src
/
wx
/
content_panel.cc
diff --git
a/src/wx/content_panel.cc
b/src/wx/content_panel.cc
index a06791d4d0766be5351a43a8f9ea4ae893a0a634..bcc6999139f248255b1ad2f61ca3697ddc1e87ef 100644
(file)
--- a/
src/wx/content_panel.cc
+++ b/
src/wx/content_panel.cc
@@
-1,5
+1,5
@@
/*
/*
- Copyright (C) 2012-20
19
Carl Hetherington <cth@carlh.net>
+ Copyright (C) 2012-20
21
Carl Hetherington <cth@carlh.net>
This file is part of DCP-o-matic.
This file is part of DCP-o-matic.
@@
-18,31
+18,33
@@
*/
*/
-#include "content_panel.h"
-#include "wx_util.h"
-#include "video_panel.h"
+
#include "audio_panel.h"
#include "audio_panel.h"
+#include "content_panel.h"
+#include "dcpomatic_button.h"
+#include "film_viewer.h"
+#include "image_sequence_dialog.h"
#include "text_panel.h"
#include "text_panel.h"
-#include "timing_panel.h"
#include "timeline_dialog.h"
#include "timeline_dialog.h"
-#include "
image_sequence_dialog
.h"
-#include "
film_viewer
.h"
-#include "
dcpomatic_button
.h"
+#include "
timing_panel
.h"
+#include "
video_panel
.h"
+#include "
wx_util
.h"
#include "lib/audio_content.h"
#include "lib/audio_content.h"
-#include "lib/text_content.h"
-#include "lib/video_content.h"
-#include "lib/ffmpeg_content.h"
-#include "lib/content_factory.h"
-#include "lib/image_content.h"
-#include "lib/dcp_content.h"
#include "lib/case_insensitive_sorter.h"
#include "lib/case_insensitive_sorter.h"
-#include "lib/
playlist.h
"
+#include "lib/
compose.hpp
"
#include "lib/config.h"
#include "lib/config.h"
+#include "lib/content_factory.h"
+#include "lib/cross.h"
+#include "lib/dcp_content.h"
+#include "lib/dcpomatic_log.h"
+#include "lib/ffmpeg_content.h"
+#include "lib/image_content.h"
#include "lib/log.h"
#include "lib/log.h"
-#include "lib/compose.hpp"
-#include "lib/string_text_file_content.h"
+#include "lib/playlist.h"
#include "lib/string_text_file.h"
#include "lib/string_text_file.h"
-#include "lib/dcpomatic_log.h"
+#include "lib/string_text_file_content.h"
+#include "lib/text_content.h"
+#include "lib/video_content.h"
#include <wx/wx.h>
#include <wx/notebook.h>
#include <wx/listctrl.h>
#include <wx/wx.h>
#include <wx/notebook.h>
#include <wx/listctrl.h>
@@
-50,15
+52,17
@@
#include <boost/filesystem.hpp>
#include <iostream>
#include <boost/filesystem.hpp>
#include <iostream>
-using std::list;
-using std::string;
+
using std::cout;
using std::cout;
-using std::vector;
-using std::max;
+using std::dynamic_pointer_cast;
using std::exception;
using std::exception;
+using std::list;
+using std::make_shared;
+using std::max;
using std::shared_ptr;
using std::shared_ptr;
+using std::string;
+using std::vector;
using std::weak_ptr;
using std::weak_ptr;
-using std::dynamic_pointer_cast;
using boost::optional;
using namespace dcpomatic;
#if BOOST_VERSION >= 106100
using boost::optional;
using namespace dcpomatic;
#if BOOST_VERSION >= 106100
@@
-67,11
+71,7
@@
using namespace boost::placeholders;
ContentPanel::ContentPanel (wxNotebook* n, shared_ptr<Film> film, weak_ptr<FilmViewer> viewer)
ContentPanel::ContentPanel (wxNotebook* n, shared_ptr<Film> film, weak_ptr<FilmViewer> viewer)
- : _video_panel (0)
- , _audio_panel (0)
- , _timeline_dialog (0)
- , _parent (n)
- , _last_selected_tab (0)
+ : _parent (n)
, _film (film)
, _film_viewer (viewer)
, _generally_sensitive (true)
, _film (film)
, _film_viewer (viewer)
, _generally_sensitive (true)
@@
-177,6
+177,7
@@
ContentPanel::selected ()
return sel;
}
return sel;
}
+
ContentList
ContentPanel::selected_video ()
{
ContentList
ContentPanel::selected_video ()
{
@@
-191,6
+192,7
@@
ContentPanel::selected_video ()
return vc;
}
return vc;
}
+
ContentList
ContentPanel::selected_audio ()
{
ContentList
ContentPanel::selected_audio ()
{
@@
-205,6
+207,7
@@
ContentPanel::selected_audio ()
return ac;
}
return ac;
}
+
ContentList
ContentPanel::selected_text ()
{
ContentList
ContentPanel::selected_text ()
{
@@
-219,6
+222,7
@@
ContentPanel::selected_text ()
return sc;
}
return sc;
}
+
FFmpegContentList
ContentPanel::selected_ffmpeg ()
{
FFmpegContentList
ContentPanel::selected_ffmpeg ()
{
@@
-234,12
+238,13
@@
ContentPanel::selected_ffmpeg ()
return sc;
}
return sc;
}
+
void
ContentPanel::film_changed (Film::Property p)
{
switch (p) {
void
ContentPanel::film_changed (Film::Property p)
{
switch (p) {
- case Film::CONTENT:
- case Film::CONTENT_ORDER:
+ case Film::
Property::
CONTENT:
+ case Film::
Property::
CONTENT_ORDER:
setup ();
break;
default:
setup ();
break;
default:
@@
-251,6
+256,7
@@
ContentPanel::film_changed (Film::Property p)
}
}
}
}
+
void
ContentPanel::item_deselected ()
{
void
ContentPanel::item_deselected ()
{
@@
-261,6
+267,7
@@
ContentPanel::item_deselected ()
signal_manager->when_idle (boost::bind (&ContentPanel::item_deselected_idle, this));
}
signal_manager->when_idle (boost::bind (&ContentPanel::item_deselected_idle, this));
}
+
void
ContentPanel::item_deselected_idle ()
{
void
ContentPanel::item_deselected_idle ()
{
@@
-269,6
+276,7
@@
ContentPanel::item_deselected_idle ()
}
}
}
}
+
void
ContentPanel::item_selected ()
{
void
ContentPanel::item_selected ()
{
@@
-276,6
+284,7
@@
ContentPanel::item_selected ()
check_selection ();
}
check_selection ();
}
+
void
ContentPanel::check_selection ()
{
void
ContentPanel::check_selection ()
{
@@
-401,6
+410,7
@@
ContentPanel::check_selection ()
SelectionChanged ();
}
SelectionChanged ();
}
+
void
ContentPanel::add_file_clicked ()
{
void
ContentPanel::add_file_clicked ()
{
@@
-411,13
+421,15
@@
ContentPanel::add_file_clicked ()
return;
}
return;
}
+ auto path = Config::instance()->add_files_path();
+
/* The wxFD_CHANGE_DIR here prevents a `could not set working directory' error 123 on Windows when using
non-Latin filenames or paths.
*/
auto d = new wxFileDialog (
_splitter,
_("Choose a file or files"),
/* The wxFD_CHANGE_DIR here prevents a `could not set working directory' error 123 on Windows when using
non-Latin filenames or paths.
*/
auto d = new wxFileDialog (
_splitter,
_("Choose a file or files"),
-
wxT (""
),
+
std_to_wx(path ? path->string() : home_directory().string()
),
wxT (""),
wxT ("All files|*.*|Subtitle files|*.srt;*.xml|Audio files|*.wav;*.w64;*.flac;*.aif;*.aiff"),
wxFD_MULTIPLE | wxFD_CHANGE_DIR
wxT (""),
wxT ("All files|*.*|Subtitle files|*.srt;*.xml|Audio files|*.wav;*.w64;*.flac;*.aif;*.aiff"),
wxFD_MULTIPLE | wxFD_CHANGE_DIR
@@
-432,15
+444,20
@@
ContentPanel::add_file_clicked ()
wxArrayString paths;
d->GetPaths (paths);
wxArrayString paths;
d->GetPaths (paths);
-
list
<boost::filesystem::path> path_list;
+
vector
<boost::filesystem::path> path_list;
for (unsigned int i = 0; i < paths.GetCount(); ++i) {
for (unsigned int i = 0; i < paths.GetCount(); ++i) {
- path_list.push_back (wx_to_std
(paths[i]));
+ path_list.push_back (wx_to_std(paths[i]));
}
add_files (path_list);
}
add_files (path_list);
+ if (!path_list.empty()) {
+ Config::instance()->set_add_files_path(path_list[0].parent_path());
+ }
+
d->Destroy ();
}
d->Destroy ();
}
+
void
ContentPanel::add_folder_clicked ()
{
void
ContentPanel::add_folder_clicked ()
{
@@
-486,6
+503,7
@@
ContentPanel::add_folder_clicked ()
}
}
}
}
+
void
ContentPanel::add_dcp_clicked ()
{
void
ContentPanel::add_dcp_clicked ()
{
@@
-499,12
+517,21
@@
ContentPanel::add_dcp_clicked ()
}
try {
}
try {
- _film->examine_and_add_content (shared_ptr<Content> (new DCPContent (path)));
+ _film->examine_and_add_content (make_shared<DCPContent>(path));
+ } catch (ProjectFolderError &) {
+ error_dialog (
+ _parent,
+ _(
+ "This looks like a DCP-o-matic project folder, which cannot be added to a different project. "
+ "Choose the DCP directory inside the DCP-o-matic project folder if that's what you want to import."
+ )
+ );
} catch (exception& e) {
error_dialog (_parent, e.what());
}
}
} catch (exception& e) {
error_dialog (_parent, e.what());
}
}
+
/** @return true if this remove "click" should be ignored */
bool
ContentPanel::remove_clicked (bool hotkey)
/** @return true if this remove "click" should be ignored */
bool
ContentPanel::remove_clicked (bool hotkey)
@@
-524,6
+551,7
@@
ContentPanel::remove_clicked (bool hotkey)
return false;
}
return false;
}
+
void
ContentPanel::timeline_clicked ()
{
void
ContentPanel::timeline_clicked ()
{
@@
-541,12
+569,14
@@
ContentPanel::timeline_clicked ()
_timeline_dialog->Show ();
}
_timeline_dialog->Show ();
}
+
void
ContentPanel::right_click (wxListEvent& ev)
{
_menu->popup (_film, selected (), TimelineContentViewList (), ev.GetPoint ());
}
void
ContentPanel::right_click (wxListEvent& ev)
{
_menu->popup (_film, selected (), TimelineContentViewList (), ev.GetPoint ());
}
+
/** Set up broad sensitivity based on the type of content that is selected */
void
ContentPanel::setup_sensitivity ()
/** Set up broad sensitivity based on the type of content that is selected */
void
ContentPanel::setup_sensitivity ()
@@
-578,6
+608,7
@@
ContentPanel::setup_sensitivity ()
_timing_panel->Enable (_generally_sensitive);
}
_timing_panel->Enable (_generally_sensitive);
}
+
void
ContentPanel::set_film (shared_ptr<Film> film)
{
void
ContentPanel::set_film (shared_ptr<Film> film)
{
@@
-587,8
+618,8
@@
ContentPanel::set_film (shared_ptr<Film> film)
_film = film;
_film = film;
- film_changed (Film::CONTENT);
- film_changed (Film::AUDIO_CHANNELS);
+ film_changed (Film::
Property::
CONTENT);
+ film_changed (Film::
Property::
AUDIO_CHANNELS);
if (_film) {
check_selection ();
if (_film) {
check_selection ();
@@
-597,6
+628,7
@@
ContentPanel::set_film (shared_ptr<Film> film)
setup_sensitivity ();
}
setup_sensitivity ();
}
+
void
ContentPanel::set_general_sensitivity (bool s)
{
void
ContentPanel::set_general_sensitivity (bool s)
{
@@
-604,6
+636,7
@@
ContentPanel::set_general_sensitivity (bool s)
setup_sensitivity ();
}
setup_sensitivity ();
}
+
void
ContentPanel::earlier_clicked ()
{
void
ContentPanel::earlier_clicked ()
{
@@
-614,6
+647,7
@@
ContentPanel::earlier_clicked ()
}
}
}
}
+
void
ContentPanel::later_clicked ()
{
void
ContentPanel::later_clicked ()
{
@@
-624,6
+658,7
@@
ContentPanel::later_clicked ()
}
}
}
}
+
void
ContentPanel::set_selection (weak_ptr<Content> wc)
{
void
ContentPanel::set_selection (weak_ptr<Content> wc)
{
@@
-637,6
+672,7
@@
ContentPanel::set_selection (weak_ptr<Content> wc)
}
}
}
}
+
void
ContentPanel::set_selection (ContentList cl)
{
void
ContentPanel::set_selection (ContentList cl)
{
@@
-655,6
+691,14
@@
ContentPanel::set_selection (ContentList cl)
check_selection ();
}
check_selection ();
}
+
+void
+ContentPanel::select_all ()
+{
+ set_selection (_film->content());
+}
+
+
void
ContentPanel::film_content_changed (int property)
{
void
ContentPanel::film_content_changed (int property)
{
@@
-673,6
+717,7
@@
ContentPanel::film_content_changed (int property)
}
}
}
}
+
void
ContentPanel::setup ()
{
void
ContentPanel::setup ()
{
@@
-741,6
+786,7
@@
ContentPanel::setup ()
setup_sensitivity ();
}
setup_sensitivity ();
}
+
void
ContentPanel::files_dropped (wxDropFilesEvent& event)
{
void
ContentPanel::files_dropped (wxDropFilesEvent& event)
{
@@
-749,23
+795,24
@@
ContentPanel::files_dropped (wxDropFilesEvent& event)
}
auto paths = event.GetFiles ();
}
auto paths = event.GetFiles ();
-
list
<boost::filesystem::path> path_list;
+
vector
<boost::filesystem::path> path_list;
for (int i = 0; i < event.GetNumberOfFiles(); i++) {
for (int i = 0; i < event.GetNumberOfFiles(); i++) {
- path_list.push_back (wx_to_std
(paths[i]));
+ path_list.push_back (wx_to_std(paths[i]));
}
add_files (path_list);
}
}
add_files (path_list);
}
+
void
void
-ContentPanel::add_files (
list
<boost::filesystem::path> paths)
+ContentPanel::add_files (
vector
<boost::filesystem::path> paths)
{
/* It has been reported that the paths returned from e.g. wxFileDialog are not always sorted;
I can't reproduce that, but sort them anyway. Don't use ImageFilenameSorter as a normal
alphabetical sort is expected here.
*/
{
/* It has been reported that the paths returned from e.g. wxFileDialog are not always sorted;
I can't reproduce that, but sort them anyway. Don't use ImageFilenameSorter as a normal
alphabetical sort is expected here.
*/
-
paths.sort (CaseInsensitiveSorter
());
+
std::sort (paths.begin(), paths.end(), CaseInsensitiveSorter
());
/* XXX: check for lots of files here and do something */
/* XXX: check for lots of files here and do something */
@@
-780,6
+827,7
@@
ContentPanel::add_files (list<boost::filesystem::path> paths)
}
}
}
}
+
list<ContentSubPanel*>
ContentPanel::panels () const
{
list<ContentSubPanel*>
ContentPanel::panels () const
{