Use wx_ptr more.

[dcpomatic.git] / src / wx / content_panel.h

diff --git a/src/wx/content_panel.h b/src/wx/content_panel.h
index 604a15fe49d575bc4470eb25905160bf732ed1d2..88686067221253ea674bf66ef611727cbc101706 100644 (file)
--- a/src/wx/content_panel.h
+++ b/src/wx/content_panel.h
@@ -20,20 +20,23 @@
 
 
 #include "content_menu.h"
+#include "lib/enum_indexed_vector.h"
 #include "lib/film.h"
-#include "lib/types.h"
-#include "lib/warnings.h"
-DCPOMATIC_DISABLE_WARNINGS
+#include "lib/text_type.h"
+#include <dcp/warnings.h>
+LIBDCP_DISABLE_WARNINGS
 #include <wx/splitter.h>
-DCPOMATIC_ENABLE_WARNINGS
+LIBDCP_ENABLE_WARNINGS
 #include <list>
 
 
 class AudioPanel;
+class ContentListCtrl;
 class ContentSubPanel;
 class Film;
 class FilmEditor;
 class FilmViewer;
+class LimitedContentPanelSplitter;
 class TextPanel;
 class TimelineDialog;
 class TimingPanel;
@@ -46,31 +49,10 @@ class wxSizer;
 class wxSplitterWindow;
 
 
-class LimitedSplitter : public wxSplitterWindow
-{
-public:
-       LimitedSplitter (wxWindow* parent);
-
-       bool OnSashPositionChange (int new_position) override
-       {
-               /* Try to stop the top bit of the splitter getting so small that buttons disappear */
-               return new_position > 220;
-       }
-
-       void first_shown (wxWindow* top, wxWindow* bottom);
-
-private:
-       void sized (wxSizeEvent& ev);
-
-       bool _first_shown;
-       int const _top_panel_minimum_size;
-};
-
-
 class ContentPanel
 {
 public:
-       ContentPanel (wxNotebook *, std::shared_ptr<Film>, std::weak_ptr<FilmViewer> viewer);
+       ContentPanel(wxNotebook *, std::shared_ptr<Film>, FilmViewer& viewer);
 
        ContentPanel (ContentPanel const&) = delete;
        ContentPanel& operator= (ContentPanel const&) = delete;
@@ -90,9 +72,7 @@ public:
 
        void first_shown ();
 
-       wxWindow* window () const {
-               return _splitter;
-       }
+       wxWindow* window () const;
 
        wxNotebook* notebook () const {
                return _notebook;
@@ -108,10 +88,14 @@ public:
        bool remove_clicked (bool hotkey);
        void timeline_clicked ();
 
-       std::weak_ptr<FilmViewer> film_viewer () const {
+       FilmViewer& film_viewer() const {
                return _film_viewer;
        }
 
+       void add_files(std::vector<boost::filesystem::path> files);
+       void add_dcp(boost::filesystem::path dcp);
+       void add_folder(boost::filesystem::path folder);
+
        boost::signals2::signal<void (void)> SelectionChanged;
 
 private:
@@ -125,17 +109,18 @@ private:
        void later_clicked ();
        void right_click (wxListEvent &);
        void files_dropped (wxDropFilesEvent &);
+       boost::optional<boost::filesystem::path> add_files_override_path() const;
 
        void setup ();
        void setup_sensitivity ();
+       void set_selected_state(int item, bool state);
 
-       void add_files (std::vector<boost::filesystem::path>);
        std::list<ContentSubPanel *> panels () const;
 
-       LimitedSplitter* _splitter;
+       LimitedContentPanelSplitter* _splitter;
        wxPanel* _top_panel;
        wxNotebook* _notebook;
-       wxListCtrl* _content;
+       ContentListCtrl* _content;
        wxButton* _add_file;
        wxButton* _add_folder;
        wxButton* _add_dcp;
@@ -145,15 +130,15 @@ private:
        wxButton* _timeline;
        VideoPanel* _video_panel = nullptr;
        AudioPanel* _audio_panel = nullptr;
-       TextPanel* _text_panel[static_cast<int>(TextType::COUNT)];
+       EnumIndexedVector<TextPanel*, TextType> _text_panel;
        TimingPanel* _timing_panel;
        ContentMenu* _menu;
-       TimelineDialog* _timeline_dialog = nullptr;
+       wx_ptr<TimelineDialog> _timeline_dialog;
        wxNotebook* _parent;
        wxWindow* _last_selected_tab = nullptr;
 
        std::shared_ptr<Film> _film;
-       std::weak_ptr<FilmViewer> _film_viewer;
+       FilmViewer& _film_viewer;
        bool _generally_sensitive;
        bool _ignore_deselect;
        bool _no_check_selection;