Fix alignment of labels on macOS (#2043).
[dcpomatic.git] / src / wx / editable_list.h
index a7c1a610295da46c6fdad210b7afb58881d4600d..3f2ea0ca2fe0afad40f7a1e2b5512e6c817282ac 100644 (file)
 #include "dcpomatic_button.h"
 #include <wx/wx.h>
 #include <wx/listctrl.h>
-#include <boost/function.hpp>
 #include <vector>
 
+class EditableListColumn
+{
+public:
+       EditableListColumn (wxString name_)
+               : name (name_)
+               , growable (false)
+       {}
+
+       EditableListColumn (wxString name_, boost::optional<int> width_, bool growable_)
+               : name (name_)
+               , width (width_)
+               , growable (growable_)
+       {}
+
+       wxString name;
+       boost::optional<int> width;
+       bool growable;
+};
+
 /** @param T type of things being edited.
  *  @param S dialog to edit a thing.
+ *  @param get Function to get a std::vector of the things being edited.
+ *  @param set Function set the things from a a std::vector.
+ *  @param column Function to get the display string for a given column in a given item.
  */
 template<class T, class S>
 class EditableList : public wxPanel
@@ -37,50 +58,72 @@ class EditableList : public wxPanel
 public:
        EditableList (
                wxWindow* parent,
-               std::vector<std::string> columns,
-               boost::function<std::vector<T> ()> get,
-               boost::function<void (std::vector<T>)> set,
-               boost::function<std::string (T, int)> column,
+               std::vector<EditableListColumn> columns,
+               std::function<std::vector<T> ()> get,
+               std::function<void (std::vector<T>)> set,
+               std::function<std::string (T, int)> column,
                bool can_edit = true,
-               bool title = true,
-               int column_width = 200
+               bool title = true
                )
                : wxPanel (parent)
                , _get (get)
                , _set (set)
-               , _columns (columns.size ())
+               , _columns (columns)
                , _column (column)
                , _edit (0)
+               , _default_width (200)
        {
                _sizer = new wxBoxSizer (wxHORIZONTAL);
                SetSizer (_sizer);
 
                long style = wxLC_REPORT | wxLC_SINGLE_SEL;
-               if (title) {
+               if (!title) {
                        style |= wxLC_NO_HEADER;
                }
-               _list = new wxListCtrl (this, wxID_ANY, wxDefaultPosition, wxSize (columns.size() * column_width, 100), style);
 
-               for (size_t i = 0; i < columns.size(); ++i) {
+               int total_width = 0;
+               for (auto i: _columns) {
+                       total_width += i.width.get_value_or (_default_width);
+               }
+
+#ifdef __WXGTK3__
+               /* With the GTK3 backend wxListCtrls are hard to pick out from the background of the
+                * window, so put a border in to help.
+                */
+               wxPanel* border = new wxPanel (this, wxID_ANY, wxDefaultPosition, wxDefaultSize, wxTAB_TRAVERSAL | wxBORDER_THEME);
+               _list = new wxListCtrl (border, wxID_ANY, wxDefaultPosition, wxSize(total_width, 100), style);
+               wxBoxSizer* border_sizer = new wxBoxSizer (wxHORIZONTAL);
+               border_sizer->Add (_list, 1, wxALL | wxEXPAND, 2);
+               border->SetSizer (border_sizer);
+#else
+               _list = new wxListCtrl (this, wxID_ANY, wxDefaultPosition, wxSize(total_width, 100), style);
+#endif
+
+               int j = 0;
+               for (auto i: _columns) {
                        wxListItem ip;
-                       ip.SetId (i);
-                       ip.SetText (std_to_wx (columns[i]));
-                       ip.SetWidth (column_width);
-                       _list->InsertColumn (i, ip);
+                       ip.SetId (j);
+                       ip.SetText (i.name);
+                       _list->InsertColumn (j, ip);
+                       ++j;
                }
 
+#ifdef __WXGTK3__
+               _sizer->Add (border, 1, wxEXPAND);
+#else
                _sizer->Add (_list, 1, wxEXPAND);
+#endif
 
                {
                        wxSizer* s = new wxBoxSizer (wxVERTICAL);
                        _add = new Button (this, _("Add..."));
-                       s->Add (_add, 0, wxTOP | wxBOTTOM, 2);
+                       s->Add (_add, 1, wxEXPAND | wxTOP | wxBOTTOM, DCPOMATIC_BUTTON_STACK_GAP);
                        if (can_edit) {
                                _edit = new Button (this, _("Edit..."));
-                               s->Add (_edit, 0, wxTOP | wxBOTTOM, 2);
+                               s->Add (_edit, 1, wxEXPAND | wxTOP | wxBOTTOM, DCPOMATIC_BUTTON_STACK_GAP);
                        }
                        _remove = new Button (this, _("Remove"));
-                       s->Add (_remove, 0, wxTOP | wxBOTTOM, 2);
+                       s->Add (_remove, 1, wxEXPAND | wxTOP | wxBOTTOM, DCPOMATIC_BUTTON_STACK_GAP);
                        _sizer->Add (s, 0, wxLEFT, DCPOMATIC_SIZER_X_GAP);
                }
 
@@ -92,7 +135,11 @@ public:
 
                _list->Bind (wxEVT_COMMAND_LIST_ITEM_SELECTED, boost::bind (&EditableList::selection_changed, this));
                _list->Bind (wxEVT_COMMAND_LIST_ITEM_DESELECTED, boost::bind (&EditableList::selection_changed, this));
+#if BOOST_VERSION >= 106100
+               _list->Bind (wxEVT_SIZE, boost::bind (&EditableList::resized, this, boost::placeholders::_1));
+#else
                _list->Bind (wxEVT_SIZE, boost::bind (&EditableList::resized, this, _1));
+#endif
 
                refresh ();
                selection_changed ();
@@ -136,7 +183,7 @@ private:
                list_item.SetId (n);
                _list->InsertItem (list_item);
 
-               for (int i = 0; i < _columns; ++i) {
+               for (size_t i = 0; i < _columns.size(); ++i) {
                        _list->SetItem (n, i, std_to_wx (_column (item, i)));
                }
        }
@@ -191,7 +238,7 @@ private:
                }
                dialog->Destroy ();
 
-               for (int i = 0; i < _columns; ++i) {
+               for (size_t i = 0; i < _columns.size(); ++i) {
                        _list->SetItem (item, i, std_to_wx (_column (all[item], i)));
                }
 
@@ -215,23 +262,43 @@ private:
 
        void resized (wxSizeEvent& ev)
        {
-               int const w = GetSize().GetWidth() / _columns;
-               for (int i = 0; i < _columns; ++i) {
-                       _list->SetColumnWidth (i, w);
+               int const w = _list->GetSize().GetWidth() - 2;
+
+               int fixed_width = 0;
+               int growable = 0;
+               int j = 0;
+               for (auto i: _columns) {
+                       fixed_width += i.width.get_value_or (_default_width);
+                       if (!i.growable) {
+                               _list->SetColumnWidth (j, i.width.get_value_or(_default_width));
+                       } else {
+                               ++growable;
+                       }
+                       ++j;
                }
+
+               j = 0;
+               for (auto i: _columns) {
+                       if (i.growable) {
+                               _list->SetColumnWidth (j, i.width.get_value_or(_default_width) + (w - fixed_width) / growable);
+                       }
+                       ++j;
+               }
+
                ev.Skip ();
        }
 
-       boost::function <std::vector<T> ()> _get;
-       boost::function <void (std::vector<T>)> _set;
-       int _columns;
-       boost::function<std::string (T, int)> _column;
+       std::function <std::vector<T> ()> _get;
+       std::function <void (std::vector<T>)> _set;
+       std::vector<EditableListColumn> _columns;
+       std::function<std::string (T, int)> _column;
 
        wxButton* _add;
        wxButton* _edit;
        wxButton* _remove;
        wxListCtrl* _list;
        wxBoxSizer* _sizer;
+       int _default_width;
 };
 
 #endif