dcpomatic/src/wx/content_panel.cc, branch cpp17DCP-o-matic DCP toolshttps://git.carlh.net/cgit/dcpomatic/atom?h=cpp172025-05-09T23:22:02ZWhite space: content_panel.cc2025-05-09T23:22:02ZCarl Hetheringtoncth@carlh.net2025-05-09T23:22:02Zurn:sha1:44206b1fc91f40d584c812a84a810ada5bcf7850Report content as having missing files also if there are missing fonts.2025-03-05T22:52:14ZCarl Hetheringtoncth@carlh.net2025-03-04T23:47:37Zurn:sha1:58ec6491cf182f3c314aaa0b2b2968838c286809Replace Content::paths_valid() with a free utility function.2025-03-05T22:52:14ZCarl Hetheringtoncth@carlh.net2025-03-04T22:24:59Zurn:sha1:ab766096598188759d29829043bf28e63379b730Fix misunderstanding of wxDialog lifetime handling.2025-02-25T12:04:08ZCarl Hetheringtoncth@carlh.net2025-02-25T00:53:48Zurn:sha1:6d3086dd78a35bdc179a6579e23f8b5816347dbf
Broken by d0308d53dd9f4d036d8c5fe8023920fcdfd43f39
wxDialog can be stack allocated if opened with ShowModal(), but not with
Show(). Go back to wx_ptr for those that are opened with Show().
Remove unnecessary wx_ptr2025-02-09T16:29:16ZCarl Hetheringtoncth@carlh.net2025-02-09T01:06:04Zurn:sha1:d0308d53dd9f4d036d8c5fe8023920fcdfd43f39
It was only ever used for wxDialog subclasses, which can be
stack-allocated.
Put film_util methods in a namespace.2025-01-14T22:51:12ZCarl Hetheringtoncth@carlh.net2025-01-14T09:39:12Zurn:sha1:faa7efd869e11c6e0ec4f00da8e1ad99415c709bMerge branch 'main' into v2.17.x2024-11-20T22:13:38ZCarl Hetheringtoncth@carlh.net2024-11-20T22:13:38Zurn:sha1:89a7c0c5ea6205c4f19be9c7e3186b4d7ad1e44cFix multi-selection of content.2024-11-17T13:46:52ZCarl Hetheringtoncth@carlh.net2024-11-17T13:38:20Zurn:sha1:d51d330f78f11160509a764060ecc17a731e97cf
In order to add ellipsizing of text we're using this "mode" of wxListCtrl where
`OnGetItemAttr` is called to get information about an item. This was intended
to support very long lists, so that the whole list does not have to be given
to wxListCtrl.
However, in this mode, multiple selections (e.g. shift-click) are just not
reported. There's a 13-year-old wxWidgets bug to fix this:
https://github.com/wxWidgets/wxWidgets/issues/4541
and there is mentioned the workaround applied here - listen to the focused
event. We have to delay it because on macOS (at least) the selection state
is not updated until after the focused event.
Remove use of wxT in favour of char_to_wx().2024-09-03T15:02:24ZCarl Hetheringtoncth@carlh.net2024-08-09T22:41:52Zurn:sha1:c97de27f9c5364b6f126016c5e1f31a76d5ce565
The wxWidgets docs advise against its use these days.
Remove some implicit conversions between wxString and std::string.2024-09-03T15:02:24ZCarl Hetheringtoncth@carlh.net2024-08-09T00:02:20Zurn:sha1:5527bdb269e355ca95aa91fe3907bfef0ef17ff4