From e14f261a6a0ec1c3047db37ce55655fa095a6b08 Mon Sep 17 00:00:00 2001 From: Carl Hetherington Date: Sun, 3 Mar 2019 23:59:58 +0000 Subject: [PATCH 1/1] Fix non-movement of position slider and label after dragging the position slider on OS X (probably #1452). --- src/wx/controls.cc | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/src/wx/controls.cc b/src/wx/controls.cc index 5e68baaa0..3e0666460 100644 --- a/src/wx/controls.cc +++ b/src/wx/controls.cc @@ -120,6 +120,13 @@ Controls::Controls (wxWindow* parent, shared_ptr viewer, bool editor _slider->Bind (wxEVT_SCROLL_PAGEUP, boost::bind(&Controls::slider_moved, this, true)); _slider->Bind (wxEVT_SCROLL_PAGEDOWN, boost::bind(&Controls::slider_moved, this, true)); _slider->Bind (wxEVT_SCROLL_CHANGED, boost::bind(&Controls::slider_released, this)); +#ifdef DCPOMATIC_OSX + /* _CHANGED is not received on OS X (at least, not when the + slider is dragged), so use this instead. Perhaps all + platforms could just use _THUMBRELEASE. + */ + _slider->Bind (wxEVT_SCROLL_THUMBRELEASE, boost::bind(&Controls::slider_released, this)); +#endif _rewind_button->Bind (wxEVT_LEFT_DOWN, boost::bind(&Controls::rewind_clicked, this, _1)); _back_button->Bind (wxEVT_LEFT_DOWN, boost::bind(&Controls::back_clicked, this, _1)); _forward_button->Bind (wxEVT_LEFT_DOWN, boost::bind(&Controls::forward_clicked, this, _1)); -- 2.30.2