X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Fwx%2Feditable_list.h;h=3f2ea0ca2fe0afad40f7a1e2b5512e6c817282ac;hb=ccacce39c39d16977ab6c1592fcb6e941b05ddff;hp=1f0ead3cdf09b749c4385e843406f387a03dbf71;hpb=03dd6e03f5ee261b9c1ed9328ad2762ef3b62057;p=dcpomatic.git diff --git a/src/wx/editable_list.h b/src/wx/editable_list.h index 1f0ead3cd..3f2ea0ca2 100644 --- a/src/wx/editable_list.h +++ b/src/wx/editable_list.h @@ -1,19 +1,20 @@ /* - Copyright (C) 2012-2015 Carl Hetherington + Copyright (C) 2012-2016 Carl Hetherington - This program is free software; you can redistribute it and/or modify + 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. - This program is distributed in the hope that it will be useful, + 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 this program; if not, write to the Free Software - Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. + along with DCP-o-matic. If not, see . */ @@ -21,13 +22,35 @@ #define DCPOMATIC_EDITABLE_LIST_H #include "wx_util.h" +#include "dcpomatic_button.h" #include #include -#include #include +class EditableListColumn +{ +public: + EditableListColumn (wxString name_) + : name (name_) + , growable (false) + {} + + EditableListColumn (wxString name_, boost::optional width_, bool growable_) + : name (name_) + , width (width_) + , growable (growable_) + {} + + wxString name; + boost::optional 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 EditableList : public wxPanel @@ -35,54 +58,73 @@ class EditableList : public wxPanel public: EditableList ( wxWindow* parent, - std::vector columns, - boost::function ()> get, - boost::function)> set, - boost::function column, + std::vector columns, + std::function ()> get, + std::function)> set, + std::function column, bool can_edit = true, bool title = true ) : wxPanel (parent) , _get (get) , _set (set) - , _columns (columns.size ()) + , _columns (columns) , _column (column) , _edit (0) + , _default_width (200) { - wxBoxSizer* s = new wxBoxSizer (wxVERTICAL); - SetSizer (s); - - wxFlexGridSizer* table = new wxFlexGridSizer (2, DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP); - table->AddGrowableCol (0, 1); - s->Add (table, 1, wxEXPAND); + _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() * 200, 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 (200); - _list->InsertColumn (i, ip); + ip.SetId (j); + ip.SetText (i.name); + _list->InsertColumn (j, ip); + ++j; } - table->Add (_list, 1, wxEXPAND | wxALL); +#ifdef __WXGTK3__ + _sizer->Add (border, 1, wxEXPAND); +#else + _sizer->Add (_list, 1, wxEXPAND); +#endif { wxSizer* s = new wxBoxSizer (wxVERTICAL); - _add = new wxButton (this, wxID_ANY, _("Add...")); - s->Add (_add, 0, wxTOP | wxBOTTOM, 2); + _add = new Button (this, _("Add...")); + s->Add (_add, 1, wxEXPAND | wxTOP | wxBOTTOM, DCPOMATIC_BUTTON_STACK_GAP); if (can_edit) { - _edit = new wxButton (this, wxID_ANY, _("Edit...")); - s->Add (_edit, 0, wxTOP | wxBOTTOM, 2); + _edit = new Button (this, _("Edit...")); + s->Add (_edit, 1, wxEXPAND | wxTOP | wxBOTTOM, DCPOMATIC_BUTTON_STACK_GAP); } - _remove = new wxButton (this, wxID_ANY, _("Remove")); - s->Add (_remove, 0, wxTOP | wxBOTTOM, 2); - table->Add (s, 0); + _remove = new Button (this, _("Remove")); + s->Add (_remove, 1, wxEXPAND | wxTOP | wxBOTTOM, DCPOMATIC_BUTTON_STACK_GAP); + _sizer->Add (s, 0, wxLEFT, DCPOMATIC_SIZER_X_GAP); } _add->Bind (wxEVT_COMMAND_BUTTON_CLICKED, boost::bind (&EditableList::add_clicked, this)); @@ -93,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 (); @@ -121,6 +167,11 @@ public: return all[item]; } + void layout () + { + _sizer->Layout (); + } + boost::signals2::signal SelectionChanged; private: @@ -132,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))); } } @@ -153,10 +204,13 @@ private: S* dialog = new S (this); if (dialog->ShowModal() == wxID_OK) { - add_to_control (dialog->get ()); - std::vector all = _get (); - all.push_back (dialog->get ()); - _set (all); + boost::optional const v = dialog->get (); + if (v) { + add_to_control (v.get ()); + std::vector all = _get (); + all.push_back (v.get ()); + _set (all); + } } dialog->Destroy (); @@ -175,11 +229,16 @@ private: S* dialog = new S (this); dialog->set (all[item]); if (dialog->ShowModal() == wxID_OK) { - all[item] = dialog->get (); + boost::optional const v = dialog->get (); + if (!v) { + return; + } + + all[item] = v.get (); } 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))); } @@ -203,22 +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 ()> _get; - boost::function )> _set; - int _columns; - boost::function _column; + std::function ()> _get; + std::function )> _set; + std::vector _columns; + std::function _column; wxButton* _add; wxButton* _edit; wxButton* _remove; wxListCtrl* _list; + wxBoxSizer* _sizer; + int _default_width; }; #endif